This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[Patch][AARCH64] Fix PR 81643 by modifying test case.
- From: Steve Ellcey <sellcey at cavium dot com>
- To: gcc-patches at gcc dot gnu dot org
- Date: Thu, 10 Aug 2017 10:50:57 -0700
- Subject: [Patch][AARCH64] Fix PR 81643 by modifying test case.
- Authentication-results: sourceware.org; auth=none
- Reply-to: sellcey at cavium dot com
I would like to modify gcc.target/aarch64/long_branch_1.c to no longer
look for an Ltb label. After a patch to c-typeck.c to add a predict
statement to GOTO statements, GCC no longer generates one of the types
of long branch that this test is checking for.
I tried modifying the test and creating a new test to generate this label
but was unsucessful, I also added in an abort statement that would trigger
when GCC tried to generate this label and then ran a bootstrap GCC build
and a full testsuite run. The abort never triggered. There may still
be some way to generate this label but I don't think it is worth the time
to find it and so I would like to just remove the check so that the
testcase no longer fails.
OK for checkin? Tested on aarch64.
Steve Ellcey
sellcey@cavium.com
2017-08-10 Steve Ellcey <sellcey@cavium.com>
PR target/81643
* gcc.target/aarch64/long_branch_1.c: Remove Ltb check.
diff --git a/gcc/testsuite/gcc.target/aarch64/long_branch_1.c b/gcc/testsuite/gcc.target/aarch64/long_branch_1.c
index 46f500d..9e0f423 100644
--- a/gcc/testsuite/gcc.target/aarch64/long_branch_1.c
+++ b/gcc/testsuite/gcc.target/aarch64/long_branch_1.c
@@ -88,4 +88,3 @@ start3:
/* { dg-final { scan-assembler "Lbcond" } } */
/* { dg-final { scan-assembler "Lcb" } } */
-/* { dg-final { scan-assembler "Ltb" } } */