[PATCH] D27767: NFC Changes from modernize-use-auto

Piotr Padlewski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 15 09:03:35 PST 2016


Prazek added inline comments.


================
Comment at: lib/StaticAnalyzer/Checkers/TestAfterDivZeroChecker.cpp:106
 
   if (Optional<PostStmt> P = Succ->getLocationAs<PostStmt>())
+    if (const auto *BO = P->getStmtAs<BinaryOperator>()) {
----------------
malcolm.parsons wrote:
> The check missed this one.
I don't think it is miss if getLocationAs<T> returns Optional<T>, because that would suggest that it returns T


https://reviews.llvm.org/D27767





More information about the cfe-commits mailing list