[PATCH v2] RISC-V: Change -march parsing.
Kito Cheng
kito.cheng@gmail.com
Mon Dec 23 07:21:00 GMT 2019
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.
More information about the Binutils
mailing list