On Thu, Apr 5, 2012 at 8:57 AM, H.J. Lu <hjl.to...@gmail.com> wrote: > On Thu, Apr 5, 2012 at 8:36 AM, Uros Bizjak <ubiz...@gmail.com> wrote: >> On Thu, Apr 5, 2012 at 3:28 PM, H.J. Lu <hjl.to...@gmail.com> wrote: >> >>>>>> Looking at how other targets implement this check, I don't think that >>>>>> this is a problem at all. This issue only shows on a non-bootstrapped >>>>>> build. A full bootstrap will use correct address. >>>>> >>>>> The other place where it shows up is cross compilers but who is going >>>>> to use a 3.2 compiler with GCC 4.8 anyways? >>>>> >>>> >>>> FWIW, I have no problem with checking __LP64__ only. I just want to >>>> mention this potential issue. >>>> >>> >>> Here is a different patch. It checks sizeof (void *) for those hosts >>> where it is appropriate. Does it look OK? >> >> Looks OK to me, but you need a GWP reviewer to OK this change now... >> > > Here is the patch with ChangeLog. Tested on Linux/x32. Jakub, > is this OK for trunk? >
Hi Richard, Is this patch: http://gcc.gnu.org/ml/gcc-patches/2012-04/msg00303.html OK for trunk? Thanks. -- H.J.