commit 0daabd7ebe5e51bc87808150f23e41432ababb2c
author Richard Smith <richard-llvm@metafoo.co.uk> Tue Sep 23 20:31:39 2014 +0000
committer Richard Smith <richard-llvm@metafoo.co.uk> Tue Sep 23 20:31:39 2014 +0000
tree 40785b42fa4897194eee6e5a2aff7ce0834322de
parent dedef3325f08e506e5b07d167579f39f817015f6

Don't perform ADL when looking up operator=; there is no non-member form of
that function, and apart from being slow, this is unnecessary: ADL can trigger
instantiations that are not permitted here. The standard isn't *completely*
clear here, but this seems like the intent, and in any case this approach is
permitted by [temp.inst]p7.

llvm-svn: 218330

clang/lib/Sema/SemaExpr.cpp

diff --git a/clang/lib/Sema/SemaExpr.cpp b/clang/lib/Sema/SemaExpr.cpp
index 50bc4d2..5edabc0 100644
--- a/clang/lib/Sema/SemaExpr.cpp
+++ b/clang/lib/Sema/SemaExpr.cpp
@@ -9704,7 +9704,7 @@
   UnresolvedSet<16> Functions;
   OverloadedOperatorKind OverOp
     = BinaryOperator::getOverloadedOperator(Opc);
-  if (Sc && OverOp != OO_None)
+  if (Sc && OverOp != OO_None && OverOp != OO_Equal)
     S.LookupOverloadedOperatorName(OverOp, Sc, LHS->getType(),
                                    RHS->getType(), Functions);