tests: make fp-test less chatty when running from test suite

As we like to run tests under CI with V=1 flags the softfloat tests
can add up to a fair amount of extra log lines. With an update to the
testfloat library we can now call fp-test with the -q flag and reduce
the output to a terse one line per function tested.

  make check-softfloat V=1 | wc -l
  759

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20230228190653.1602033-5-alex.bennee@linaro.org>
This commit is contained in:
Alex Bennée 2023-02-28 19:06:33 +00:00
parent c997068077
commit 6348a546ed
3 changed files with 16 additions and 7 deletions

@ -1 +1 @@
Subproject commit 5a59dcec19327396a011a17fd924aed4fec416b3
Subproject commit 40619cbb3bf32872df8c53cc457039229428a263

View file

@ -106,7 +106,8 @@ static const char commands_string[] =
" -l = thoroughness level (1 (default), 2)\n"
" -r = rounding mode (even (default), zero, down, up, tieaway, odd)\n"
" Set to 'all' to test all rounding modes, if applicable\n"
" -s = stop when a test fails";
" -s = stop when a test fails\n"
" -q = minimise noise when testing, just show each function being tested";
static void usage_complete(int argc, char *argv[])
{
@ -190,9 +191,11 @@ static void do_testfloat(int op, int rmode, bool exact)
ab_f128M_z_bool true_ab_f128M_z_bool;
ab_f128M_z_bool subj_ab_f128M_z_bool;
fputs(">> Testing ", stderr);
verCases_writeFunctionName(stderr);
fputs("\n", stderr);
if (verCases_verbosity) {
fputs(">> Testing ", stderr);
verCases_writeFunctionName(stderr);
fputs("\n", stderr);
}
if (!is_allowed(op, rmode)) {
not_implemented();
@ -837,7 +840,7 @@ static void parse_args(int argc, char *argv[])
int c;
for (;;) {
c = getopt(argc, argv, "he:f:l:r:s");
c = getopt(argc, argv, "he:f:l:r:sq");
if (c < 0) {
break;
}
@ -874,9 +877,15 @@ static void parse_args(int argc, char *argv[])
}
}
break;
/*
* The following flags are declared in testfloat/source/verCases_common.c
*/
case 's':
verCases_errorStop = true;
break;
case 'q':
verCases_verbosity = 0;
break;
case '?':
/* invalid option or missing argument; getopt prints error info */
exit(EXIT_FAILURE);

View file

@ -609,7 +609,7 @@ softfloat_tests = {
# The full test suite can take a bit of time, default to a quick run
# "-l 2 -r all" can take more than a day for some operations and is best
# run manually
fptest_args = ['-s', '-l', '1']
fptest_args = ['-q', '-s', '-l', '1']
fptest_rounding_args = ['-r', 'all']
# Conversion Routines: