Hi Marek,
Thank you for pointing out. Yes I should add.
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 82e538e..f6bcb07 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2016-03-01 Venkataramanan Kumar <venkataramanan.ku...@amd.com>
+
+ PR tree-optimization/68621
+ * gcc.dg/tree-ssa/ifc-8.c: Adjust test.
+
2016-02-29 Bill Schmidt <wschm...@linux.vnet.ibm.com>
PR target/70011
diff --git a/gcc/testsuite/gcc.dg/tree-ssa/ifc-8.c
b/gcc/testsuite/gcc.dg/tree-ssa/ifc-8.c
index 89a3410..81a4075 100644
--- a/gcc/testsuite/gcc.dg/tree-ssa/ifc-8.c
+++ b/gcc/testsuite/gcc.dg/tree-ssa/ifc-8.c
@@ -1,9 +1,10 @@
/* { dg-do compile } */
-/* { dg-options "-Ofast -fdump-tree-ifcvt-details -fno-common
-ftree-loop-if-convert-stores" } */
+/* { dg-options "-Ofast -fdump-tree-ifcvt-details
-ftree-loop-if-convert-stores" } */
+/* { dg-require-visibility "" } */
#define LEN 4096
- __attribute__((aligned (32))) float array[LEN];
+ __attribute__((visibility("hidden"), aligned (32))) float array[LEN] = {};
void test ()
{
Regards,
Venkat.
> -----Original Message-----
> From: Marek Polacek [mailto:pola...@redhat.com]
> Sent: Tuesday, March 01, 2016 10:23 PM
> To: Kumar, Venkataramanan
> Cc: Richard Beiner (richard.guent...@gmail.com); gcc-patches@gcc.gnu.org;
> hjl.to...@gmail.com
> Subject: Re: [PATCH] Fix PR68621
>
> On Tue, Mar 01, 2016 at 04:48:40PM +0000, Kumar, Venkataramanan wrote:
> > /* { dg-do compile } */
> > -/* { dg-options "-Ofast -fdump-tree-ifcvt-details -fno-common -ftree-
> loop-if-convert-stores" } */
> > +/* { dg-options "-Ofast -fdump-tree-ifcvt-details -ftree-loop-if-convert-
> stores" } */
> >
> > #define LEN 4096
> > - __attribute__((aligned (32))) float array[LEN];
> > + __attribute__((visibility("hidden"), aligned (32))) float array[LEN] = {};
>
> Don't you need
> /* { dg-require-visibility "" } */
> if you use __attribute__ ((visibility))?
>
> Marek
