[PATCH] D23426: [AArch64] Registering default MCInstrAnalysis

Mike Aizatsky via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 11 14:23:31 PDT 2016


aizatsky added a comment.

In https://reviews.llvm.org/D23426#513089, @t.p.northover wrote:

> It looks like this ought to be testable from llvm-objdump (it seems to ise the MCInstrAnalysis to produce better output for branches).


No, not yet. It requires evaluateBranch to work. And that one doesn't work for AArch64 because offset arguments are not correctly annotated in assembly. (https://github.com/google/sanitizers/issues/706)

I was going to try to address the second issue in the next change. BTW if you have any pointers towards what needs to be changed, I would be grateful for any help/pointers.


https://reviews.llvm.org/D23426





More information about the llvm-commits mailing list