[cfe-commits] r95244 - in /cfe/trunk: include/clang/AST/ExprObjC.h lib/AST/Expr.cpp lib/CodeGen/CGExprConstant.cpp lib/CodeGen/CGObjC.cpp lib/CodeGen/CGObjCGNU.cpp lib/CodeGen/CGObjCMac.cpp lib/CodeGen/CGObjCRuntime.h lib/Sema/SemaExprObjC.cpp

Daniel Dunbar daniel at zuster.org
Wed Feb 3 12:11:43 PST 2010


Author: ddunbar
Date: Wed Feb  3 14:11:42 2010
New Revision: 95244

URL: http://llvm.org/viewvc/llvm-project?rev=95244&view=rev
Log:
Revert "Numerous changes to selector handling:", this breaks a whole bunch of
working code, for no apparent reason.

Modified:
    cfe/trunk/include/clang/AST/ExprObjC.h
    cfe/trunk/lib/AST/Expr.cpp
    cfe/trunk/lib/CodeGen/CGExprConstant.cpp
    cfe/trunk/lib/CodeGen/CGObjC.cpp
    cfe/trunk/lib/CodeGen/CGObjCGNU.cpp
    cfe/trunk/lib/CodeGen/CGObjCMac.cpp
    cfe/trunk/lib/CodeGen/CGObjCRuntime.h
    cfe/trunk/lib/Sema/SemaExprObjC.cpp

Modified: cfe/trunk/include/clang/AST/ExprObjC.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/AST/ExprObjC.h?rev=95244&r1=95243&r2=95244&view=diff

==============================================================================
--- cfe/trunk/include/clang/AST/ExprObjC.h (original)
+++ cfe/trunk/include/clang/AST/ExprObjC.h Wed Feb  3 14:11:42 2010
@@ -96,22 +96,18 @@
 /// ObjCSelectorExpr used for @selector in Objective-C.
 class ObjCSelectorExpr : public Expr {
   Selector SelName;
-  ObjCMethodDecl *Method;
   SourceLocation AtLoc, RParenLoc;
 public:
   ObjCSelectorExpr(QualType T, Selector selInfo,
                    SourceLocation at, SourceLocation rp)
-  : Expr(ObjCSelectorExprClass, T, false, false), SelName(selInfo), Method(0),
-    AtLoc(at), RParenLoc(rp){}
+  : Expr(ObjCSelectorExprClass, T, false, false), SelName(selInfo), AtLoc(at),
+    RParenLoc(rp){}
   explicit ObjCSelectorExpr(EmptyShell Empty)
    : Expr(ObjCSelectorExprClass, Empty) {}
 
   Selector getSelector() const { return SelName; }
   void setSelector(Selector S) { SelName = S; }
 
-  ObjCMethodDecl *getMethodDecl() const { return Method; }
-  void setMethodDecl(ObjCMethodDecl *M) { Method = M; }
-
   SourceLocation getAtLoc() const { return AtLoc; }
   SourceLocation getRParenLoc() const { return RParenLoc; }
   void setAtLoc(SourceLocation L) { AtLoc = L; }

Modified: cfe/trunk/lib/AST/Expr.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/Expr.cpp?rev=95244&r1=95243&r2=95244&view=diff

==============================================================================
--- cfe/trunk/lib/AST/Expr.cpp (original)
+++ cfe/trunk/lib/AST/Expr.cpp Wed Feb  3 14:11:42 2010
@@ -1454,7 +1454,6 @@
   case StringLiteralClass:
   case ObjCStringLiteralClass:
   case ObjCEncodeExprClass:
-  case ObjCSelectorExprClass:
     return true;
   case CompoundLiteralExprClass: {
     // This handles gcc's extension that allows global initializers like

Modified: cfe/trunk/lib/CodeGen/CGExprConstant.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGExprConstant.cpp?rev=95244&r1=95243&r2=95244&view=diff

==============================================================================
--- cfe/trunk/lib/CodeGen/CGExprConstant.cpp (original)
+++ cfe/trunk/lib/CodeGen/CGExprConstant.cpp Wed Feb  3 14:11:42 2010
@@ -701,14 +701,6 @@
                                     CGM.GetStringForStringLiteral(E), false);
   }
 
