Re: PATCH: Update default arch for x86

[Jack Howarth <howarth at bromo dot med dot uc dot edu>]

On Sat, Feb 27, 2010 at 09:41:41AM -0800, H.J. Lu wrote:
> On Sat, Feb 27, 2010 at 9:31 AM, Matthias Klose <doko@ubuntu.com> wrote:
> > On 27.02.2010 17:54, H.J. Lu wrote:
> >>
> >> On Sat, Feb 27, 2010 at 8:32 AM, Matthias Klose<doko@ubuntu.com> ?wrote:
> >
> >>> thanks! works when omitting the --with-arch-32=i486 flag, but not with
> >>> it.
> >>>
> >>
> >> Try this one.
> >
> > thanks, works as expected.
> >
> 
> I am checking this in to fix bootstrap.
> 
> Thanks.
> 
> 
> -- 
> H.J.

HJ,
   The current gcc trunk is picking the wrong arch for i686-apple-darwin10
in with your final patch. I see...

# GNU C++ (GCC) version 4.5.0 20100227 (experimental) (i686-apple-darwin10)
#       compiled by GNU C version 4.5.0 20100227 (experimental), GMP version 5.0.0, MPFR version 2.4.1, MPC version 0.8
# GGC heuristics: --param ggc-min-expand=30 --param ggc-min-heapsize=4096
# options passed:  -D__DYNAMIC__ t.cc -fPIC -mmacosx-version-min=10.6.3
# -mtune=generic -march=pentiumpro -fverbose-asm
# options enabled:  -fPIC -falign-loops -fargument-alias -fauto-inc-dec
# -fbranch-count-reg -fcommon -fdelete-null-pointer-checks -fearly-inlining
# -feliminate-unused-debug-types -fexceptions -ffunction-cse -fgcse-lm
# -fident -finline-functions-called-once -fira-share-save-slots
# -fira-share-spill-slots -fivopts -fkeep-static-consts
# -fleading-underscore -fmerge-debug-strings -fmove-loop-invariants
# -fpeephole -freg-struct-return -fsched-critical-path-heuristic
# -fsched-dep-count-heuristic -fsched-group-heuristic -fsched-interblock
# -fsched-last-insn-heuristic -fsched-rank-heuristic -fsched-spec
# -fsched-spec-insn-heuristic -fsched-stalled-insns-dep -fshow-column
# -fsigned-zeros -fsplit-ivs-in-unroller -ftrapping-math -ftree-cselim
# -ftree-forwprop -ftree-loop-im -ftree-loop-ivcanon -ftree-loop-optimize
# -ftree-parallelize-loops= -ftree-phiprop -ftree-pta -ftree-reassoc
# -ftree-scev-cprop -ftree-slp-vectorize -ftree-vect-loop-version
# -funit-at-a-time -fvect-cost-model -fverbose-asm
# -fzero-initialized-in-bss -gstrict-dwarf -m128bit-long-double -m32
# -m80387 -maccumulate-outgoing-args -malign-stringops -mfancy-math-387
# -mfp-ret-in-387 -mfused-madd -mieee-fp -mno-red-zone -mno-sse4
# -mpush-args -msahf

from 'touch t.s' and 'gcc-4 -fverbose-asm t.cc -S'/

HJ,
   The current gcc trunk is picking the wrong arch for i686-apple-darwin10
in with your final patch. I see...

# GNU C++ (GCC) version 4.5.0 20100227 (experimental) (i686-apple-darwin10)
#       compiled by GNU C version 4.5.0 20100227 (experimental), GMP version 5.0.0, MPFR version 2.4.1, MPC version 0.8
# GGC heuristics: --param ggc-min-expand=30 --param ggc-min-heapsize=4096
# options passed:  -D__DYNAMIC__ t.cc -fPIC -mmacosx-version-min=10.6.3
# -mtune=generic -march=pentiumpro -fverbose-asm
# options enabled:  -fPIC -falign-loops -fargument-alias -fauto-inc-dec
# -fbranch-count-reg -fcommon -fdelete-null-pointer-checks -fearly-inlining
# -feliminate-unused-debug-types -fexceptions -ffunction-cse -fgcse-lm
# -fident -finline-functions-called-once -fira-share-save-slots
# -fira-share-spill-slots -fivopts -fkeep-static-consts
# -fleading-underscore -fmerge-debug-strings -fmove-loop-invariants
# -fpeephole -freg-struct-return -fsched-critical-path-heuristic
# -fsched-dep-count-heuristic -fsched-group-heuristic -fsched-interblock
# -fsched-last-insn-heuristic -fsched-rank-heuristic -fsched-spec
# -fsched-spec-insn-heuristic -fsched-stalled-insns-dep -fshow-column
# -fsigned-zeros -fsplit-ivs-in-unroller -ftrapping-math -ftree-cselim
# -ftree-forwprop -ftree-loop-im -ftree-loop-ivcanon -ftree-loop-optimize
# -ftree-parallelize-loops= -ftree-phiprop -ftree-pta -ftree-reassoc
# -ftree-scev-cprop -ftree-slp-vectorize -ftree-vect-loop-version
# -funit-at-a-time -fvect-cost-model -fverbose-asm
# -fzero-initialized-in-bss -gstrict-dwarf -m128bit-long-double -m32
# -m80387 -maccumulate-outgoing-args -malign-stringops -mfancy-math-387
# -mfp-ret-in-387 -mfused-madd -mieee-fp -mno-red-zone -mno-sse4
# -mpush-args -msahf

from 'touch t.cc' and 'gcc-4 -fverbose-asm t.cc -S' on a compiler built with...

Using built-in specs.
COLLECT_GCC=gcc-4
COLLECT_LTO_WRAPPER=/sw/lib/gcc4.5/libexec/gcc/i686-apple-darwin10/4.5.0/lto-wrapper
Target: i686-apple-darwin10
Configured with: ../gcc-4.5-20100227/configure --prefix=/sw --prefix=/sw/lib/gcc4.5 --mandir=/sw/share/man --infodir=/sw/share/info --enable-languages=c,c++,fortran,objc,obj-c++,java --with-gmp=/sw --with-libiconv-prefix=/sw --with-ppl=/sw --with-cloog=/sw --with-mpc=/sw --with-system-zlib --x-includes=/usr/X11R6/include --x-libraries=/usr/X11R6/lib --disable-libjava-multilib --build=i686-apple-darwin10 --host=i686-apple-darwin10 --target=i686-apple-darwin10
Thread model: posix
gcc version 4.5.0 20100227 (experimental) (GCC)