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

Ziqing Luo via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 5 13:21:07 PDT 2023


ziqingluo-90 marked an inline comment as done.
ziqingluo-90 added inline comments.


================
Comment at: clang/lib/Sema/AnalysisBasedWarnings.cpp:2363-2364
+      if (isa<LambdaExpr>(Node)) {
+        // to visit implicit children of `LambdaExpr`s:
+        IsVisitingLambda = true;
+
----------------
NoQ wrote:
> I suspect that you might have reinvented `shouldVisitLambdaBody()`.
I tried `shouldVisitLambdaBody`  but it didn't work (I didn't look into why though)


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

https://reviews.llvm.org/D146342



More information about the cfe-commits mailing list