This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[PATCH] Fix new split-1.c testcase
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: gcc-patches at gcc dot gnu dot org
- Cc: Segher Boessenkool <segher at kernel dot crashing dot org>, Ian Lance Taylor <iant at golang dot org>
- Date: Sun, 11 Jun 2017 11:40:49 +0000
- Subject: [PATCH] Fix new split-1.c testcase
- Authentication-results: sourceware.org; auth=none
The new split-1.c testcase fails on targets that do not support split
stack (like 32-bit PowerPC Linux). This patch fixes it by only running
the testcase if split stack is supported. It also adds the reorder
flag to the options, so that the test actually tests what it says it
tests.
Is this okay for trunk?
Segher
2017-06-11 Segher Boessenkool <segher@kernel.crashing.org>
gcc/testsuite/
* gcc.dg/tree-prof/split-1.c: Require effective target split_stack.
Add -freorder-blocks-and-partition to options.
---
gcc/testsuite/gcc.dg/tree-prof/split-1.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/gcc/testsuite/gcc.dg/tree-prof/split-1.c b/gcc/testsuite/gcc.dg/tree-prof/split-1.c
index a42fccf..4de1123 100644
--- a/gcc/testsuite/gcc.dg/tree-prof/split-1.c
+++ b/gcc/testsuite/gcc.dg/tree-prof/split-1.c
@@ -1,7 +1,8 @@
/* Test case that we don't get a link-time error when using
-fsplit-stack with -freorder-blocks-and-partition. */
+/* { dg-require-effective-target split_stack } */
/* { dg-require-effective-target freorder } */
-/* { dg-options "-O2 -fsplit-stack" } */
+/* { dg-options "-O2 -fsplit-stack -freorder-blocks-and-partition" } */
extern unsigned int sleep (unsigned int);
--
1.9.3