[PATCH] D46421: [analyzer][CrossTU] Extend CTU to VarDecls with initializer

Gábor Horváth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 1 08:29:23 PDT 2018


xazax.hun added a comment.

Sorry for the limited activity. Unfortunately, I have very little time reviewing patches lately.
I think we need to answer the following questions:

- Does this change affect the analysis of the constructors of global objects? If so, how?
- Do we want to import non-const object's initializer expressions? The analyzer might not end up using the value anyways.
- How big can the index get with this modification for large projects?

Overall the direction looks good to me, this will be a very useful addition, thanks for working on this.


https://reviews.llvm.org/D46421





More information about the cfe-commits mailing list