[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