* New gas testsuite failures for LDXTA test
@ 2016-07-15 9:10 Nick Clifton
2016-07-18 15:28 ` Jose E. Marchesi
0 siblings, 1 reply; 7+ messages in thread
From: Nick Clifton @ 2016-07-15 9:10 UTC (permalink / raw)
To: jose.marchesi; +Cc: binutils
Hi Jose,
I am seeing some new gas testsuite failures due to your ldxta test:
Checking Binutils in: sparc-aout ... GAS: 26 LD: 1 done
GAS REGRESSION: sparc LDTXA
Checking Binutils in: sparc-coff ... GAS: 22 done
GAS REGRESSION: sparc LDTXA
Please could you have a look at these ? It is OK if you want to skip
the test for these targets, but it would be good if they could be
fixed so that they run properly.
Cheers
Nick
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: New gas testsuite failures for LDXTA test
2016-07-15 9:10 New gas testsuite failures for LDXTA test Nick Clifton
@ 2016-07-18 15:28 ` Jose E. Marchesi
2016-07-18 15:45 ` Nick Clifton
0 siblings, 1 reply; 7+ messages in thread
From: Jose E. Marchesi @ 2016-07-18 15:28 UTC (permalink / raw)
To: Nick Clifton; +Cc: binutils
Nick.
I am seeing some new gas testsuite failures due to your ldxta test:
Checking Binutils in: sparc-aout ... GAS: 26 LD: 1 done
GAS REGRESSION: sparc LDTXA
Checking Binutils in: sparc-coff ... GAS: 22 done
GAS REGRESSION: sparc LDTXA
Please could you have a look at these ? It is OK if you want to skip
the test for these targets, but it would be good if they could be
fixed so that they run properly.
Looking a it. Sorry for the regression.
Could I have a copy of the script(s) used to detect regressions in all
targets automatically?
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: New gas testsuite failures for LDXTA test
2016-07-18 15:28 ` Jose E. Marchesi
@ 2016-07-18 15:45 ` Nick Clifton
2016-07-18 15:57 ` Jose E. Marchesi
2016-07-27 14:13 ` [PATCH] gas: avoid spurious failures in non-ELF targets in the SPARC testsuite Jose E. Marchesi
0 siblings, 2 replies; 7+ messages in thread
From: Nick Clifton @ 2016-07-18 15:45 UTC (permalink / raw)
To: Jose E. Marchesi; +Cc: binutils
Hi Jose,
> Could I have a copy of the script(s) used to detect regressions in all
> targets automatically?
Already posted to the list:
https://sourceware.org/ml/binutils/2016-07/msg00194.html
Cheers
Nick
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: New gas testsuite failures for LDXTA test
2016-07-18 15:45 ` Nick Clifton
@ 2016-07-18 15:57 ` Jose E. Marchesi
2016-07-27 14:13 ` [PATCH] gas: avoid spurious failures in non-ELF targets in the SPARC testsuite Jose E. Marchesi
1 sibling, 0 replies; 7+ messages in thread
From: Jose E. Marchesi @ 2016-07-18 15:57 UTC (permalink / raw)
To: Nick Clifton; +Cc: binutils
> Could I have a copy of the script(s) used to detect regressions in all
> targets automatically?
Already posted to the list:
https://sourceware.org/ml/binutils/2016-07/msg00194.html
Sweet! Thanks :)
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] gas: avoid spurious failures in non-ELF targets in the SPARC testsuite.
2016-07-18 15:45 ` Nick Clifton
2016-07-18 15:57 ` Jose E. Marchesi
@ 2016-07-27 14:13 ` Jose E. Marchesi
2016-07-28 7:59 ` Nick Clifton
1 sibling, 1 reply; 7+ messages in thread
From: Jose E. Marchesi @ 2016-07-27 14:13 UTC (permalink / raw)
To: Nick Clifton; +Cc: binutils
Hi people!
Many of the existing sparc tests fail in non-ELF targets (coff and
a.out) due to spurious differences in the expected results:
- Unlike ELF, a.out text sections are aligned to 2**3 and padded
accordingly. The padding instruction is a `nop' (01 00 00 00).
- Likewise, coff text sections are also aligned to 2**3 and padded
accordingly. However, the padding instruction in these targets is an
`illtrap 0' (00 00 00 00).
- Unlike ELF, a.out and coff binaries don't contain hardware
capabilities bits that could be used by BFD to determine the opcodes
architecture corresponding to the instructions encoded in the objects
(v9, v9a, v9b, v9c, etc). Consequently, in both a.out and coff tests
we would need to pass proper `-m sparc:vXXX' options when invoking
objdump before comparing results.
In order to fix these issues, the most obvious solution would be to have
three variants of .d files per impacted test. For example, for save.d
we would have: save-elf.d, save-aout.d and save-coff.d. Using the
`#source' directive, a single save.s file would provide the input for
all of them. However, this approach has the following problems:
- The #target and #notarget .d directives are very limited: they use
globs instead of regular expressions, and thus it is not possible (or
too messy) to use them to discriminate between elf, coff and a.out
sparc targets.
- It adds little or no value to have variants of all these tests for all
the target types, and it would be a burden to maintain them. Actually
the features tested in the spuriously failing tests (relatively modern
sparc instructions, registers and asis) are not really found in
running coff or a.out sparc systems.
This patch changes sparc.exp so it will run these tests only in
ELF-targets, using the more standard `is_elf_format' from
binutils-common.exp instead of the ad-hoc (and less convenient, as it
must be called before _every_ single elf-only test) sparc_elf_setup.
Incidentally, the patch also fixes the #name entry for save-args.d.
Tested in sparc*-*-linux-gnu, sparc-aout and sparc-coff targets.
Ok for master?
gas/ChangeLog:
2016-07-27 Jose E. Marchesi <jose.marchesi@oracle.com>
* testsuite/gas/sparc/sparc.exp: Use is_elf_format to discriminate
ELF targets.
Run natural, natural-32, pr4587, ticc-imm-reg, v8-movwr-imm,
pause, save-args, cbcond, cfr, crypto edge, flush, hpcvis3, ima,
ld_st_fsr, ldtw_sttw, ldd_std, ldx_stx, ldx_efsr, mwait, mcdper,
sparc5vis4, xcrypto, v9branch1 and imm-plus-rreg only in ELF
targets.
(sparc_elf_setup): Delete.
* testsuite/gas/sparc/save-args.d: Fix a copy-paste typo in the
test's #name entry.
diff --git a/gas/testsuite/gas/sparc/save-args.d b/gas/testsuite/gas/sparc/save-args.d
index 19e6d72..af21100 100644
--- a/gas/testsuite/gas/sparc/save-args.d
+++ b/gas/testsuite/gas/sparc/save-args.d
@@ -1,6 +1,6 @@
#as: -Av8
#objdump: -dr
-#name: software traps
+#name: save args
.*: +file format .*
diff --git a/gas/testsuite/gas/sparc/sparc.exp b/gas/testsuite/gas/sparc/sparc.exp
index f0a129d..8cb0904 100644
--- a/gas/testsuite/gas/sparc/sparc.exp
+++ b/gas/testsuite/gas/sparc/sparc.exp
@@ -31,25 +31,47 @@ proc gas_64_check { } {
return [regexp "elf64\[_-\]sparc" [lindex $status 1]]
}
-proc sparc_elf_setup { } {
- setup_xfail "sparc*-*-*aout*" "sparc*-*-sunos4*"
- setup_xfail "sparc*-fujitsu-none" "sparc*-*-*n*bsd*"
- setup_xfail "sparc*-*-coff" "sparc*-*-lynxos*"
- clear_xfail "sparc64*-*-*n*bsd*"
- clear_xfail "sparc*-*-netbsdelf*"
-}
-
if [istarget sparc*-*-*] {
- run_dump_test "synth"
- # The next four tests are ELF only.
- sparc_elf_setup
- run_dump_test "unalign"
- sparc_elf_setup
- run_dump_test "pcrel"
- sparc_elf_setup
- run_dump_test "plt"
- sparc_elf_setup
- run_dump_test "gotop32"
+
+ if [is_elf_format] {
+ # The next five tests are ELF only.
+ run_dump_test "unalign"
+ run_dump_test "pcrel"
+ run_dump_test "plt"
+ run_dump_test "gotop32"
+ run_dump_test "pc2210"
+ # The features tested in the tests below are not ELF-specific.
+ # However, these features are not present in practice in COFF
+ # and a.out targets. Running them only in ELF targets allows
+ # us to don't have to replicate pattern test files as that
+ # would bring no practical advantage.
+ run_dump_test "natural"
+ run_dump_test "natural-32"
+ run_list_test "pr4587" ""
+ run_dump_test "ticc-imm-reg"
+ run_dump_test "v8-movwr-imm"
+ run_dump_test "pause"
+ run_dump_test "save-args"
+ run_dump_test "cbcond"
+ run_dump_test "cfr"
+ run_dump_test "crypto"
+ run_dump_test "edge"
+ run_dump_test "flush"
+ run_dump_test "hpcvis3"
+ run_dump_test "ima"
+ run_dump_test "ld_st_fsr"
+ run_dump_test "ldtw_sttw"
+ run_dump_test "ldd_std"
+ run_dump_test "ldx_stx"
+ run_dump_test "ldx_efsr"
+ run_dump_test "mwait"
+ run_dump_test "mcdper"
+ run_dump_test "sparc5vis4"
+ run_dump_test "xcrypto"
+ run_dump_test "v9branch1"
+ run_dump_test "imm-plus-rreg"
+ }
+
if [gas_64_check] {
run_dump_test "asi"
run_dump_test "membar"
@@ -68,39 +90,12 @@ if [istarget sparc*-*-*] {
run_dump_test "plt64"
run_dump_test "gotop64"
}
- run_dump_test "imm-plus-rreg"
- run_dump_test "ticc-imm-reg"
- run_dump_test "v8-movwr-imm"
- run_dump_test "save-args"
- run_dump_test "v9branch1"
+
+ run_dump_test "synth"
run_dump_test "v9branch2"
run_dump_test "v9branch3"
run_dump_test "v9branch4"
run_dump_test "v9branch5"
- run_dump_test "pc2210"
- run_dump_test "hpcvis3"
- run_dump_test "ima"
- run_dump_test "crypto"
- run_dump_test "xcrypto"
- run_dump_test "cbcond"
- run_dump_test "pause"
- run_dump_test "cfr"
- run_dump_test "ldtw_sttw"
- run_dump_test "ldtxa"
- run_dump_test "ldd_std"
- run_dump_test "ldx_stx"
- run_dump_test "ldx_efsr"
- run_dump_test "ld_st_fsr"
- run_dump_test "edge"
- run_dump_test "flush"
- run_dump_test "mwait"
- run_dump_test "mcdper"
- run_dump_test "sparc5vis4"
- run_dump_test "natural"
- run_dump_test "natural-32"
-
- run_list_test "pr4587" ""
-
run_dump_test "pr19910-1"
run_list_test "pr19910-2"
}
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] gas: avoid spurious failures in non-ELF targets in the SPARC testsuite.
2016-07-27 14:13 ` [PATCH] gas: avoid spurious failures in non-ELF targets in the SPARC testsuite Jose E. Marchesi
@ 2016-07-28 7:59 ` Nick Clifton
2016-07-29 7:15 ` Jose E. Marchesi
0 siblings, 1 reply; 7+ messages in thread
From: Nick Clifton @ 2016-07-28 7:59 UTC (permalink / raw)
To: Jose E. Marchesi; +Cc: binutils
Hi Jose,
> 2016-07-27 Jose E. Marchesi <jose.marchesi@oracle.com>
>
> * testsuite/gas/sparc/sparc.exp: Use is_elf_format to discriminate
> ELF targets.
> Run natural, natural-32, pr4587, ticc-imm-reg, v8-movwr-imm,
> pause, save-args, cbcond, cfr, crypto edge, flush, hpcvis3, ima,
> ld_st_fsr, ldtw_sttw, ldd_std, ldx_stx, ldx_efsr, mwait, mcdper,
> sparc5vis4, xcrypto, v9branch1 and imm-plus-rreg only in ELF
> targets.
> (sparc_elf_setup): Delete.
> * testsuite/gas/sparc/save-args.d: Fix a copy-paste typo in the
> test's #name entry.
Approved - please apply.
I do not think that we need to worry too much about COFF and AOUT based
targets missing out on some of the tests...
Cheers
Nick
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] gas: avoid spurious failures in non-ELF targets in the SPARC testsuite.
2016-07-28 7:59 ` Nick Clifton
@ 2016-07-29 7:15 ` Jose E. Marchesi
0 siblings, 0 replies; 7+ messages in thread
From: Jose E. Marchesi @ 2016-07-29 7:15 UTC (permalink / raw)
To: Nick Clifton; +Cc: binutils
> 2016-07-27 Jose E. Marchesi <jose.marchesi@oracle.com>
>
> * testsuite/gas/sparc/sparc.exp: Use is_elf_format to discriminate
> ELF targets.
> Run natural, natural-32, pr4587, ticc-imm-reg, v8-movwr-imm,
> pause, save-args, cbcond, cfr, crypto edge, flush, hpcvis3, ima,
> ld_st_fsr, ldtw_sttw, ldd_std, ldx_stx, ldx_efsr, mwait, mcdper,
> sparc5vis4, xcrypto, v9branch1 and imm-plus-rreg only in ELF
> targets.
> (sparc_elf_setup): Delete.
> * testsuite/gas/sparc/save-args.d: Fix a copy-paste typo in the
> test's #name entry.
Approved - please apply.
Pushed to master.
Thanks!
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-07-29 7:15 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-07-15 9:10 New gas testsuite failures for LDXTA test Nick Clifton
2016-07-18 15:28 ` Jose E. Marchesi
2016-07-18 15:45 ` Nick Clifton
2016-07-18 15:57 ` Jose E. Marchesi
2016-07-27 14:13 ` [PATCH] gas: avoid spurious failures in non-ELF targets in the SPARC testsuite Jose E. Marchesi
2016-07-28 7:59 ` Nick Clifton
2016-07-29 7:15 ` Jose E. Marchesi
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).