Przemyslaw Wirkus <przemyslaw.wir...@arm.com> writes:
>> >> >> gcc/ChangeLog:
>> >> >>
>> >> >> * config/aarch64/aarch64-option-extensions.def
>> >> >> (AARCH64_OPT_EXTENSION): New +flagm option in -march for AArch64.
>> >> >> * config/aarch64/aarch64.h (AARCH64_FL_FLAGM): Add new flagm
>> >> >> extension bit mask.
>> >> >> (AARCH64_FL_FOR_ARCH8_4): Add flagm to Armv8.4-A.
>> >> >
>> >> > OK, thanks, and sorry for the slow review.
>> >>
>> >> Just remembered that we also need documentation for the new feature
>> >> flag in doc/invoke.texi.
>> >
>> > Done. Thank you for the reminder.
>> >
>> > commit 48ff86adfd96a0f5132273719932b48a14941881
>>
>> I think you also need to add flagm to:
>>
>> -----------------------------------------------------------------------
>> The table below summarizes the permissible values for @var{arch} and the
>> features that they enable by default:
>>
>> @multitable @columnfractions 0.20 0.20 0.60 @headitem @var{arch} value
>> @tab Architecture @tab Includes by default … @item @samp{armv8.4-a}
>> @tab Armv8.4-A @tab @samp{armv8.3-a}, @samp{+fp16fml},
>> @samp{+dotprod}
>> -----------------------------------------------------------------------
>
> Yes, I've just noticed that :/
> I will update this with my next patch which adds yet another -march flag if
> that's OK with you?
Yeah, that'd be fine.
Richard
