weliveindetail wrote: > General comment: like I said in the RFC thread, would it be possible to have > the behavior in `SymbolLocatorSymStore::LocateExecutableSymbolFile()` > somewhere in a `llvm/` sub-module instead? It would be useful for other > usages, [...]
Yes, I don't disagree. In this early version there isn't much to move to LLVM. For the future: In my experience it's a good practice to split this in 2 steps: Implement features in subprojects first and then hoist common functionality to LLVM on-demand. https://github.com/llvm/llvm-project/pull/183302 _______________________________________________ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
