[PATCH] D40566: Check if MemberExpr arguments are type-dependent.

Erik Pilkington via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 29 10:20:19 PST 2017


erik.pilkington added a comment.

Hi, thanks for working on this!
Why don't we just set the TypeDependent bit to false when building the underlying MemberExpr here? Based on the section of the standard you quoted that node shouldn't be considered type-dependent, but it is by clang. I think this patch is just hiding that underlying problem by fishing out the real type-dependence after constructing the MemberExpr when really that information should be included in the MemberExpr to begin with.
Thanks,
Erik


https://reviews.llvm.org/D40566





More information about the cfe-commits mailing list