sim: frv: clean up redundant test coverage
authorMike Frysinger <vapier@gentoo.org>
Tue, 5 Jan 2021 07:19:02 +0000 (02:19 -0500)
committerMike Frysinger <vapier@gentoo.org>
Fri, 15 Jan 2021 06:43:47 +0000 (01:43 -0500)
commit89bfc2a429cc536abc019d1d58f786ff95d99571
tree9982a4986fcb03b803c9557a9440c8b76b759fc6
parent7cf91a24819bc02a7e83e0c2a1bfc69bf1ff513c
sim: frv: clean up redundant test coverage

The frv-elf subdir contained five tests:
* cache: A cache test of some sort.
* exit47: A program to test exit status of 47 from sim.
* grloop: Some basic limited loop test program.
* hello: Standard "hello world" output program.
* loop: An infinite loop program.

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.

The cache.s code isn't referenced anywhere because it requires some
custom args to the run program, and when this testcase was added, we
didn't have any support for that.  We do now, so we can add a header
to enable that.  Turns out the code crashes even with those, so turn
around and mark it xfail.  Maybe someone someday will care.

That leaves the small exit47, grloop, and hello tests.  Now that the
sim test harness supports testing for custom exit status, we can move
them all to sim/frv/ to maintain test coverage.

The remaining differences between frv-elf & sim/frv are:
* frv-elf/ runs for frv-*-elf while sim/frv/ runs for frv*-*-*.
* frv-elf/ runs "*.s" files while sim/frv/ only has .cgs and such.

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 frv-elf/.
* The ".s" suffix is the standard "this is an assembly file" suffix.
  Since FRV is a CGEN target, we can reuse the existing convention of
  ".ms" to mean "miscellaneous .s" as in "this is an assembly file,
  and run/bucket its test results in the miscellaneous category".

So moving frv-elf/{cache,exit47,grloop,hello}.s to sim/frv/*.ms makes
sense and simplifies things quite a bit for the target while also
slightly increasing the coverage for some tuples.
18 files changed:
sim/testsuite/ChangeLog
sim/testsuite/configure
sim/testsuite/configure.ac
sim/testsuite/frv-elf/ChangeLog [deleted file]
sim/testsuite/frv-elf/Makefile.in [deleted file]
sim/testsuite/frv-elf/cache.s [deleted file]
sim/testsuite/frv-elf/configure [deleted file]
sim/testsuite/frv-elf/configure.ac [deleted file]
sim/testsuite/frv-elf/exit47.s [deleted file]
sim/testsuite/frv-elf/grloop.s [deleted file]
sim/testsuite/frv-elf/hello.s [deleted file]
sim/testsuite/frv-elf/loop.s [deleted file]
sim/testsuite/sim/frv/ChangeLog
sim/testsuite/sim/frv/cache.ms [new file with mode: 0644]
sim/testsuite/sim/frv/exit47.ms [new file with mode: 0644]
sim/testsuite/sim/frv/grloop.ms [new file with mode: 0644]
sim/testsuite/sim/frv/hello.ms [new file with mode: 0644]
sim/testsuite/sim/frv/misc.exp [new file with mode: 0644]