This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] RISC-V: Allow ISA subsets to be disabled


On Fri, 31 Mar 2017 02:22:53 PDT (-0700), Nick Clifton wrote:
> Hi Alex,
>
>> It's a shame gcc and as are inconsistent in their argument handling in
>> this way, at least on x86-64.
>
> Agreed.
>
> I had always thought that a later option overriding an earlier option was
> standard behaviour, but when I checked the gcc documentation I could not
> find any specific requirement for this.  I was quite surprised.  Personally
> I think that gcc is wrong.  To my mind it is intuitive that a later option
> would override an earlier one.  If it does not then the tool concerned, at
> the very least, ought to issue a warning message that the later option is
> being ignored.
>
> So, in summary - I think that this is a gcc bug, not a binutils bug.

I agree (though I guess that's implicit from having posted the patch :)).
We're trying really hard to make sure RISC-V has a sane set of machine-specific
options that are compatible everywhere (binutils, GCC, and LLVM) so we don't
get into this sort of mess in the future.

Committed.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]