NoQ added inline comments.
================
Comment at: lib/StaticAnalyzer/Checkers/InnerPointerChecker.cpp:207-208
+
+ for (unsigned I = 0, E = FD->getNumParams(); I != E; ++I) {
+ QualType ParamTy = FD->getParamDecl(I)->getType();
+ if (!ParamTy->isReferenceType() ||
----------------
NoQ wrote:
> We need tests for operators here, due to the problem that i recently
> encountered in D49627: there's different numbering for arguments and
> parameters within in-class operators. Namely, this-argument is counted as an
> argument (shifting other argument indices by 1) but not as a parameter.
(i.e., tests and most likely some actual code branch)
https://reviews.llvm.org/D49656
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
