On Sat, Mar 31, 2012 at 11:20:27AM -0700, H.J. Lu wrote:
> On Fri, Mar 30, 2012 at 1:36 PM, H.J. Lu <hjl.to...@gmail.com> wrote:
> > On Fri, Mar 30, 2012 at 1:23 PM, Jack Howarth <howa...@bromo.med.uc.edu>
> > wrote:
> >> On Fri, Mar 30, 2012 at 11:32:37AM -0700, H.J. Lu wrote:
> >>> On Fri, Mar 30, 2012 at 11:05 AM, Jack Howarth <howa...@bromo.med.uc.edu>
> >>> wrote:
> >>> > On Fri, Mar 30, 2012 at 09:18:13AM -0700, H.J. Lu wrote:
> >>> >> On Fri, Mar 30, 2012 at 8:11 AM, Rainer Orth
> >>> >> <r...@cebitec.uni-bielefeld.de> wrote:
> >>> >> > Mike Stump <mikest...@comcast.net> writes:
> >>> >> >
> >>> >> >>> Here is the new patch. OK for trunk if there are no regressions on
> >>> >> >>> Linux/ia32 and Linux/x86-64?
> >>> >> >>
> >>> >> >> Too bad you didn't test 32-bit darwin, causes:
> >>> >> >>
> >>> >> >> http://gcc.gnu.org/PR52784
> >>> >> >>
> >>> >> >> Could you please revert or fix, thanks.
> >>> >> >
> >>> >> > Same problem on Solaris 10 and 11/x86.
> >>> >> >
> >>> >> > Rainer
> >>> >> >
> >>> >>
> >>> >> When i[34567]86-*-* targets are configured with --enable-targets=all,
> >>> >> TARGET_BI_ARCH is defined as 1, but TARGET_64BIT_DEFAULT
> >>> >> isn't defined. It leads to
> >>> >>
> >>> >> if (!TARGET_64BIT)
> >>> >> ix86_isa_flags &= ~(OPTION_MASK_ABI_64 | OPTION_MASK_ABI_X32);
> >>> >>
> >>> >> Since TARGET_64BIT is false by default, -m64 and -mx32 don't work
> >>> >> correctly. This patch changes TARGET_BI_ARCH to 3 for
> >>> >> i[34567]86-*-* targets configured with --enable-targets=all. Tested on
> >>> >> Linux/ia32 with bootstrap and Linux/ia32 with --enable-targets=all
> >>> >> --disable-bootstrap. Please try on other OSes.
> >>> >
> >>> > H.J.,
> >>> > This patch solves the bootstrap of current gcc trunk on
> >>> > i386-apple-darwin10. Thanks.
> >>> > Jack
> >>> >
> >>>
> >>> Here is a smaller patch.
> >>
> >> H.J.,
> >> The smaller patch also solves the bootstrap failure on
> >> i386-apple-darwin10.
> >> Jack
> >>
> >
> > Please ignore the smaller patch since preprocessor may handle
> > TARGET_64BIT_DEFAULT properly.
> >
>
> Please try this even smaller patch. If it works for you, I will
> check it in as an obvious fix.
H.J.,
The latest gcc-pr52784-2.patch patch also allows current gcc trunk to
bootstrap on i386-apple-darwin10.
Jack
>
> Thanks.
>
>
> --
> H.J.
