[PATCH] D57020: [llvm-objdump] - Split disassembleObject() into two methods. NFCI.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 21 07:34:54 PST 2019


grimar created this revision.
grimar added reviewers: jhenderson, davide, rupprecht, khemant, jakehehrlich, sidneym.

Currently, `disassembleObject()` is a ~550 lines length function.

This patch splits it into two, where first do all helper objects initializations
and calls the second which does all the rest job.
This is kind of straightforward split, no code was moved around or changed.

This one is rebased on top of D57019 <https://reviews.llvm.org/D57019>.


https://reviews.llvm.org/D57020

Files:
  tools/llvm-objdump/llvm-objdump.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57020.182793.patch
Type: text/x-patch
Size: 6029 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190121/710aa23c/attachment.bin>


More information about the llvm-commits mailing list