-  llvm::Constant *VisitObjCSelectorExpr(const ObjCSelectorExpr *E) {
-    ObjCMethodDecl *OMD = E->getMethodDecl();
-    if (OMD)
-      return CGM.getObjCRuntime().GetConstantTypedSelector(OMD);
-    else 
-      return CGM.getObjCRuntime().GetConstantSelector(E->getSelector());
-  }
-
   llvm::Constant *VisitObjCEncodeExpr(ObjCEncodeExpr *E) {
     // This must be an @encode initializing an array in a static initializer.
     // Don't emit it as the address of the string, emit the string data itself

Modified: cfe/trunk/lib/CodeGen/CGObjC.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGObjC.cpp?rev=95244&r1=95243&r2=95244&view=diff

==============================================================================
--- cfe/trunk/lib/CodeGen/CGObjC.cpp (original)
+++ cfe/trunk/lib/CodeGen/CGObjC.cpp Wed Feb  3 14:11:42 2010
@@ -38,11 +38,7 @@
   // Note that this implementation allows for non-constant strings to be passed
   // as arguments to @selector().  Currently, the only thing preventing this
   // behaviour is the type checking in the front end.
-  ObjCMethodDecl *OMD = E->getMethodDecl();
-  if (OMD)
-    return CGM.getObjCRuntime().GetSelector(Builder, OMD);
-  else 
-    return CGM.getObjCRuntime().GetSelector(Builder, E->getSelector());
+  return CGM.getObjCRuntime().GetSelector(Builder, E->getSelector());
 }
 
 llvm::Value *CodeGenFunction::EmitObjCProtocolExpr(const ObjCProtocolExpr *E) {

Modified: cfe/trunk/lib/CodeGen/CGObjCGNU.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGObjCGNU.cpp?rev=95244&r1=95243&r2=95244&view=diff

==============================================================================
--- cfe/trunk/lib/CodeGen/CGObjCGNU.cpp (original)
+++ cfe/trunk/lib/CodeGen/CGObjCGNU.cpp Wed Feb  3 14:11:42 2010
@@ -156,17 +156,9 @@
                            const ObjCMethodDecl *Method);
   virtual llvm::Value *GetClass(CGBuilderTy &Builder,
                                 const ObjCInterfaceDecl *OID);
-  virtual llvm::Constant *GetConstantSelector(Selector Sel);
-  virtual llvm::Constant *GetConstantTypedSelector(
-     const ObjCMethodDecl *Method);
-  llvm::Value *GetSelector(CGBuilderTy &Builder,
-                           Selector Sel) {
-    return cast<llvm::Constant>((GetConstantSelector(Sel)));
-  }
-  llvm::Value *GetSelector(CGBuilderTy &Builder,
-                           const ObjCMethodDecl *Method) {
-    return cast<llvm::Constant>(GetConstantTypedSelector(Method));
-  }
+  virtual llvm::Value *GetSelector(CGBuilderTy &Builder, Selector Sel);
+  virtual llvm::Value *GetSelector(CGBuilderTy &Builder, const ObjCMethodDecl
+      *Method);
 
   virtual llvm::Function *GenerateMethod(const ObjCMethodDecl *OMD,
                                          const ObjCContainerDecl *CD);
@@ -347,18 +339,18 @@
   return Builder.CreateCall(ClassLookupFn, ClassName);
 }
 
