On Mon, Jun 23, 2014 at 11:51 PM, Uros Bizjak <ubiz...@gmail.com> wrote: > On Mon, Jun 23, 2014 at 6:29 PM, H.J. Lu <hjl.to...@gmail.com> wrote: > >>>>> > --- gcc/config/i386/driver-i386.c.jj 2014-05-14 14:45:54.000000000 >>>>> > +0200 >>>>> > +++ gcc/config/i386/driver-i386.c 2014-06-20 18:59:57.805006358 >>>>> > +0200 >>>>> > @@ -745,6 +745,11 @@ const char *host_detect_local_cpu (int a >>>>> > /* Assume Core 2. */ >>>>> > cpu = "core2"; >>>>> > } >>>>> > + else if (has_longmode) >>>>> > + /* Perhaps some emulator? Assume x86-64, otherwise gcc >>>>> > + -march=native would be unusable for 64-bit >>>>> > compilations, >>>>> > + as all the CPUs below are 32-bit only. */ >>>>> > + cpu = "x86-64"; >>>>> > else if (has_sse3) >>>>> > /* It is Core Duo. */ >>>>> > cpu = "pentium-m"; >>>>> > >>>>> > Jakub >>>>> >>>>> host_detect_local_cpu guesses the cpu based on the real processors. >>>>> It doesn't work with emulators due to some conflicts. This isn't the >>>>> only only place which has the same issue. I prefer something like >>>>> this. >>>> >>>> I'm fine with your patch too. Let's wait what Uros (or other i?86 >>>> maintainers) pick up. >>> >>> This looks OK to me. >>> >>> Thanks, >>> Uros. >> >> This is what I checked in. > > This version was NOT approved. Please revert it ASAP and proceed with > approved version. > > Uros.
I reverted my change. Sorry for my misunderstanding. -- H.J.
