diff options
author | Alan Modra <amodra@gmail.com> | 2018-07-10 23:13:10 +0930 |
---|---|---|
committer | Alan Modra <amodra@gmail.com> | 2018-07-10 23:58:29 +0930 |
commit | 624940689a97573ed1e24ae054dcb2fd6906f557 (patch) | |
tree | d2a1b7d952b229e307d18eaebe01f35dfb8a0c29 /ld/testsuite/ld-gc | |
parent | 3541979fff2861c9537f299e31ac17a436bf1228 (diff) | |
download | gdb-624940689a97573ed1e24ae054dcb2fd6906f557.zip gdb-624940689a97573ed1e24ae054dcb2fd6906f557.tar.gz gdb-624940689a97573ed1e24ae054dcb2fd6906f557.tar.bz2 |
ld testsuite use of notarget
Much of the time it is better to use xfail rather than notarget to
hide a testsuite failure no one is planning on fixing.
1) When a target is fixed, an inappropriate xfail results in XPASS
whereas notarget is silent. Leaving in an inappropriate notarget
entry risks the bug reappearing later, silently.
2) Tests might be used as a template for a new test, carrying over
notarget and/or xfail. Inappropriate notarget entries are silent.
3) An xfailed test that mysteriously XPASSes might actually indicate a
bug.
* testsuite/ld-elf/attributes.d: Use xfail rather than notarget.
* testsuite/ld-elf/hash.d: Likewise.
* testsuite/ld-elf/orphan-10.d: Likewise.
* testsuite/ld-elf/overlay.d: Likewise.
* testsuite/ld-elf/pr14926.d: Likewise.
* testsuite/ld-elf/pr17550d.d: Likewise.
* testsuite/ld-elf/pr19539.d: Likewise.
* testsuite/ld-elf/pr22319.d: Likewise.
* testsuite/ld-misc/defsym1.d: Likewise.
* testsuite/ld-pe/pr19803.d: Likewise.
* testsuite/ld-scripts/assign-loc.d: Likewise.
* testsuite/ld-scripts/defined4.d: Likewise.
* testsuite/ld-scripts/empty-address-1.d: Likewise.
* testsuite/ld-scripts/empty-address-2a.d: Likewise.
* testsuite/ld-scripts/empty-address-2b.d: Likewise.
* testsuite/ld-scripts/fill.d: Likewise.
* testsuite/ld-scripts/fill16.d: Likewise.
* testsuite/ld-scripts/pr14962.d: Likewise.
* testsuite/ld-scripts/pr14962-2.d: Likewise.
* testsuite/ld-unique/pr21529.d: Likewise.
* testsuite/ld-elf/compress1a.d: Likewise, and adjust targets.
* testsuite/ld-elf/compressed1a.d: Likewise.
* testsuite/ld-elf/compressed1d.d: Likewise.
* testsuite/ld-elf/compressed1e.d: Likewise.
* testsuite/ld-elf/eh5.d: Likewise.
* testsuite/ld-elf/group8a.d: Likewise.
* testsuite/ld-elf/linkonce2.d: Likewise.
* testsuite/ld-elf/orphan-9.d: Likewise.
* testsuite/ld-elf/pr17550a.d: Likewise.
* testsuite/ld-elf/pr17550b.d: Likewise.
* testsuite/ld-elf/pr17550c.d: Likewise.
* testsuite/ld-elf/pr19162.d: Likewise.
* testsuite/ld-elf/pr21884.d: Likewise.
* testsuite/ld-gc/pr19167.d: Likewise.
* testsuite/ld-gc/pr20022.d: Likewise.
* testsuite/ld-gc/start.d: Likewise.
* testsuite/ld-elf/orphan-11.d: Delete notarget.
* testsuite/ld-elf/orphan-12.d: Likewise.
* testsuite/ld-gc/stop.d: Likewise.
* testsuite/ld-scripts/empty-address-3a.d: Likewise.
* testsuite/ld-scripts/empty-address-3b.d: Likewise.
* testsuite/ld-ifunc/pr17154-x86-64-now.d: Delete duplicate notarget.
* testsuite/ld-scripts/section-match-1.d: Comment fix.
Diffstat (limited to 'ld/testsuite/ld-gc')
-rw-r--r-- | ld/testsuite/ld-gc/pr19167.d | 2 | ||||
-rw-r--r-- | ld/testsuite/ld-gc/pr20022.d | 2 | ||||
-rw-r--r-- | ld/testsuite/ld-gc/start.d | 2 | ||||
-rw-r--r-- | ld/testsuite/ld-gc/stop.d | 1 |
4 files changed, 3 insertions, 4 deletions
diff --git a/ld/testsuite/ld-gc/pr19167.d b/ld/testsuite/ld-gc/pr19167.d index 578646e..9555cc8 100644 --- a/ld/testsuite/ld-gc/pr19167.d +++ b/ld/testsuite/ld-gc/pr19167.d @@ -3,7 +3,7 @@ #ld: --gc-sections -e _start #objdump: -s -j _foo #target: *-*-linux* *-*-gnu* arm*-*-uclinuxfdpiceabi -#notarget: frv-*-linux* metag-*-linux* +#xfail: frv-*-* metag-*-* #... Contents of section _foo: diff --git a/ld/testsuite/ld-gc/pr20022.d b/ld/testsuite/ld-gc/pr20022.d index 6600f4a..681c68c 100644 --- a/ld/testsuite/ld-gc/pr20022.d +++ b/ld/testsuite/ld-gc/pr20022.d @@ -2,7 +2,7 @@ #ld: --gc-sections -e _start tmpdir/pr20022.so #readelf: -SsW #target: *-*-linux* *-*-gnu* arm*-*-uclinuxfdpiceabi -#notarget: frv-*-linux* metag-*-linux* mips*-*-* +#xfail: frv-*-* metag-*-* mips*-*-* # Skip on targets without dynamic relocations in .text section. #... diff --git a/ld/testsuite/ld-gc/start.d b/ld/testsuite/ld-gc/start.d index b848623..85b0612 100644 --- a/ld/testsuite/ld-gc/start.d +++ b/ld/testsuite/ld-gc/start.d @@ -2,7 +2,7 @@ #ld: --gc-sections -e _start #nm: -n #target: *-*-linux* *-*-gnu* arm*-*-uclinuxfdpiceabi -#notarget: frv-*-linux* metag-*-linux* +#xfail: frv-*-* metag-*-* #... [0-9a-f]+ D +__start__foo diff --git a/ld/testsuite/ld-gc/stop.d b/ld/testsuite/ld-gc/stop.d index 84dc465..ebd3597 100644 --- a/ld/testsuite/ld-gc/stop.d +++ b/ld/testsuite/ld-gc/stop.d @@ -2,7 +2,6 @@ #ld: --gc-sections -e _start #nm: -n #target: *-*-linux* *-*-gnu* arm*-*-uclinuxfdpiceabi -#notarget: frv-*-linux* metag-*-linux* #failif #... |