[cfe-commits] r130233 - in /cfe/trunk: lib/CodeGen/CGException.cpp test/CodeGenObjC/simplify-exceptions.mm
Daniel Dunbar
daniel at zuster.org
Tue Apr 26 12:43:00 PDT 2011
Author: ddunbar
Date: Tue Apr 26 14:43:00 2011
New Revision: 130233
URL: http://llvm.org/viewvc/llvm-project?rev=130233&view=rev
Log:
IRgen/Darwin: Fix refacto introduced in Triple changes.
Added:
cfe/trunk/test/CodeGenObjC/simplify-exceptions.mm
Modified:
cfe/trunk/lib/CodeGen/CGException.cpp
Modified: cfe/trunk/lib/CodeGen/CGException.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGException.cpp?rev=130233&r1=130232&r2=130233&view=diff
==============================================================================
--- cfe/trunk/lib/CodeGen/CGException.cpp (original)
+++ cfe/trunk/lib/CodeGen/CGException.cpp Tue Apr 26 14:43:00 2011
@@ -266,7 +266,7 @@
/// when it really needs it.
void CodeGenModule::SimplifyPersonality() {
// For now, this is really a Darwin-specific operation.
- if (Context.Target.getTriple().isOSDarwin())
+ if (!Context.Target.getTriple().isOSDarwin())
return;
// If we're not in ObjC++ -fexceptions, there's nothing to do.
Added: cfe/trunk/test/CodeGenObjC/simplify-exceptions.mm
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGenObjC/simplify-exceptions.mm?rev=130233&view=auto
==============================================================================
--- cfe/trunk/test/CodeGenObjC/simplify-exceptions.mm (added)
+++ cfe/trunk/test/CodeGenObjC/simplify-exceptions.mm Tue Apr 26 14:43:00 2011
@@ -0,0 +1,18 @@
+// RUN: %clang_cc1 -triple x86_64-apple-darwin10 -emit-llvm \
+// RUN: -fexceptions -fobjc-exceptions -fobjc-nonfragile-abi \
+// RUN: -o %t %s
+// RUN: FileCheck < %t %s
+//
+// <rdar://problem/7471679> [irgen] [eh] Exception code built with clang (x86_64) crashes
+
+// Check that we don't emit unnecessary personality function references.
+struct t0_A { t0_A(); };
+struct t0_B { t0_A a; };
+
+// CHECK: define {{.*}} @_Z2t0v(){{.*}} {
+// CHECK-NOT: objc_personality
+// CHECK: }
+t0_B& t0() {
+ static t0_B x;
+ return x;
+}
More information about the cfe-commits
mailing list