[PATCH] D112420: [clang][ARM] PACBTI-M assembly support

Victor Campos via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 28 07:29:11 PDT 2021


vhscampos added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMInstrThumb2.td:4083
 def : t2InstAlias<"csdb$p",   (t2HINT 20, pred:$p), 1>;
+def : t2InstAlias<"pacbti$p r12,lr,sp", (t2HINT 13, pred:$p), 1>;
+def : t2InstAlias<"bti$p", (t2HINT 15, pred:$p), 1>;
----------------
ostannard wrote:
> Why are these needed in addition to the PACBTIHintSpaceInst instructions below?
Since these instructions are in the HINT space, without specifying inst aliases, they were printed in the HINT form when using llvm-mc instead of in their PACBTI form. Therefore I added these t2InstAlias instances.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112420



More information about the llvm-commits mailing list