From: Chris Lattner Date: Tue, 31 Mar 2009 08:55:07 +0000 (+0000) Subject: do not *copy* objc interface types, just use their reference. X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=9a1a9c41cf7e2735f059bb0c1d85f3207f637b22;p=clang do not *copy* objc interface types, just use their reference. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@68102 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/CodeGen/CodeGenTypes.cpp b/lib/CodeGen/CodeGenTypes.cpp index 5367c71b2c..62a77448be 100644 --- a/lib/CodeGen/CodeGenTypes.cpp +++ b/lib/CodeGen/CodeGenTypes.cpp @@ -322,12 +322,11 @@ const llvm::Type *CodeGenTypes::ConvertNewType(QualType T) { } // The function type can be built; call the appropriate routines to // build it. - if (const FunctionProtoType *FPT = dyn_cast(&Ty)) { + if (const FunctionProtoType *FPT = dyn_cast(&Ty)) return GetFunctionType(getFunctionInfo(FPT), FPT->isVariadic()); - } else { - const FunctionNoProtoType *FNPT = cast(&Ty); - return GetFunctionType(getFunctionInfo(FNPT), true); - } + + const FunctionNoProtoType *FNPT = cast(&Ty); + return GetFunctionType(getFunctionInfo(FNPT), true); } case Type::ExtQual: @@ -341,16 +340,13 @@ const llvm::Type *CodeGenTypes::ConvertNewType(QualType T) { // previously checked that the ObjCRuntime subclass in use does not support // late-bound ivars. // We are issuing warnings elsewhere! - ObjCInterfaceType OIT = cast(Ty); - ObjCInterfaceDecl *ID = OIT.getDecl(); - const RecordDecl *RD = Context.addRecordToClass(ID); - return ConvertTagDeclType(cast(RD)); + ObjCInterfaceDecl *ID = cast(Ty).getDecl(); + return ConvertTagDeclType(Context.addRecordToClass(ID)); } case Type::ObjCQualifiedInterface: { - ObjCQualifiedInterfaceType QIT = cast(Ty); - - return ConvertTypeRecursive(Context.getObjCInterfaceType(QIT.getDecl())); + ObjCInterfaceDecl *ID = cast(Ty).getDecl(); + return ConvertTypeRecursive(Context.getObjCInterfaceType(ID)); } case Type::ObjCQualifiedId: