https://github.com/clayborg approved this pull request.

There is no LLDB_PLUGIN_DEFINE() macro in ProcessTrace.cpp, so that is why it 
was being manually initialized. But this plugin in a problem in that if it's 
ProcessTrace::CreateInstance(...) is called, it will always return a valid 
Process plug-in which is bad as it isn't filtering on an architecture, os, or 
other features of the target.

https://github.com/llvm/llvm-project/pull/71455
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to