[PATCH] D121768: [X86][tablgen] Auto-generate trivial fields and trivial interfaces for target features

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

In D121768#3393915 <https://reviews.llvm.org/D121768#3393915>, @skan wrote:

> In D121768#3393913 <https://reviews.llvm.org/D121768#3393913>, @craig.topper wrote:
>
>> So you want feature specific knobs that you have to pass through those front end command line interfaces with something like clang’s -mllvm?
>>
>> In my opinion we should add more interfaces  in X86TargetParser.cpp so that all frontends can learn what features llvm supports without a hardcoded list like in Options.td. And those frontends should use the target-feature attribute.
>
> Some front-ends are not open-source. We could not teach it.

If we provide good library interfaces the non open source projects can implement it themselves.

There’s no guarantee that a way to set an llvm override exists in another frontend. Last I knew ispc for example didn’t have an equivalent of -mllvm. I think I heard once that Apples version of clang doesn’t support -mllvm.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121768