[PATCH] D48836: [cfi-verify] Support AArch64.
Peter Collingbourne via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 3 14:38:06 PDT 2018
pcc added inline comments.
================
Comment at: tools/llvm-cfi-verify/lib/FileAnalysis.cpp:157
bool FileAnalysis::isCFITrap(const Instr &InstrMeta) const {
- return MII->getName(InstrMeta.Instruction.getOpcode()) == "TRAP";
+ return MII->getName(InstrMeta.Instruction.getOpcode()) == "TRAP" ||
+ MII->getName(InstrMeta.Instruction.getOpcode()) == "BRK";
----------------
vlad.tsyrklevich wrote:
> Instead of doing string comparisons here for every valid value, lets define a TrapOpcode variable during initialization to X86::TRAP / AArch64::BRK depending on the target. (We can bail on other architectures that we haven't explicitly tested on to avoid confusing users who might otherwise believe the results.)
I don't think we can access backend-specific things like X86::TRAP here though. It might be worth adding an interface to MCInstrAnalysis for asking what the trap instruction is and have that assert on architectures that we don't support yet.
Repository:
rL LLVM
https://reviews.llvm.org/D48836
More information about the llvm-commits
mailing list