RE: [Patch, testsuite] Skip case ipa-sra-2.c for strict_align target.

2011-09-07 Thread Terry Guo
Hello, > > > > Hm, the testcase looks like it should actually pass even for strict- > align > > targets. Martin? > > > > Well, tree_non_mode_aligned_mem_p clearly does not really work very > well and needs a more-or-less reimplementation, there has been a > discussion about this already in Augus

Re: [Patch, testsuite] Skip case ipa-sra-2.c for strict_align target.

2011-09-07 Thread Martin Jambor
Hi, On Wed, Sep 07, 2011 at 11:35:53AM +0200, Richard Guenther wrote: > On Wed, Sep 7, 2011 at 9:47 AM, Terry Guo wrote: > > Hello, > > > > The SRA optimization requires the reference to struct/union member must be > > aligned to their natural boundary e.g. (int *) must be aligned to 4 byte > > b

RE: [Patch, testsuite] Skip case ipa-sra-2.c for strict_align target.

2011-09-07 Thread Terry Guo
Hello, > > Hm, the testcase looks like it should actually pass even for strict- > align > targets. Martin? > > Richard. > I debugged the GCC and observed the below situation. Here is the code snippet of this case: static int __attribute__((noinline)) ox (struct bovid *cow) { cow->red = cow

Re: [Patch, testsuite] Skip case ipa-sra-2.c for strict_align target.

2011-09-07 Thread Richard Guenther
On Wed, Sep 7, 2011 at 9:47 AM, Terry Guo wrote: > Hello, > > The SRA optimization requires the reference to struct/union member must be > aligned to their natural boundary e.g. (int *) must be aligned to 4 byte > boundary. This is done in function tree_non_mode_aligned_mem_p. For target > like x8