Szelethus added a comment.
The code from in `ExprInspectionChecker.cpp` is duplicated from D75677
<https://reviews.llvm.org/D75677>, isn't it?
================
Comment at: clang/lib/StaticAnalyzer/Checkers/ExprInspectionChecker.cpp:55
- ExplodedNode *reportBug(llvm::StringRef Msg, CheckerContext &C) const;
+ // Optional parameter `ExprVal` for expression value to be marked interesting
+ ExplodedNode *reportBug(llvm::StringRef Msg, CheckerContext &C,
----------------
Please terminate this sentence.
================
Comment at: clang/test/Analysis/container-modeling.cpp:100-101
- V.emplace_back(n); // expected-note 2{{Container 'V' extended to the right
by 1 position}}
+ V.emplace_back(n); // expected-note{{Container 'V' extended to the right by
1 position}}
+ // expected-note@-1{{Container 'V' extended to the right
by 1 position}}
----------------
Why did we unpack this? And the rest?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75514/new/
https://reviews.llvm.org/D75514
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
