[PATCH] testsuite: mips: use noinline attribute instead of -fno-inline
Richard Sandiford
richard.sandiford@arm.com
Mon Jul 26 13:55:24 GMT 2021
Xi Ruoyao <xry111@mengyan1223.wang> writes:
> On Fri, 2021-07-23 at 11:18 +0800, Xi Ruoyao wrote:
>> On Fri, 2021-07-23 at 04:21 +0200, Maciej W. Rozycki wrote:
>> > On Fri, 9 Jul 2021, Richard Sandiford via Gcc-patches wrote:
>> >
>> > > > > > > The "smallest fix" is simply adding -fno-inline into
>> > > > > > > mips.exp.
>> > > > > > > However
>> > > > > > > I don't like it because I agree with you that mips.exp
>> > > > > > > shouldn't
>> > > > > > > care
>> > > > > > > about dg-options, at least don't do it too much.
>> > > > > > As I said in the other message, I think the smallest fix is
>> > > > > > the
>> > > > > > way
>> > > > > > to
>> > > > > > go though.
>> > > > > THanks for chiming in Richard. I didn't know all the
>> > > > > background
>> > > > > here.
>> > > > > Let's just go with the small fix based on your
>> > > > > recommendation. We
>> > > > > can
>> > > > > always revisit if we keep running into issues in this code.
>> > > >
>> > > > Pushed at 3b33b113.
>> > >
>> > > It looks like that was the originally posted patch though. It
>> > > probably
>> > > wasn't very clear, but by smallest fix, I meant adding inline to:
>> >
>> > Xi, will you revert your commit that was not approved and apply the
>> > correct fix?
>>
>> Sorry, somehow I didn't see Richard's reply. Perhaps a
>> misconfiguration
>> on my mail server.
I don't know where the problem lies, but for some reason I've been
getting rejects when sending messages directly (via reply-all).
I should have said something on-list, sorry.
I'll try replying only via the list to see if that helps.
>> The "correct" fix is
>>
>> --- a/gcc/testsuite/gcc.target/mips/mips.exp
>> +++ b/gcc/testsuite/gcc.target/mips/mips.exp
>> @@ -325,6 +325,7 @@ foreach option {
>> finite-math-only
>> fixed-hi
>> fixed-lo
>> + inline
>> lax-vector-conversions
>> omit-frame-pointer
>> optimize-sibling-calls
>>
>> right? I'll do a regtest and if there is no problem I'll commit it.
>
> Done at 863737b8 and 19e05058.
Great, thanks!
Richard
More information about the Gcc-patches
mailing list