ArcsinX wrote:

> Sorry for that, I think the database has already addressed with mangler in 
> https://github.com/llvm/llvm-project/blob/main/clang-tools-extra/clangd/ClangdLSPServer.cpp#L566-L567,
>  so I just want to ask why we need mangler again in this place.

`getProjectModules()` accepts `tooling::CompilationDatabase` which has no 
mangler inside `getCompileCommand()` call, but `ModulesBuilder` accepts 
`GlobalCompilationDatabase` (clangd internal implementation) and mangler is 
applied here automatically at `getCompileCommand()` call

https://github.com/llvm/llvm-project/pull/200001
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to