diff mbox

[i386] Fix PR 57756

Message ID CAAs8Hmxodcmo4mKYcuopXpgPVXDxwQFfXdBdY97SCn17vf+53w@mail.gmail.com
State New
Headers show

Commit Message

Sriraman Tallam Oct. 18, 2013, 2:47 a.m. UTC
On Thu, Oct 17, 2013 at 4:51 PM, Sriraman Tallam <tmsriram@google.com> wrote:
> On Thu, Oct 17, 2013 at 3:08 PM, Sriraman Tallam <tmsriram@google.com> wrote:
>> On Thu, Oct 17, 2013 at 1:23 PM, Mike Stump <mikestump@comcast.net> wrote:
>>> On Oct 17, 2013, at 10:23 AM, Sriraman Tallam <tmsriram@google.com> wrote:
>>>>> You probably want to do something similar to what I did in the powerpc.
>>>>
>>>> I would need the help of target maintainers to fix it this way since
>>>> it touches every target and it would take time for me to build and
>>>> test every target.
>>>
>>> For changes that only need a compile to ensure one didn't brake a port, a configure and build of a target is 2 minutes.  Over night (6 hours), you can 180 targets.  Before you laugh, there are people that have done this sort of building in the past as well.  The hardest part, literally, would be to come up with the list of targets.
>>
>> Thanks. This is the list of affected targets extracted from the link
>> JBG pointed to:  http://toolchain.lug-owl.de/buildbot/?limit=1500
>>
>> "alpha-linux,bfin-elf,cr16-elf,epiphany-elf,frv-linux,h8300-elf,hppa-linux,m32c-elf,m68k-linux,mep-elf,microblaze-linux,mips64el-linux,mips64-linux,mipsel-linux,mips-linux,mmix,nios2-elf,powerpc64le-linux,powerpc64-linux,powerpcle-linux,powerpc-linux,ppc64-linux,ppc-linux,rl78-elf,rs6000-ibm-aix4.3,s390-linux,s390x-linux,sparc64-linux,sparc-linux,spu,spu-elf,vax-linux,xtensa-linux"
>>
>> Status: With the patch below, x86_64, i386 and powerpc native
>> bootstrap is fine. x86_64 has one test regression in
>> gfortran.dg/lto/pr45586 which I am looking into. Also, mips and
>> mips-64 cross-compiler builds fine.
>>
>> I can check cross-compile builds on all the affected targets listed
>> above. If this all works, I suggest using this patch to put out the
>> current fires and proceed with individual changes to each target
>> later.  Also, there are some test case failures when built in non-sse
>> configs in i386. I can get to it after unbreaking this. Is this
>> reasonable?
>
> I am running cross-compile builds on all the 33 failing targets. Like
> Mike said, it is only taking a few minutes per target. If there is no
> objection, I *will check in* this patch in another 2 hours provided
> all 33 targets succeed. This will atleast unblock all targets.

I can build cross-compile on 32/33 targets. I cannot build
nios2-unknown-elf alone, I get "*** Configuration nios2-unknown-elf
not supported" error. I ran "configure ... target=<blah>" and "make
all-gcc" for all the affected targets.

I have submitted this one line fix.

PR target/57756
* opth-gen.awk: Define target_flags_explicit


Known problems even after this:
* i386 test regressions, -mno--sse mode reported by Jakub and a
pr45586 fortran test failure.  I am working on fixing them.

Thanks
Sri



>
> Thanks
> Sri
>
>
>>
>> Thanks
>> Sri
>>
>>
>> Index: opth-gen.awk
>> ===================================================================
>> --- opth-gen.awk (revision 203779)
>> +++ opth-gen.awk (working copy)
>> @@ -114,6 +114,7 @@
>>  print "extern struct gcc_options global_options;"
>>  print "extern const struct gcc_options global_options_init;"
>>  print "extern struct gcc_options global_options_set;"
>> +print "#define target_flags_explicit global_options_set.x_target_flags"
>>  print "#endif"
>>  print "#endif"
>>  print ""

Comments

Mike Stump Oct. 18, 2013, 3:37 a.m. UTC | #1
On Oct 17, 2013, at 7:47 PM, Sriraman Tallam <tmsriram@google.com> wrote:
> I can build cross-compile on 32/33 targets. I cannot build
> nios2-unknown-elf alone, I get "*** Configuration nios2-unknown-elf
> not supported" error.

nios2 is not a contributed port yet on trunk.

> I have submitted this one line fix.

Or, does it mean that you checked it in to trunk?  I committed it, is a better way to say that, to distinguish it from merely sending an email to this list.  Submitting it, is merely sending email.
Sriraman Tallam Oct. 18, 2013, 3:57 a.m. UTC | #2
On Thu, Oct 17, 2013 at 8:37 PM, Mike Stump <mikestump@comcast.net> wrote:
> On Oct 17, 2013, at 7:47 PM, Sriraman Tallam <tmsriram@google.com> wrote:
>> I can build cross-compile on 32/33 targets. I cannot build
>> nios2-unknown-elf alone, I get "*** Configuration nios2-unknown-elf
>> not supported" error.
>
> nios2 is not a contributed port yet on trunk.
>
>> I have submitted this one line fix.
>
> Or, does it mean that you checked it in to trunk?

Yes I have committed this patch.

Thanks
Sri

