[PATCH] D113779: [Clang] Add mfp16, mfp16fml and mdotprod flags for ARM target features.

Sjoerd Meijer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 7 06:56:44 PDT 2022


SjoerdMeijer added a comment.

In D113779#3496589 <https://reviews.llvm.org/D113779#3496589>, @fhahn wrote:

> In D113779#3207936 <https://reviews.llvm.org/D113779#3207936>, @SjoerdMeijer wrote:
>
>>> If anybody has contacts to GCC that would be very helpful. Unfortunately I don't think I will be able to drive this.
>>
>> Ok, I will bring this up internally first with some folks that work on GCC and see what happens. To be continued...
>
> Hi, did you get an update by any chance?

Sorry for the delay. I have just left a message for our GNU toolchain guys with an invitation to add comments here.
I know there are strong opinions that -march should be the only way to set extensions, but personally I am open to use case that it might be difficult to override -march, so I am not blocking this.
I am still of the opinion that options are already an enormous mess, and introducing another way is not making things necessarily better. But if we document this, and add -m options for existing extensions, it may not be worse. So that will be my request for this patch, that we document this somewhere in the Clang docs, and then ideally we see the patches for existing extensions.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D113779/new/

https://reviews.llvm.org/D113779



More information about the cfe-commits mailing list