================
@@ -7939,7 +7927,14 @@ ExprResult Sema::ActOnStartCXXMemberReference(Scope *S, 
Expr *Base,
           }
         }
         return ExprError();
+      } else if (Result.isUnset()) {
+        // BuildOverloadedArrowExpr returns an empty expression to indicate
+        // that we need to build a dependent overloaded arrow expression.
----------------
cor3ntin wrote:
I have 2 concerns with that
 1/ It's a bit too clever of an interface
 2/ We should make sure this can only happens in Sema, right? I wonder if we 
need an ActOnOverloadedArrowExpr to wrap that logic such that (it would 
duplicate the lookup code but i think that's a better outcome)

WDYT?

https://github.com/llvm/llvm-project/pull/104458
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to