* [PATCH][gdb/testsuite] Workaround unnecessary .s file with gfortran 4.8
@ 2022-06-10 7:11 Tom de Vries
2022-06-28 15:11 ` [committed][gdb/testsuite] " Tom de Vries
0 siblings, 1 reply; 2+ messages in thread
From: Tom de Vries @ 2022-06-10 7:11 UTC (permalink / raw)
To: gdb-patches
Hi,
After running test-case gdb.fortran/namelist.exp with gfortran 4.8.5, I'm left
with:
...
$ git sti
On branch master
Your branch is up to date with 'origin/master'.
Untracked files:
(use "git add <file>..." to include in what will be committed)
gdb/testsuite/lib/compiler.s
nothing added to commit but untracked files present (use "git add" to track)
...
We're running into PR gcc/60447, which was fixed in gcc 4.9.0.
Workaround this by first copying the source file to the temp dir, such that
the .s file is left there instead:
...
$ ls build/gdb/testsuite/temp/<runtest pid>/
compiler.c compiler.F90 compiler.s
...
Tested on x86_64-linux.
Any comments?
Thanks,
- Tom
[gdb/testsuite] Workaround unnecessary .s file with gfortran 4.8
---
gdb/testsuite/lib/gdb.exp | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp
index 41e1d549510..d1a4f3720c7 100644
--- a/gdb/testsuite/lib/gdb.exp
+++ b/gdb/testsuite/lib/gdb.exp
@@ -4139,6 +4139,12 @@ proc get_compiler_info {{language "c"}} {
set cppout [read $file]
close $file
} else {
+ # Copy $ifile to temp dir, to work around PR gcc/60447. This will leave the
+ # superfluous .s file in the temp dir instead of in the source dir.
+ set tofile [file tail $ifile]
+ set tofile [standard_temp_file $tofile]
+ file copy -force $ifile $tofile
+ set ifile $tofile
set cppout [ gdb_compile "${ifile}" "" preprocess [list "$language" quiet getting_compiler_info] ]
}
eval log_file $saved_log
^ permalink raw reply [flat|nested] 2+ messages in thread
* [committed][gdb/testsuite] Workaround unnecessary .s file with gfortran 4.8
2022-06-10 7:11 [PATCH][gdb/testsuite] Workaround unnecessary .s file with gfortran 4.8 Tom de Vries
@ 2022-06-28 15:11 ` Tom de Vries
0 siblings, 0 replies; 2+ messages in thread
From: Tom de Vries @ 2022-06-28 15:11 UTC (permalink / raw)
To: gdb-patches
On 6/10/22 09:11, Tom de Vries wrote:
> Hi,
>
> After running test-case gdb.fortran/namelist.exp with gfortran 4.8.5, I'm left
> with:
> ...
> $ git sti
> On branch master
> Your branch is up to date with 'origin/master'.
>
> Untracked files:
> (use "git add <file>..." to include in what will be committed)
> gdb/testsuite/lib/compiler.s
>
> nothing added to commit but untracked files present (use "git add" to track)
> ...
>
> We're running into PR gcc/60447, which was fixed in gcc 4.9.0.
>
> Workaround this by first copying the source file to the temp dir, such that
> the .s file is left there instead:
> ...
> $ ls build/gdb/testsuite/temp/<runtest pid>/
> compiler.c compiler.F90 compiler.s
> ...
>
> Tested on x86_64-linux.
>
> Any comments?
Committed.
Thanks,
- Tom
> [gdb/testsuite] Workaround unnecessary .s file with gfortran 4.8
>
> ---
> gdb/testsuite/lib/gdb.exp | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp
> index 41e1d549510..d1a4f3720c7 100644
> --- a/gdb/testsuite/lib/gdb.exp
> +++ b/gdb/testsuite/lib/gdb.exp
> @@ -4139,6 +4139,12 @@ proc get_compiler_info {{language "c"}} {
> set cppout [read $file]
> close $file
> } else {
> + # Copy $ifile to temp dir, to work around PR gcc/60447. This will leave the
> + # superfluous .s file in the temp dir instead of in the source dir.
> + set tofile [file tail $ifile]
> + set tofile [standard_temp_file $tofile]
> + file copy -force $ifile $tofile
> + set ifile $tofile
> set cppout [ gdb_compile "${ifile}" "" preprocess [list "$language" quiet getting_compiler_info] ]
> }
> eval log_file $saved_log
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-06-28 15:11 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-10 7:11 [PATCH][gdb/testsuite] Workaround unnecessary .s file with gfortran 4.8 Tom de Vries
2022-06-28 15:11 ` [committed][gdb/testsuite] " Tom de Vries
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).