[Lldb-commits] [PATCH] D155107: Add support for llvm::MCInstPrinter::setPrintBranchImmAsAddress

David Spickett via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Jul 27 01:09:36 PDT 2023


DavidSpickett accepted this revision.
DavidSpickett added a comment.

> Also you can test it as long as you put the right skipif annotations on it as it'll be architecture specific. Or use a corefile and just check that the backend for that architecture is enabled.

Given that:

- We are a thin layer around llvm-mc
- There is some confusion about exactly what each target does with its assembly already
- The target that benefits most from this doesn't have a buildbot

This is ok to go in without specific testing.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D155107/new/

https://reviews.llvm.org/D155107



More information about the lldb-commits mailing list