[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