This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[PATCH] Add Cortex-A15 tuning to gcc.dg/uninit-pred-8_a.c
- From: Kyrill Tkachov <kyrylo dot tkachov at foss dot arm dot com>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Cc: 'Jakub Jelinek' <jakub at redhat dot com>
- Date: Thu, 12 Jan 2017 16:56:11 +0000
- Subject: [PATCH] Add Cortex-A15 tuning to gcc.dg/uninit-pred-8_a.c
- Authentication-results: sourceware.org; auth=none
Hi all,
As discussed in the PR it's better to pin down the tuning for arm targets on this test so that the BRANCH_COST remains constant.
This makes the test more stable across toolchains configured for different default CPUs.
Ok to apply?
Thanks,
Kyrill
2017-01-12 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
PR tree-optimization/78319
* gcc.dg/uninit-pred-8_a.c: Add -mtune=cortex-a15 for arm.
Remove xfail.
diff --git a/gcc/testsuite/gcc.dg/uninit-pred-8_a.c b/gcc/testsuite/gcc.dg/uninit-pred-8_a.c
index c45fba0..c4cdf48 100644
--- a/gcc/testsuite/gcc.dg/uninit-pred-8_a.c
+++ b/gcc/testsuite/gcc.dg/uninit-pred-8_a.c
@@ -1,6 +1,8 @@
/* { dg-do compile } */
/* { dg-options "-Wuninitialized -O2" } */
+/* Pick a particular tuning to pin down BRANCH_COST. */
+/* { dg-additional-options "-mtune=cortex-a15" { target arm*-*-* } } */
int g;
void bar();
@@ -16,9 +18,8 @@ int foo (int n, int l, int m, int r)
if (m) g++;
else bar();
- /* marking this test as xfail on arm-none-eabi, see PR78319. */
if ( n || m || r || l)
- blah(v); /* { dg-bogus "uninitialized" "bogus warning" { xfail arm-none-eabi } } */
+ blah(v); /* { dg-bogus "uninitialized" "bogus warning" } */
if ( n )
blah(v); /* { dg-bogus "uninitialized" "bogus warning" } */