jimingham wrote: I'm not super-fond of this change. UnixSignals.cpp is already an awkward fit for source/Target. It was justified because it is useful to be able to set signal handling in the dummy target (i.e. in your .lldbinit) before you could know what process plugin you are actually going to be using. That doesn't justify moving all the specific files here. And that general practice could end up making source/Target and include/lldb/Target cluttered up with so many plugin specific files that it gets hard to see the structure.
https://github.com/llvm/llvm-project/pull/221579 _______________________________________________ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
