Now that LLDB proper has built-in support for intel-pt traces, we can remove the old plugin written by Intel. It has less features and it's hard to work with.
As a test, I ran "ninja lldbIntelFeatures" and it worked.
Differential D102866
[lldb][intel-pt] Remove old plugin wallace on May 20 2021, 12:42 PM. Authored by
Details Now that LLDB proper has built-in support for intel-pt traces, we can remove the old plugin written by Intel. It has less features and it's hard to work with. As a test, I ran "ninja lldbIntelFeatures" and it worked.
Diff Detail
Event TimelineComment Actions I would be ok with this. Can you track down the original author and add them to the review of this to be sure they aren't using it anymore, and if they need tracing that they have everything they need with your new functionality? Comment Actions I'll contact them again. Last time I did they didn't reply. And btw, this feature was broken for years and last year i revived this plugin, so I imagine no one was using it. And in terms of features, the new implementation supports the same features Comment Actions Btw, lldb-server for a few months hasn't been supporting the functionalities required by the old plugin, and as no one has complained, I imagine it's been not actively used. |