[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