[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