labath added a comment. In https://reviews.llvm.org/D33035#767029, @abhishek.aggarwal wrote:
> 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. Yes I remember that. But, as you say, that was 1.5 years ago, and we haven't heard anything since. Honestly, I had assumed you abandoned that work until you reappeared last month. :) So I think it's worth updating that thread, as new things may have come up since then (for example, the decision whether to merge the intel stuff into a single library). https://reviews.llvm.org/D33035 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
