NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land.
LGTM with one minor nit! ================ Comment at: clang/lib/StaticAnalyzer/Checkers/CastValueChecker.cpp:233-235 + std::string ObjectName = Lexer::getSourceText( + CharSourceRange::getTokenRange(Call.getArgExpr(0)->getSourceRange()), + C.getSourceManager(), C.getASTContext().getLangOpts()); ---------------- Mmm, this will need to be de-duplicated out. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66423/new/ https://reviews.llvm.org/D66423 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
