erichkeane added a comment. T
================ Comment at: clang/lib/Sema/SemaExpr.cpp:2671 // Check whether this might be a C++ implicit instance member access. // C++ [class.mfct.non-static]p3: ---------------- So what I was asking above: You're making a ton of changes to this section, without properly updating the comment that is trying to explain what is happening. Please update this to match, particularly the bit around the % operands. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137531/new/ https://reviews.llvm.org/D137531 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits