diff options
| author | Mike Frysinger <vapier@gentoo.org> | 2021-01-05 01:42:20 -0500 |
|---|---|---|
| committer | Mike Frysinger <vapier@gentoo.org> | 2021-01-15 01:34:57 -0500 |
| commit | 7cf91a24819bc02a7e83e0c2a1bfc69bf1ff513c (patch) | |
| tree | 103449b784b4830bfde9af46311f8c587afd83ef /sim/testsuite/ChangeLog | |
| parent | 37a9c3a53e3705c5120506e457455642702845e6 (diff) | |
| download | gdb-7cf91a24819bc02a7e83e0c2a1bfc69bf1ff513c.zip gdb-7cf91a24819bc02a7e83e0c2a1bfc69bf1ff513c.tar.gz gdb-7cf91a24819bc02a7e83e0c2a1bfc69bf1ff513c.tar.bz2 | |
sim: m32r: clean up redundant test coverage
The m32r-elf subdir contained three tests:
* exit47: A program to test exit status of 47 from sim.
* hello: Standard "hello world" output program.
* loop: An infinite loop program.
There's already a sim/m32r/hello.ms test that does exactly the same
thing as m32r-elf/hello.s, so we can delete that.
The loop.s test is never referenced anywhere, and is all of 2 lines.
Anyone who really needs a while(1); test case and re-implement it
themselves locally.
That leaves the single exit47 test. Now that the sim test harness
supports testing for custom exit status, we can easily move that to
sim/m32r/exit47.ms to maintain test coverage.
The remaining differences between m32r-elf & sim/m32r are:
* m32r-elf/ runs for m32r-*-elf while sim/m32r/ runs for m32r*-*-*.
* m32r-elf/ runs "*.s" files while sim/m32r/ runs "*.ms" files.
On closer inspection, these are also meaningless distinctions:
* There is nothing specific to the tests that require an *-elf
target. Normally that would mean newlib+libgloss type stuff,
but there's no such requirement in m32r-elf/.
* The ".s" suffix is the standard "this is an assembly file"
suffix. Turns out ".ms" is just how sim/m32r/ (and a few other
CGEN based targets) categorize/bucket test cases. It simply
means "miscellaneous .s" as in "this is an assembly file, and
run/bucket its test results in the miscellaneous category".
So moving m32r-elf/exit47.s to sim/m32r/exit47.ms makes sense and
simplifies things quite a bit for the target while also slightly
increasing the coverage for some tuples.
Diffstat (limited to 'sim/testsuite/ChangeLog')
| -rw-r--r-- | sim/testsuite/ChangeLog | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/sim/testsuite/ChangeLog b/sim/testsuite/ChangeLog index 1245ab4..36c5058 100644 --- a/sim/testsuite/ChangeLog +++ b/sim/testsuite/ChangeLog @@ -1,5 +1,11 @@ 2021-01-15 Mike Frysinger <vapier@gentoo.org> + * configure.ac (target): Delete m32r-*-elf case. + * configure: Regenerate. + * m32r-elf/: Delete directory. + +2021-01-15 Mike Frysinger <vapier@gentoo.org> + * lib/sim-defs.exp (sim_run): Delete status and return return_code. (run_sim_test): Define status option. Change sim_run return to return_code. Define status. Log return_code. |