diff mbox series

[Committed] Avoid FAIL of gcc.target/i386/pr110792.c

Message ID 004b01d9c8b5$465df9e0$d319eda0$@nextmovesoftware.com
State New
Headers show
Series [Committed] Avoid FAIL of gcc.target/i386/pr110792.c | expand

Commit Message

Roger Sayle Aug. 6, 2023, 10:28 p.m. UTC 
My apologies (again), I managed to mess up the 64-bit version of the
test case for PR 110792.  Unlike the 32-bit version, the 64-bit case
contains exactly the same load instructions, just in a different order
making the correct and incorrect behaviours impossible to distinguish
with a scan-assembler-not.  Somewhere between checking that this test
failed in a clean tree without the patch, and getting the escaping
correct, I'd failed to notice that this also FAILs in the patched tree.
Doh!  Instead of removing the test completely, I've left it as a
compilation test.

The original fix is tested by the 32-bit test case.

Committed to mainline as obvious.  Sorry for the inconvenience.


2023-08-06  Roger Sayle  <roger@nextmovesoftware.com>

gcc/testsuite/ChangeLog
        PR target/110792
        * gcc.target/i386/pr110792.c: Remove dg-final scan-assembler-not.
diff mbox series

Patch

diff --git a/gcc/testsuite/gcc.target/i386/pr110792.c b/gcc/testsuite/gcc.target/i386/pr110792.c
index b65125c48b6..eea4e1877db 100644
--- a/gcc/testsuite/gcc.target/i386/pr110792.c
+++ b/gcc/testsuite/gcc.target/i386/pr110792.c
@@ -15,4 +15,3 @@  unsigned __int128 whirl(unsigned char x0)
    asm("":::"memory");
    return tt;
 }
-/* { dg-final { scan-assembler-not "movq\tWHIRL_S\\+8\\(%rdi\\), %rdi" } } */