[PATCH] D137531: [clang] Add the check of membership in decltype for the issue #58674

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 15 08:36:02 PST 2022


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



More information about the cfe-commits mailing list