Re: Fix minor reorg.c bug affecting MIPS targets

2017-05-16 Thread Paul Hua
Hi: commited as r248137. Thanks, paul. On Wed, May 17, 2017 at 2:46 AM, Toma Tabacu wrote: > From: Jeff Law >> On 05/16/2017 10:01 AM, Toma Tabacu wrote: >>> Hello Paul, >>> >>> You're seeing this problem because mips.exp can't handle -O* in dg-options. >>> The other tests in gcc.target/mips us

RE: Fix minor reorg.c bug affecting MIPS targets

2017-05-16 Thread Toma Tabacu
From: Jeff Law > On 05/16/2017 10:01 AM, Toma Tabacu wrote: >> Hello Paul, >> >> You're seeing this problem because mips.exp can't handle -O* in dg-options. >> The other tests in gcc.target/mips use a dg-skip-if to skip for -O0 and -O1 >> instead of having -O2 in dg-options. >> This is supposed to

Re: Fix minor reorg.c bug affecting MIPS targets

2017-05-16 Thread Jeff Law
On 05/16/2017 10:01 AM, Toma Tabacu wrote: > Hello Paul, > > You're seeing this problem because mips.exp can't handle -O* in dg-options. > The other tests in gcc.target/mips use a dg-skip-if to skip for -O0 and -O1 > instead of having -O2 in dg-options. > This is supposed to ensure that the tests

RE: Fix minor reorg.c bug affecting MIPS targets

2017-05-16 Thread Toma Tabacu
s possible. I believe that Matthew can confirm this. Regards, Toma Tabacu > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On > Behalf Of Paul Hua > Sent: 16 May 2017 10:11 > To: Jeff Law > Cc: gcc-patches > Sub

Re: Fix minor reorg.c bug affecting MIPS targets

2017-05-16 Thread Paul Hua
Hi: There are new failures between r248067 and r248036 on mips64el-unknown-linux-gnu. ERROR: gcc.target/mips/reorgbug-1.c -O0 : Unrecognised option: -O2 for " dg-options 1 "-O2 -msoft-float -mips2" " UNRESOLVED: gcc.target/mips/reorgbug-1.c -O0 : Unrecognised option: -O2 for " dg-options

Fix minor reorg.c bug affecting MIPS targets

2017-05-15 Thread Jeff Law
There's a subtle bug in reorg.c's relax_delay_slots that my tester tripped this weekend. Not sure what changed code generation wise as the affected port built just fine last week. But it is what is is. Assume before this code we've set TARGET_LABEL to the code_label associated with DELA