[PATCH] D100475: llvm-objdump: move SourcePrinter to shared file

Tim Northover via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 20 04:29:57 PDT 2021


t.p.northover updated this revision to Diff 338820.
t.p.northover added a comment.

Thanks, this should now be just the NFC refactoring, with most clang-tidy/clang-format suggestions applied (I ignored the DWARF one for obvious reasons).

I'm not sure what's going on with those test failures, it works on my local Linux machine. But now that this is non-MachO that's a problem for another time.


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

https://reviews.llvm.org/D100475

Files:
  llvm/tools/llvm-objdump/CMakeLists.txt
  llvm/tools/llvm-objdump/SourcePrinter.cpp
  llvm/tools/llvm-objdump/SourcePrinter.h
  llvm/tools/llvm-objdump/llvm-objdump.cpp
  llvm/tools/llvm-objdump/llvm-objdump.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D100475.338820.patch
Type: text/x-patch
Size: 51264 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210420/f8166157/attachment.bin>


More information about the llvm-commits mailing list