[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