[PATCH] D147156: [llvm-exegesis] Factor out DisassemblerHelper from the Analysis class

Pavel Kosov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 29 06:58:43 PDT 2023


kpdev42 created this revision.
kpdev42 added reviewers: courbet, aidengrossman.
kpdev42 added a project: LLVM.
Herald added a subscriber: mstojanovic.
Herald added a project: All.
kpdev42 requested review of this revision.

As part of preparing the reports, the Analysis class needs to print
machine instructions in a disassembled form. For this purpose, the class
has four fields (namely Context_, AsmInfo_, InstPrinter_ and Disasm_).
All the constructor of the Analysis class does is conditionally
initializing these four fields.

This commit factors out the logic for decoding machine code and printing
it in an assembler form into a separate DisassemblerHelper class.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D147156

Files:
  llvm/tools/llvm-exegesis/lib/Analysis.cpp
  llvm/tools/llvm-exegesis/lib/Analysis.h
  llvm/tools/llvm-exegesis/lib/CMakeLists.txt
  llvm/tools/llvm-exegesis/lib/DisassemblerHelper.cpp
  llvm/tools/llvm-exegesis/lib/DisassemblerHelper.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D147156.509354.patch
Type: text/x-patch
Size: 7295 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230329/f80de148/attachment-0001.bin>


More information about the llvm-commits mailing list