[PATCH] D71791: [CFG] Fix an assertion failure with static initializers
    Artem Dergachev via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Mon Dec 23 15:28:17 PST 2019
    
    
  
NoQ accepted this revision.
NoQ added a comment.
Woohoo tests!~~
================
Comment at: clang/unittests/Analysis/CFGBuildResult.h:69
+  Finder.matchAST(AST->getASTContext());
+  Callback.TheBuildResult.setAST(std::move(AST));
   return std::move(Callback.TheBuildResult);
----------------
It should be slightly more visually appealing to construct the AST before the callback and pass it into the constructor of the callback.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71791/new/
https://reviews.llvm.org/D71791
    
    
More information about the cfe-commits
mailing list