[PATCH 1/2] Support GAS flag -mfloat-abi; remove .option soft/hard-float

Palmer Dabbelt palmer@dabbelt.com
Tue Nov 1 23:08:00 GMT 2016


On Tue, 01 Nov 2016 15:52:45 PDT (-0700), joseph@codesourcery.com wrote:
> On Tue, 1 Nov 2016, Palmer Dabbelt wrote:
>
>> diff --git a/gas/config/tc-riscv.c b/gas/config/tc-riscv.c
>> index 592c95a..b9c349e 100644
>> --- a/gas/config/tc-riscv.c
>> +++ b/gas/config/tc-riscv.c
>> @@ -1717,8 +1717,7 @@ enum options
>>    OPTION_MARCH,
>>    OPTION_PIC,
>>    OPTION_NO_PIC,
>> -  OPTION_MSOFT_FLOAT,
>> -  OPTION_MHARD_FLOAT,
>> +  OPTION_MFLOAT_ABI,
>>    OPTION_MRVC,
>>    OPTION_MNO_RVC,
>>    OPTION_END_OF_ENUM
>> @@ -1734,20 +1733,20 @@ struct option md_longopts[] =
>>    {"fno-pic", no_argument, NULL, OPTION_NO_PIC},
>>    {"mrvc", no_argument, NULL, OPTION_MRVC},
>>    {"mno-rvc", no_argument, NULL, OPTION_MNO_RVC},
>> -  {"msoft-float", no_argument, NULL, OPTION_MSOFT_FLOAT},
>> -  {"mhard-float", no_argument, NULL, OPTION_MHARD_FLOAT},
>> +  {"mfloat-abi", required_argument, NULL, OPTION_MFLOAT_ABI},
>
> Whenever you add or change assembler options you need to update the
> manual: c-riscv.texi, and the list of options for the manpage under "@c
> man begin TARGET" in as.texinfo (this port appears to be missing the
> latter).

Sorry about that.  I can re-spin the patch so the documentation matches, and
I've added the TARGET section as another patch.



More information about the Binutils mailing list