Enable some more operator overloading tests, and don't look into an identifier for functions that might not have one

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@59818 91177308-0d34-0410-b5e6-96231b3b80d8

lib/AST/Expr.cpp

@@ -168,6 +168,9 @@ unsigned CallExpr::isBuiltinCall() const {
   if (!FDecl)
     return 0;
   
+  if (!FDecl->getIdentifier())
+    return 0;
+
   return FDecl->getIdentifier()->getBuiltinID();
 }
 

test/SemaCXX/overloaded-builtin-operators.cpp

@@ -89,14 +89,10 @@ struct ConstIntPtrRef {
 
 void test_with_ptrs(VolatileIntPtr vip, ConstIntPtr cip, ShortRef sr,
                     VolatileIntPtrRef vipr, ConstIntPtrRef cipr) {
-#if 0
-  // FIXME: Enable these tests once we have operator overloading for
-  // operator[].
   const int& cir1 = cip[sr];
   const int& cir2 = sr[cip];
   volatile int& vir1 = vip[sr];
   volatile int& vir2 = sr[vip];
-#endif
   bool b1 = (vip == cip);
   long p1 = vip - cip;