This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH v2] RISC-V: Change -march parsing.
Hi Maxim:
LGTM, just need Palmer or Jim to approve :)
Hi Palmer:
>> +static bfd_boolean
>> +riscv_ext_x_valid_p (const char *arg)
>> +{
>> + if (!strcasecmp (arg, "x"))
>> + return FALSE;
>> +
>> + return TRUE;
>
> This allows any X extension through but then goes ahead to not provide any
> support for those extensions. I can understand providing some scaffolding to
> users to turn on their custom extensions, but we should fail quickly on any
> extensions that aren't supported.
We bypass any X extension now, and one concern is we can't testing any
X parsing/combine logic if we reject that.