[PATCH] D72172: [MC] Add parameter `Address` to MCInstPrinter::printInst
James Henderson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 6 01:03:30 PST 2020
jhenderson accepted this revision.
jhenderson added a comment.
This revision is now accepted and ready to land.
I don't see any problems with this, so LGTM, although with the caveat that I'm not familiar with some of the usage sites, so it might be worth getting others to confirm the value passed in for the address makes sense in some situations (e.g. llvm-mca).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72172/new/
https://reviews.llvm.org/D72172
More information about the llvm-commits
mailing list