ilya-biryukov added a comment.

In https://reviews.llvm.org/D40562#939950, @arphaman wrote:

> This change breaks cached completions for declarations in namespaces in 
> libclang. What exactly are you trying to achieve here? We could introduce 
> another flag maybe.


Am I right to assume this cache is there to reduce the amount of `Decl`s we 
need to deserialize from `Preamble`s? Maybe we could fix the cache to also 
include namespace-level `Decl`s? It should improve performance of the cached 
completions.

But for a quick workaround we could introduce a separate flag.


https://reviews.llvm.org/D40562



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to