This is the mail archive of the gcc-bugs@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug tree-optimization/49937] [4.7 Regression] g++.dg/tree-ssa/fwprop-align.C


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49937

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #9 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-08-10 08:53:09 UTC ---
(In reply to comment #7)
> (In reply to comment #4)
> > Created attachment 24959 [details]
> > patch
> > 
> > Patch I'm testing.
> 
> Me too: fixes the bug with no regressions for cris-elf at r177605.  Thanks.
> What do you think if anything, should be done for BIGGEST_ALIGNMENT usage?
> Ok, let's take it to the list.

I _think_ now that get_{pointer,object}_alignment is returning a conservative
result (thus, 8 for any pointer that hasn't alignment meta information
attached) limiting the result is pointless - certainly limiting it to
BIGGEST_ALIGNMENT.

Thus I'd propose to get rid of that argument to those functions.  On that
course see if any caller cannot deal with an alignment bigger than
BIGGEST_ALIGNMENT (which would be odd).

Well, this one is fixed at least.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]