From: Alan Modra Date: Thu, 22 Oct 2020 01:33:00 +0000 (+1030) Subject: [RS6000] Power10 vec-splati-runnable multiple test failures X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=5c1506304c52c29497a12c65734caaaa99996f77;p=gcc.git [RS6000] Power10 vec-splati-runnable multiple test failures FAIL: gcc.target/powerpc/vec-splati-runnable.c 1 blank line(s) in output FAIL: gcc.target/powerpc/vec-splati-runnable.c (test for excess errors) Excess errors: rs6000_emit_xxspltidp_v2df called ... and running the test fails. As the comment says /* Although the instruction says the results are not defined, it does seem to work, at least on Mambo. But no guarentees! */ So the simulator works but not real hardware. gcc/ * config/rs6000/rs6000.c (rs6000_emit_xxspltidp_v2df): Delete debug printf. Remove trailing ".\n" from inform message. Break long line. gcc/testsuite/ * gcc.target/powerpc/vec-splati-runnable.c: Don't abort on undefined output. --- diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c index d05b85c3cdb..4978e8b4abf 100644 --- a/gcc/config/rs6000/rs6000.c +++ b/gcc/config/rs6000/rs6000.c @@ -27047,11 +27047,10 @@ rs6000_const_f32_to_i32 (rtx operand) void rs6000_emit_xxspltidp_v2df (rtx dst, long value) { - printf("rs6000_emit_xxspltidp_v2df called %ld\n", value); - printf("rs6000_emit_xxspltidp_v2df called 0x%lx\n", value); if (((value & 0x7F800000) == 0) && ((value & 0x7FFFFF) != 0)) inform (input_location, - "the result for the xxspltidp instruction is undefined for subnormal input values.\n"); + "the result for the xxspltidp instruction " + "is undefined for subnormal input values"); emit_insn( gen_xxspltidp_v2df_inst (dst, GEN_INT (value))); } diff --git a/gcc/testsuite/gcc.target/powerpc/vec-splati-runnable.c b/gcc/testsuite/gcc.target/powerpc/vec-splati-runnable.c index afb0bfdef3a..e5a4935644f 100644 --- a/gcc/testsuite/gcc.target/powerpc/vec-splati-runnable.c +++ b/gcc/testsuite/gcc.target/powerpc/vec-splati-runnable.c @@ -5,7 +5,7 @@ #define DEBUG 0 -#ifdef DEBUG +#if DEBUG #include #endif @@ -100,7 +100,7 @@ main (int argc, char *argv []) printf(" vresult_d[%i] = %e, expected_vresult_d[%i] = %e\n", i, vresult_d[i], i, expected_vresult_d[i]); #else - abort(); + ; #endif }