HighCommander4 wrote:

> So I just tested this code on our codebase, the AST part is working, but the 
> background indexer is still not showing (all) the references, at least the 
> first `make_unique` I tested only worked with the AST. [...]

> It works on my 2 cpp file setup, so some more complexity is needed to 
> reproduce it.

I can confirm I see the same thing.

Moreover, as an attempted first step to reduce the problem, I tried using a 
`compile_commands.json` file with just one entry pointing to an (unreduced) 
source file containing an interesting `make_unique` call, and this time it 
worked (i.e the call was found as a reference from an included header even 
before the source file was opened).

This suggests that the remaining problem isn't something intrinsic to the 
background indexer vs. the dynamic indexer (e.g. different SymbolCollector 
options), since it _can_ work with the background indexer -- but rather, 
somehow the references are lost when information collected while indexing one 
translation unit is merged with information collected while indexing others. 
Unfortunately, that also makes this much harder to reduce...

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

Reply via email to