This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[committed] Fix x86_64-ism in selftest (PR bootstrap/78705)
- From: David Malcolm <dmalcolm at redhat dot com>
- To: gcc-patches at gcc dot gnu dot org
- Cc: David Malcolm <dmalcolm at redhat dot com>
- Date: Tue, 6 Dec 2016 21:04:12 -0500
- Subject: [committed] Fix x86_64-ism in selftest (PR bootstrap/78705)
- Authentication-results: sourceware.org; auth=none
In r243317 I accidentally introduced an assumption in a selftest
in i386.c that Pmode == DImode. This patch conditionalizes the
overzealous string comparison to avoid breaking the build on
32-bit i386.c targets.
Committed to trunk as r243332 as obvious.
Sorry about the breakage.
gcc/ChangeLog:
PR bootstrap/78705
* config/i386/i386.c (selftest::ix86_test_dumping_memory_blockage):
Conditionalize the string comparison on Pmode == DImode.
---
gcc/config/i386/i386.c | 14 ++++++++------
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
index 2e6be02..1cd1cd8 100644
--- a/gcc/config/i386/i386.c
+++ b/gcc/config/i386/i386.c
@@ -51190,12 +51190,14 @@ ix86_test_dumping_memory_blockage ()
/* Verify that the repeated references to the SCRATCH show use
reuse IDS. The first should be prefixed with a reuse ID,
- and the second should be dumped as a "reuse_rtx" of that ID. */
- ASSERT_RTL_DUMP_EQ_WITH_REUSE
- ("(cinsn 1 (set (mem/v:BLK (0|scratch:DI) [0 A8])\n"
- " (unspec:BLK [\n"
- " (mem/v:BLK (reuse_rtx 0) [0 A8])\n"
- " ] UNSPEC_MEMORY_BLOCKAGE)))\n", pat, &r);
+ and the second should be dumped as a "reuse_rtx" of that ID.
+ The expected string assumes Pmode == DImode. */
+ if (Pmode == DImode)
+ ASSERT_RTL_DUMP_EQ_WITH_REUSE
+ ("(cinsn 1 (set (mem/v:BLK (0|scratch:DI) [0 A8])\n"
+ " (unspec:BLK [\n"
+ " (mem/v:BLK (reuse_rtx 0) [0 A8])\n"
+ " ] UNSPEC_MEMORY_BLOCKAGE)))\n", pat, &r);
}
/* Run all target-specific selftests. */
--
1.8.5.3