[Lldb-commits] [clang] [clang-tools-extra] [libcxx] [lldb] [llvm] Add clang basic module directory (PR #93388)

David Stone via lldb-commits lldb-commits at lists.llvm.org
Sun May 26 11:53:59 PDT 2024


davidstone wrote:

> I don't like the PR since I don't feel it makes the code cleaner and it may make the downstream suffering backporting.
> 
> If there are deeper reasons or following patches, we can discuss them seperately.

On its own, I would agree. My goal is to split several more things out of `Module.h` but still keep all the module things together. I'm fine letting this sit until I get some of those other changes together and we can look over a broader set of changes and see if the direction is good.

https://github.com/llvm/llvm-project/pull/93388


More information about the lldb-commits mailing list