[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