[Lldb-commits] [PATCH] D33035: Tool for using Intel(R) Processor Trace hardware feature
Abhishek via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon May 29 06:28:22 PDT 2017
abhishek.aggarwal added a comment.
In https://reviews.llvm.org/D33035#754640, @labath wrote:
> I don't really like that we are adding a public shared library for every tiny intel feature. Could we at least merge this "plugin" with the existing "intel-mpx plugin" to create one "intel support" library?
> Also, adding an external dependency probably deserves a discussion on lldb-dev.
Hi Paval ... Before starting the development of this whole feature, we had submitted the full proposal to lldb dev list 1.5 years ago. During the discussions, it was proposed to keep the external dependency outside LLDB (i.e. not to be bundled in liblldb shared library). The External dependency required for this tool is not and will never be a part of lldb repository. Users who are interested to use this tool, will download this external dependency separately.
Comment at: tools/intel-pt/CMakeLists.txt:42
> any reason you're not using add_lldb_library here?
No. I will try with that.
More information about the lldb-commits