I committed it, is a better way to say that, to distinguish it from
merely sending an email to this list.  Submitting it, is merely
sending email.
Sriraman Tallam Oct. 18, 2013, 10:56 p.m. UTC | #3
On Thu, Oct 17, 2013 at 7:47 PM, Sriraman Tallam <tmsriram@google.com> wrote:
> On Thu, Oct 17, 2013 at 4:51 PM, Sriraman Tallam <tmsriram@google.com> wrote:
>> On Thu, Oct 17, 2013 at 3:08 PM, Sriraman Tallam <tmsriram@google.com> wrote:
>>> On Thu, Oct 17, 2013 at 1:23 PM, Mike Stump <mikestump@comcast.net> wrote:
>>>> On Oct 17, 2013, at 10:23 AM, Sriraman Tallam <tmsriram@google.com> wrote:
>>>>>> You probably want to do something similar to what I did in the powerpc.
>>>>>
>>>>> I would need the help of target maintainers to fix it this way since
>>>>> it touches every target and it would take time for me to build and
>>>>> test every target.
>>>>
>>>> For changes that only need a compile to ensure one didn't brake a port, a configure and build of a target is 2 minutes.  Over night (6 hours), you can 180 targets.  Before you laugh, there are people that have done this sort of building in the past as well.  The hardest part, literally, would be to come up with the list of targets.
>>>
>>> Thanks. This is the list of affected targets extracted from the link
>>> JBG pointed to:  http://toolchain.lug-owl.de/buildbot/?limit=1500
>>>
>>> "alpha-linux,bfin-elf,cr16-elf,epiphany-elf,frv-linux,h8300-elf,hppa-linux,m32c-elf,m68k-linux,mep-elf,microblaze-linux,mips64el-linux,mips64-linux,mipsel-linux,mips-linux,mmix,nios2-elf,powerpc64le-linux,powerpc64-linux,powerpcle-linux,powerpc-linux,ppc64-linux,ppc-linux,rl78-elf,rs6000-ibm-aix4.3,s390-linux,s390x-linux,sparc64-linux,sparc-linux,spu,spu-elf,vax-linux,xtensa-linux"
>>>
>>> Status: With the patch below, x86_64, i386 and powerpc native
>>> bootstrap is fine. x86_64 has one test regression in
>>> gfortran.dg/lto/pr45586 which I am looking into. Also, mips and
>>> mips-64 cross-compiler builds fine.
>>>
>>> I can check cross-compile builds on all the affected targets listed
>>> above. If this all works, I suggest using this patch to put out the
>>> current fires and proceed with individual changes to each target
>>> later.  Also, there are some test case failures when built in non-sse
>>> configs in i386. I can get to it after unbreaking this. Is this
>>> reasonable?
>>
>> I am running cross-compile builds on all the 33 failing targets. Like
>> Mike said, it is only taking a few minutes per target. If there is no
>> objection, I *will check in* this patch in another 2 hours provided
>> all 33 targets succeed. This will atleast unblock all targets.
>
> I can build cross-compile on 32/33 targets. I cannot build
> nios2-unknown-elf alone, I get "*** Configuration nios2-unknown-elf
> not supported" error. I ran "configure ... target=<blah>" and "make
> all-gcc" for all the affected targets.
>
> I have submitted this one line fix.
>
> PR target/57756
> * opth-gen.awk: Define target_flags_explicit
>
> Index: gcc/opth-gen.awk
> ===================================================================
> --- gcc/opth-gen.awk (revision 203807)
> +++ gcc/opth-gen.awk (working copy)
> @@ -114,6 +114,7 @@ print "};"
>  print "extern struct gcc_options global_options;"
>  print "extern const struct gcc_options global_options_init;"
>  print "extern struct gcc_options global_options_set;"
> +print "#define target_flags_explicit global_options_set.x_target_flags"
>  print "#endif"
>  print "#endif"
>  print ""
>
> Known problems even after this:
> * i386 test regressions, -mno--sse mode reported by Jakub and a
> pr45586 fortran test failure.  I am working on fixing them.

The fortran test failure is not because of this commit or my original
commit to clean up target handling in rev. 203634. This failure also
showed up in prior revisions:
http://gcc.gnu.org/ml/gcc-testresults/2013-10/msg01174.html (rev.
203632). False alarm.

Sri

>
> Thanks
> Sri
>
>
>
>>
>> Thanks
>> Sri
>>
>>
>>>
>>> Thanks
>>> Sri
>>>
>>>
>>> Index: opth-gen.awk
>>> ===================================================================
>>> --- opth-gen.awk (revision 203779)
>>> +++ opth-gen.awk (working copy)
>>> @@ -114,6 +114,7 @@
>>>  print "extern struct gcc_options global_options;"
>>>  print "extern const struct gcc_options global_options_init;"
>>>  print "extern struct gcc_options global_options_set;"
>>> +print "#define target_flags_explicit global_options_set.x_target_flags"
>>>  print "#endif"
>>>  print "#endif"
>>>  print ""
diff mbox

Patch

Index: gcc/opth-gen.awk
===================================================================
--- gcc/opth-gen.awk (revision 203807)
+++ gcc/opth-gen.awk (working copy)
@@ -114,6 +114,7 @@  print "};"
 print "extern struct gcc_options global_options;"
 print "extern const struct gcc_options global_options_init;"
 print "extern struct gcc_options global_options_set;"
+print "#define target_flags_explicit global_options_set.x_target_flags"
 print "#endif"
 print "#endif"
 print ""