diff options
| author | Saleem Abdulrasool <compnerd@compnerd.org> | 2014-10-14 17:20:18 +0000 |
|---|---|---|
| committer | Saleem Abdulrasool <compnerd@compnerd.org> | 2014-10-14 17:20:18 +0000 |
| commit | 3b43447619d815e1918e683318538a6f5cde087a (patch) | |
| tree | a63dd5264305aa6c2daf3a19d613dd681549b1d3 | |
| parent | 64ab4de44339d84e8ecf578e260b68aeb142187b (diff) | |
| download | bcm5719-llvm-3b43447619d815e1918e683318538a6f5cde087a.tar.gz bcm5719-llvm-3b43447619d815e1918e683318538a6f5cde087a.zip | |
CodeGen: correct block mangling in ObjC
Mangling for blocks defined within blocks in an ObjectiveC context were also
broken by SVN r219393. Because the code in mangleName assumed that the code was
either C or C++, we would trigger assertions when trying to mangle the inner
blocks in an ObjectiveC context.
Add a test and use the ObjectiveC specific mangling when dealing with an
ObjectiveC method declaration.
llvm-svn: 219697
| -rw-r--r-- | clang/lib/AST/Mangle.cpp | 7 | ||||
| -rw-r--r-- | clang/test/CodeGenObjC/mangle-blocks.m | 29 |
2 files changed, 35 insertions, 1 deletions
diff --git a/clang/lib/AST/Mangle.cpp b/clang/lib/AST/Mangle.cpp index fba835451e2..b5dbdca9c97 100644 --- a/clang/lib/AST/Mangle.cpp +++ b/clang/lib/AST/Mangle.cpp @@ -135,7 +135,10 @@ void MangleContext::mangleName(const NamedDecl *D, raw_ostream &Out) { bool MCXX = shouldMangleCXXName(D); const TargetInfo &TI = Context.getTargetInfo(); if (CC == SOF_OTHER || (MCXX && TI.getCXXABI() == TargetCXXABI::Microsoft)) { - mangleCXXName(D, Out); + if (const ObjCMethodDecl *OMD = dyn_cast<ObjCMethodDecl>(D)) + mangleObjCMethodName(OMD, Out); + else + mangleCXXName(D, Out); return; } @@ -147,6 +150,8 @@ void MangleContext::mangleName(const NamedDecl *D, raw_ostream &Out) { if (!MCXX) Out << D->getIdentifier()->getName(); + else if (const ObjCMethodDecl *OMD = dyn_cast<ObjCMethodDecl>(D)) + mangleObjCMethodName(OMD, Out); else mangleCXXName(D, Out); diff --git a/clang/test/CodeGenObjC/mangle-blocks.m b/clang/test/CodeGenObjC/mangle-blocks.m new file mode 100644 index 00000000000..f0339c15308 --- /dev/null +++ b/clang/test/CodeGenObjC/mangle-blocks.m @@ -0,0 +1,29 @@ +// RUN: %clang_cc1 -triple i386-apple-ios -fblocks -emit-llvm -o - %s -Wno-objc-root-class \ +// RUN: | FileCheck %s + +void __assert_rtn(const char *, const char *, int, const char *); + +@interface Test +- (void (^)(void)) mangle; +@end + +@implementation Test +- (void (^)(void)) mangle { + return ^() { + void (^b)(void) = ^() { + __assert_rtn(__func__, __FILE__, __LINE__, "mangle"); + }; + }; +} +@end + +// CHECK: @"__func__.__14-[Test mangle]_block_invoke_2" = private unnamed_addr constant [34 x i8] c"__14-[Test mangle]_block_invoke_2\00", align 1 +// CHECK: @.str = private unnamed_addr constant {{.*}}, align 1 +// CHECK: @.str1 = private unnamed_addr constant [7 x i8] c"mangle\00", align 1 + +// CHECK: define internal void @"__14-[Test mangle]_block_invoke"(i8* %.block_descriptor) + +// CHECK: define internal void @"__14-[Test mangle]_block_invoke_2"(i8* %.block_descriptor){{.*}}{ +// CHECK: call void @__assert_rtn(i8* getelementptr inbounds ([34 x i8]* @"__func__.__14-[Test mangle]_block_invoke_2", i32 0, i32 0), i8* getelementptr inbounds {{.*}}, i32 14, i8* getelementptr inbounds ([7 x i8]* @.str1, i32 0, i32 0)) +// CHECK: } + |

