[Lldb-commits] [PATCH] D85070: [intel-pt] Refactor 2: create a new folder structure

walter erquinigo via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Sat Aug 1 00:40:56 PDT 2020


wallace created this revision.
wallace added a reviewer: clayborg.
Herald added subscribers: lldb-commits, mgorny.
Herald added a project: LLDB.
wallace requested review of this revision.
Herald added a subscriber: JDevlieghere.

Depends on D85068 <https://reviews.llvm.org/D85068>.

The existing folder structure was not great and was using relative includes
for headers in C++ files, which I'm not fond of. I've split the code into two
main groups: Core and API. This will help us have a clear separation between
the public and private code.

Besides, I've updated the CMakeFile accordingly.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D85070

Files:
  lldb/tools/intel-features/cli-wrapper.cpp
  lldb/tools/intel-features/intel-pt/CMakeLists.txt
  lldb/tools/intel-features/intel-pt/Decoder.cpp
  lldb/tools/intel-features/intel-pt/Decoder.h
  lldb/tools/intel-features/intel-pt/PTDecoder.cpp
  lldb/tools/intel-features/intel-pt/PTDecoder.h
  lldb/tools/intel-features/intel-pt/cli-wrapper-pt.cpp
  lldb/tools/intel-features/intel-pt/cli-wrapper-pt.h
  lldb/tools/intel-features/intel-pt/include/intel-pt/API/PTDecoder.h
  lldb/tools/intel-features/intel-pt/include/intel-pt/API/cli-wrapper-pt.h
  lldb/tools/intel-features/intel-pt/include/intel-pt/Core/Decoder.h
  lldb/tools/intel-features/intel-pt/interface/PTDecoder.i
  lldb/tools/intel-features/intel-pt/source/API/CMakeLists.txt
  lldb/tools/intel-features/intel-pt/source/API/PTDecoder.cpp
  lldb/tools/intel-features/intel-pt/source/API/cli-wrapper-pt.cpp
  lldb/tools/intel-features/intel-pt/source/CMakeLists.txt
  lldb/tools/intel-features/intel-pt/source/Core/CMakeLists.txt
  lldb/tools/intel-features/intel-pt/source/Core/Decoder.cpp
  lldb/tools/intel-features/scripts/CMakeLists.txt
  lldb/tools/intel-features/scripts/lldb-intel-features.swig

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D85070.282378.patch
Type: text/x-patch
Size: 7723 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20200801/cd2b5080/attachment.bin>


More information about the lldb-commits mailing list