[PATCH] D146342: [-Wunsafe-buffer-usage] Move the whole analysis to the end of a translation unit

[Artem Dergachev via Phabricator via cfe-commits]

NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.

Ok looks great to me now!



================
Comment at: clang/lib/Sema/AnalysisBasedWarnings.cpp:2364
+                         Node->getBeginLoc())) {
+      UnsafeBufferUsageReporter R(S);
+      clang::checkUnsafeBufferUsage(Node, R, UnsafeBufferEmitFixits);
----------------
So WDYT about capturing the reporter from outside lambda instead?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D146342/new/

https://reviews.llvm.org/D146342

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits