mchoo7 wrote: A bit of background context:
This is the first patch of refactoring lldbPluginProcessUtility. As you see in https://github.com/llvm/llvm-project/blob/de3dd2dc277f4bab62dc840c26e794544f69393b/lldb/source/Plugins/Process/Utility/CMakeLists.txt#L1, some LLDB developers are already aware that lldbPluginProcessUtility has some issues. Not only it has cyclic dependency graph as written in [Process/Utility gn build files](https://github.com/llvm/llvm-project/blob/main/llvm/utils/gn/secondary/lldb/source/Plugins/Process/Utility/BUILD.gn), the name Utility is misleading at the first place as it can be an umbrella term for any source file. `Process/Utility` ended up being home for files that are not properly located thus unrelated files are sitting under same directory (e.g. architecture-specific register definitions and OS signals). As a developer who's planning to add more architecture support for FreeBSD's lldb, this became quite annoying to deal with, and I thought things like `RegisterContext*` and `RegisterInfo*` should be separated to make adding new architectural support easier. The final goal of this patch series is to remove lldbPluginProcessUtility completely and relocate files to proper location. Some files like OSSignals are merged into existing libraries and some like RegisterContext will be isolated into new libraries. The full patch series is available on [my fork repo](https://github.com/mchoo7/llvm-project/tree/pu). Right now it consists of 12 commits which will be 12 PRs, but some commits need final touch as it is generated by LLM and thus violates some LLVM coding styles. https://github.com/llvm/llvm-project/pull/221579 _______________________________________________ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
