[PATCH] D128029: [AArch64] Add target feature "all"
David Spickett via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 5 08:55:32 PDT 2022
DavidSpickett added a comment.
Thanks for implementing this!
I've been trying to use it in lldb's disassembler and found that for the "predres" extension +all doesn't work as expected.
$ ./bin/llvm-mc --triple aarch64 --assemble -o - -mattr=+all --show-encoding <<< "cfp rctx, x0"
.text
<stdin>:1:5: error: CFPRCTX requires: predres
cfp rctx, x0
^
I think I know what causes this and it's that `AArch64AsmParser::parseSysAlias` uses the feature bits from the SubTargetInfo. This is before "+all" has set all the other feature bits. It should be using the feature bits stored in the AsmParser, so I'm looking at changing it to do that.
Anything that isn't a sys instruction alias is fine.
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