RE: [PATCH] check alias sets in add_inter_loop_mem_dep (ddg.c)

["Bingfeng Mei" <bmei at broadcom dot com>]

Yes, it seems to be better. Please check in the patch. Thx.

Bingfeng

> -----Original Message-----
> From: Revital1 Eres [mailto:ERES@il.ibm.com] 
> Sent: 22 January 2009 12:48
> To: Bingfeng Mei; Ayal Zaks
> Cc: gcc-patches@gcc.gnu.org
> Subject: RE: [PATCH] check alias sets in 
> add_inter_loop_mem_dep (ddg.c)
> 
> Hello,
> >
> > Thanks for check-in and help on the testcases.
> 
> Sure, np.
> Ayal pointed out that the __strict keyword in testcase sms-7.c
> does not effect the ddg since both b and c are loads.
> So if that's OK I'll commit the following fix.
> 
> Thanks,
> Revital
> 
>         * gcc.dg/sms-7.c: Fix test.
> 
> 
> Index: gcc.dg/sms-7.c
> ===================================================================.
> --- gcc.dg/sms-7.c      (revision 143566)
> +++ gcc.dg/sms-7.c      (working copy)
> @@ -3,7 +3,7 @@
> 
>  extern void abort (void);
> 
> -void foo (int *a, short * __restrict__ b, short * __restrict__ c)
> +void foo (int * __restrict__ a, int * __restrict__ b, short * c))
>  {
>     int i;
>     for(i = 0; i < 100; i+=4)
> @@ -15,8 +15,8 @@ void foo (int *a, short * __restrict__ b
>       }
>  }
> 
> -int a[100];,
> -short b[100], c[100];
> +int a[100], b[100];];
> +short c[100];
> 
>  int main()
>  {
> 
> 
>