v.g.vassilev added a comment. In D127284#3969446 <https://reviews.llvm.org/D127284#3969446>, @v.g.vassilev wrote:
>> UPDATE: fails with clean build too >> >> Might be best to revert it for now while we figure out what's wrong > > I fail to reproduce it, can you give me access to some node where I can > debug? I suspect that should be something easy to fix. > > UPDATE: Reproduced it. Debugging. @Michael137, I think I understand what happens. This patch introduces a new language option which is not benign from modules perspective. However, lldb sets up modules the old way and then switches to the incremental processing mode. We have two ways to fix this: - Pass `-Xclang -fincremental-extensions` in `ClangModulesDeclVendor.cpp` and delete `instance->getPreprocessor().enableIncrementalProcessing();`; or - Apply https://reviews.llvm.org/D139258 which already does that. Can you check if https://reviews.llvm.org/D139258 passes the lldb testsuite. It does pass my reproduction by hand (I still cannot run full of the lldb-api tests)? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D127284/new/ https://reviews.llvm.org/D127284 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
