[PATCH] D128029: [AArch64] Add target feature "all"

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 17 10:19:44 PDT 2022


MaskRay added a comment.

In D128029#3592359 <https://reviews.llvm.org/D128029#3592359>, @nickdesaulniers wrote:

> Honestly, I think this feature list should just exist in the disassembler.  Is this generally useful when building code? I don't think so; its meaning and ABI will change over time as more extensions are added to the architecture.

It is generally not useful when building code, so I have emphasized that `clang -march=` does not support `all`. I think such a feature list does not exist in the disassembler.
There is some maintenance cost, but lacking some stuff when new target features emerge does not matter: people running llvm-objdump may notice and we can add new target features to "all".


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128029



More information about the llvm-commits mailing list