-llvm::Constant *CGObjCGNU::GetConstantSelector(Selector Sel) {
+llvm::Value *CGObjCGNU::GetSelector(CGBuilderTy &Builder, Selector Sel) {
   llvm::GlobalAlias *&US = UntypedSelectors[Sel.getAsString()];
   if (US == 0)
-    US = new llvm::GlobalAlias(SelectorTy,
+    US = new llvm::GlobalAlias(llvm::PointerType::getUnqual(SelectorTy),
                                llvm::GlobalValue::PrivateLinkage,
                                ".objc_untyped_selector_alias"+Sel.getAsString(),
                                NULL, &TheModule);
 
-  return US;
+  return Builder.CreateLoad(US);
 }
 
-llvm::Constant *CGObjCGNU::GetConstantTypedSelector(const ObjCMethodDecl
+llvm::Value *CGObjCGNU::GetSelector(CGBuilderTy &Builder, const ObjCMethodDecl
     *Method) {
 
   std::string SelName = Method->getSelector().getAsString();
@@ -370,17 +362,17 @@
 
   // If it's already cached, return it.
   if (TypedSelectors[Selector]) {
-    return TypedSelectors[Selector];
+    return Builder.CreateLoad(TypedSelectors[Selector]);
   }
 
   // If it isn't, cache it.
   llvm::GlobalAlias *Sel = new llvm::GlobalAlias(
-          SelectorTy,
+          llvm::PointerType::getUnqual(SelectorTy),
           llvm::GlobalValue::PrivateLinkage, ".objc_selector_alias" + SelName,
           NULL, &TheModule);
   TypedSelectors[Selector] = Sel;
 
-  return Sel;
+  return Builder.CreateLoad(Sel);
 }
 
 llvm::Constant *CGObjCGNU::MakeConstantString(const std::string &Str,
@@ -1537,43 +1529,40 @@
 
   // Now that all of the static selectors exist, create pointers to them.
   int index = 0;
-  llvm::SmallVector<std::pair<llvm::GlobalAlias*,llvm::Value*>, 16> selectors;
   for (std::map<TypedSelector, llvm::GlobalAlias*>::iterator
      iter=TypedSelectors.begin(), iterEnd =TypedSelectors.end();
      iter != iterEnd; ++iter) {
     llvm::Constant *Idxs[] = {Zeros[0],
       llvm::ConstantInt::get(llvm::Type::getInt32Ty(VMContext), index++), Zeros[0]};
-    llvm::Constant *SelPtr = 
-        llvm::ConstantExpr::getGetElementPtr(SelectorList, Idxs, 2);
+    llvm::Constant *SelPtr = new llvm::GlobalVariable(TheModule, SelStructPtrTy,
+        true, llvm::GlobalValue::InternalLinkage,
+        llvm::ConstantExpr::getGetElementPtr(SelectorList, Idxs, 2),
+        ".objc_sel_ptr");
     // If selectors are defined as an opaque type, cast the pointer to this
     // type.
     if (isSelOpaque) {
-      SelPtr = llvm::ConstantExpr::getBitCast(SelPtr,SelectorTy);
+      SelPtr = llvm::ConstantExpr::getBitCast(SelPtr,
+        llvm::PointerType::getUnqual(SelectorTy));
     }
-    selectors.push_back(
-        std::pair<llvm::GlobalAlias*,llvm::Value*>((*iter).second, SelPtr));
+    (*iter).second->setAliasee(SelPtr);
   }
   for (llvm::StringMap<llvm::GlobalAlias*>::iterator
       iter=UntypedSelectors.begin(), iterEnd = UntypedSelectors.end();
       iter != iterEnd; iter++) {
     llvm::Constant *Idxs[] = {Zeros[0],
       llvm::ConstantInt::get(llvm::Type::getInt32Ty(VMContext), index++), Zeros[0]};
-    llvm::Constant *SelPtr = 
-       llvm::ConstantExpr::getGetElementPtr(SelectorList, Idxs, 2);
+    llvm::Constant *SelPtr = new llvm::GlobalVariable
+      (TheModule, SelStructPtrTy,
+       true, llvm::GlobalValue::InternalLinkage,
+       llvm::ConstantExpr::getGetElementPtr(SelectorList, Idxs, 2),
+       ".objc_sel_ptr");
     // If selectors are defined as an opaque type, cast the pointer to this
     // type.
     if (isSelOpaque) {
-      SelPtr = llvm::ConstantExpr::getBitCast(SelPtr, SelectorTy);
+      SelPtr = llvm::ConstantExpr::getBitCast(SelPtr,
+        llvm::PointerType::getUnqual(SelectorTy));
     }
-    selectors.push_back(
-        std::pair<llvm::GlobalAlias*,llvm::Value*>((*iter).second, SelPtr));
-  }
-  for (llvm::SmallVectorImpl<std::pair<
-            llvm::GlobalAlias*,llvm::Value*> >::iterator
-     iter=selectors.begin(), iterEnd =selectors.end();
-     iter != iterEnd; ++iter) {
-    iter->first->replaceAllUsesWith(iter->second);
-    iter->first->eraseFromParent();
+    (*iter).second->setAliasee(SelPtr);
   }
   // Number of classes defined.
   Elements.push_back(llvm::ConstantInt::get(llvm::Type::getInt16Ty(VMContext),

Modified: cfe/trunk/lib/CodeGen/CGObjCMac.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGObjCMac.cpp?rev=95244&r1=95243&r2=95244&view=diff

==============================================================================
--- cfe/trunk/lib/CodeGen/CGObjCMac.cpp (original)
+++ cfe/trunk/lib/CodeGen/CGObjCMac.cpp Wed Feb  3 14:11:42 2010
@@ -953,14 +953,6 @@
   CGObjCCommonMac(CodeGen::CodeGenModule &cgm) :
     CGM(cgm), VMContext(cgm.getLLVMContext()) { }
 
-  virtual llvm::Constant *GetConstantSelector(Selector Sel) {
-    assert(0 && "Constant Selectors are not yet supported on the Mac runtimes");
-    return 0;
-  }
-  virtual llvm::Constant *GetConstantTypedSelector(
-     const ObjCMethodDecl *Method) {
-    return GetConstantSelector(Method->getSelector());
-  }
   virtual llvm::Constant *GenerateConstantString(const StringLiteral *SL);
 
   virtual llvm::Function *GenerateMethod(const ObjCMethodDecl *OMD,

Modified: cfe/trunk/lib/CodeGen/CGObjCRuntime.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGObjCRuntime.h?rev=95244&r1=95243&r2=95244&view=diff

==============================================================================
--- cfe/trunk/lib/CodeGen/CGObjCRuntime.h (original)
+++ cfe/trunk/lib/CodeGen/CGObjCRuntime.h Wed Feb  3 14:11:42 2010
@@ -95,12 +95,6 @@
   /// this compilation unit with the runtime library.
   virtual llvm::Function *ModuleInitFunction() = 0;
 
-  virtual llvm::Constant *GetConstantSelector(Selector Sel) = 0;
-
-  /// Get a typed selector.
-  virtual llvm::Constant *GetConstantTypedSelector(
-     const ObjCMethodDecl *Method) = 0;
-
   /// Get a selector for the specified name and type values. The
   /// return value should have the LLVM type for pointer-to
   /// ASTContext::getObjCSelType().

Modified: cfe/trunk/lib/Sema/SemaExprObjC.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaExprObjC.cpp?rev=95244&r1=95243&r2=95244&view=diff

==============================================================================
--- cfe/trunk/lib/Sema/SemaExprObjC.cpp (original)
+++ cfe/trunk/lib/Sema/SemaExprObjC.cpp Wed Feb  3 14:11:42 2010
@@ -140,20 +140,7 @@
     Diag(SelLoc, diag::warn_undeclared_selector) << Sel;
 
   QualType Ty = Context.getObjCSelType();
-  ObjCSelectorExpr *E = 
-      new (Context) ObjCSelectorExpr(Ty, Sel, AtLoc, RParenLoc);
-  // Make sure that we have seen this selector.  There are lots of checks we
-  // should be doing on this selector.  For example, when this is passed as the
-  // second argument to objc_msgSend() on the Mac runtime, or as the selector
-  // argument to the -performSelector:.  We can do these checks at run time
-  // with the GNU runtimes, but the Apple runtimes let you sneak stack
-  // corruption in easily by passing the wrong selector to these functions if
-  // there is no static checking.
-  //
-  // Only log a warning on the GNU runtime.
-  E->setMethodDecl(LookupInstanceMethodInGlobalPool(Sel, 
-      SourceRange(LParenLoc,  LParenLoc), !LangOpts.NeXTRuntime));
-  return E;
+  return new (Context) ObjCSelectorExpr(Ty, Sel, AtLoc, RParenLoc);
 }
 
 Sema::ExprResult Sema::ParseObjCProtocolExpression(IdentifierInfo *ProtocolId,





More information about the cfe-commits mailing list