[cfe-commits] r102074 - in /cfe/trunk/lib/Sema: Sema.h SemaExprObjC.cpp TreeTransform.h

Douglas Gregor dgregor at apple.com
Thu Apr 22 09:50:51 PDT 2010


Author: dgregor
Date: Thu Apr 22 11:50:51 2010
New Revision: 102074

URL: http://llvm.org/viewvc/llvm-project?rev=102074&view=rev
Log:
Remove the SelectorLoc argument to Sema::BuildInstanceMesssage and
Sema::BuildClassMessage; we weren't using it, and template
instantiation was faking it anyway.

Modified:
    cfe/trunk/lib/Sema/Sema.h
    cfe/trunk/lib/Sema/SemaExprObjC.cpp
    cfe/trunk/lib/Sema/TreeTransform.h

Modified: cfe/trunk/lib/Sema/Sema.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/Sema.h?rev=102074&r1=102073&r2=102074&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/Sema.h (original)
+++ cfe/trunk/lib/Sema/Sema.h Thu Apr 22 11:50:51 2010
@@ -3887,7 +3887,6 @@
                                      SourceLocation SuperLoc,
                                      Selector Sel,
                                      SourceLocation LBracLoc, 
-                                     SourceLocation SelectorLoc,
                                      SourceLocation RBracLoc,
                                      MultiExprArg Args);
                                      
@@ -3904,7 +3903,6 @@
                                         SourceLocation SuperLoc,
                                         Selector Sel,
                                         SourceLocation LBracLoc, 
-                                        SourceLocation SelectorLoc, 
                                         SourceLocation RBracLoc,
                                         MultiExprArg Args);
 

Modified: cfe/trunk/lib/Sema/SemaExprObjC.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaExprObjC.cpp?rev=102074&r1=102073&r2=102074&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/SemaExprObjC.cpp (original)
+++ cfe/trunk/lib/Sema/SemaExprObjC.cpp Thu Apr 22 11:50:51 2010
@@ -643,16 +643,14 @@
     QualType SuperTy = Context.getObjCInterfaceType(Super);
     SuperTy = Context.getObjCObjectPointerType(SuperTy);
     return BuildInstanceMessage(ExprArg(*this), SuperTy, SuperLoc,
-                                Sel, LBracLoc, SelectorLoc, RBracLoc,
-                                move(Args));
+                                Sel, LBracLoc, RBracLoc, move(Args));
   }
   
   // Since we are in a class method, this is a class message to
   // the superclass.
   return BuildClassMessage(/*ReceiverTypeInfo=*/0,
                            Context.getObjCInterfaceType(Super),
-                           SuperLoc, Sel, LBracLoc, SelectorLoc,
-                           RBracLoc, move(Args));
+                           SuperLoc, Sel, LBracLoc, RBracLoc, move(Args));
 }
 
 /// \brief Build an Objective-C class message expression.
@@ -677,8 +675,6 @@
 ///
 /// \param LBracLoc The location of the opening square bracket ']'.
 ///
-/// \param SelectorLoc The location of the first identifier in the selector.
-///
 /// \param RBrac The location of the closing square bracket ']'.
 ///
 /// \param Args The message arguments.
@@ -687,7 +683,6 @@
                                                SourceLocation SuperLoc,
                                                Selector Sel,
                                                SourceLocation LBracLoc, 
-                                               SourceLocation SelectorLoc,
                                                SourceLocation RBracLoc,
                                                MultiExprArg ArgsIn) {
   if (ReceiverType->isDependentType()) {
@@ -781,7 +776,7 @@
 
   return BuildClassMessage(ReceiverTypeInfo, ReceiverType, 
                            /*SuperLoc=*/SourceLocation(), Sel, 
-                           LBracLoc, SelectorLoc, RBracLoc, move(Args));
+                           LBracLoc, RBracLoc, move(Args));
 }
 
 /// \brief Build an Objective-C instance message expression.
@@ -806,8 +801,6 @@
 ///
 /// \param LBracLoc The location of the opening square bracket ']'.
 ///
-/// \param SelectorLoc The location of the first identifier in the selector.
-///
 /// \param RBrac The location of the closing square bracket ']'.
 ///
 /// \param Args The message arguments.
@@ -816,7 +809,6 @@
                                                   SourceLocation SuperLoc,
                                                   Selector Sel,
                                                   SourceLocation LBracLoc, 
-                                                  SourceLocation SelectorLoc, 
                                                   SourceLocation RBracLoc,
                                                   MultiExprArg ArgsIn) {
   // If we have a receiver expression, perform appropriate promotions
@@ -1001,7 +993,6 @@
 
   return BuildInstanceMessage(move(ReceiverE), Receiver->getType(),
                               /*SuperLoc=*/SourceLocation(),
-                              Sel, LBracLoc, SelectorLoc, RBracLoc,
-                              move(Args));
+                              Sel, LBracLoc, RBracLoc, move(Args));
 }
 

Modified: cfe/trunk/lib/Sema/TreeTransform.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/TreeTransform.h?rev=102074&r1=102073&r2=102074&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/TreeTransform.h (original)
+++ cfe/trunk/lib/Sema/TreeTransform.h Thu Apr 22 11:50:51 2010
@@ -1704,7 +1704,6 @@
                                      /*SuperLoc=*/SourceLocation(),
                                      Sel,
                                      LBracLoc,
-                                     /*FIXME:*/LBracLoc,
                                      RBracLoc,
                                      move(Args));
   }
@@ -1723,7 +1722,6 @@
                                         /*SuperLoc=*/SourceLocation(),
                                         Sel,
                                         LBracLoc,
-                                        /*FIXME:*/LBracLoc,
                                         RBracLoc,
                                         move(Args));
   }
@@ -2339,7 +2337,7 @@
       return QualType();
   }
 
-  BlockPointerLoc NewT = TLB.push<BlockPointerLoc>(Result);
+  BlockPointerTypeLoc NewT = TLB.push<BlockPointerTypeLoc>(Result);
   NewT.setSigilLoc(TL.getSigilLoc());
   return Result;
 }





More information about the cfe-commits mailing list