AbbasSabra marked 2 inline comments as done.
AbbasSabra added a comment.
> Please, add these reviewers for your upcoming [analyzer] patches.
Thanks, I was planning to add reviewers after making sure that the pre-checks
are green.
> Nice to see some fixes for visual c++ stuff.
Thanks for the quick review!
================
Comment at: clang/lib/StaticAnalyzer/Core/CoreEngine.cpp:352
+ case Stmt::SEHLeaveStmtClass:
case Stmt::ContinueStmtClass:
----------------
steakhal wrote:
> You should probably extend the `ExprEngine.cpp:1312` in a similar fashion.
isn't it already handled in !!ExprEngine.cpp:1239!!?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D102280/new/
https://reviews.llvm.org/D102280
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
