steakhal added inline comments.
================ Comment at: clang/lib/StaticAnalyzer/Checkers/ArrayBoundCheckerV2.cpp:46 + void impl(SVal Loc, bool isLoad, const Stmt *S, CheckerContext &C) const; + ---------------- donat.nagy wrote: > I'd call this function `performCheck` or something similar, but "`impl`" is > also fine especially if it's a traditional name (that I haven't encountered > yet in clang SA code). Yes, I'll rename it. I was already thinking about it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D159106/new/ https://reviews.llvm.org/D159106 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits