[PATCH] D69726: [analyzer] DynamicSize: Store the dynamic size
Artem Dergachev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Apr 1 20:35:53 PDT 2021
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
I don't see any further bugs so I think this is good to go!
I think you should separate out the NFC / debugging facility changes before committing.
================
Comment at: clang/lib/StaticAnalyzer/Checkers/ExprInspectionChecker.cpp:294
+
+ SmallString<128> Msg;
+ llvm::raw_svector_ostream Out(Msg);
----------------
128 is too much imho :)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69726/new/
https://reviews.llvm.org/D69726
More information about the cfe-commits
mailing list