[PATCH] D23426: [AArch64] Registering default MCInstrAnalysis

Tim Northover via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 11 14:31:44 PDT 2016


t.p.northover added a comment.

> And that one doesn't work for AArch64 because offset arguments are not correctly annotated in assembly. (https://github.com/google/sanitizers/issues/706).


Looking at the default implementation my best guess is that `MI.getOperand(0).getImm()` actually returns `Offset / 4` for branches on AArch64 (because instructions must always be 4-byte aligned).

But I could be wrong, because I don't know exactly how it's failing.

Tim.


https://reviews.llvm.org/D23426





More information about the llvm-commits mailing list