ymandel accepted this revision.
ymandel added inline comments.
================
Comment at: clang/unittests/Analysis/FlowSensitive/TestingSupport.h:191
+ AnalysisOutputs AO{AnnotatedCode, Context, Target, CFCtx,
+ Analysis, InitEnv, {}};
+ if (AI.SetupTest) {
----------------
formatting seems funny here?
================
Comment at: clang/unittests/Analysis/FlowSensitive/TestingSupport.h:194
+ auto Error = AI.SetupTest(AO);
+ if (Error) {
+ return Error;
----------------
no braces for a single-statement `if`. could also put the binding in the if
condition: `if (auto Error = `...
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D132377/new/
https://reviews.llvm.org/D132377
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
