]> granicus.if.org Git - clang/commitdiff
[SEH] Fix x64 __exception_code in __except blocks
authorReid Kleckner <rnk@google.com>
Wed, 7 Oct 2015 01:07:13 +0000 (01:07 +0000)
committerReid Kleckner <rnk@google.com>
Wed, 7 Oct 2015 01:07:13 +0000 (01:07 +0000)
Use llvm.eh.exceptioncode to get the code out of EAX for x64. For
32-bit, the filter is responsible for storing it to memory for us.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@249497 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/CGException.cpp
test/CodeGen/exceptions-seh-new.c

index ebeb7270852e432597a70cf62b97d042b4f10960..2490098d67c6026643066fec43c9a4ab5e3a00e2 100644 (file)
@@ -1898,14 +1898,21 @@ void CodeGenFunction::ExitSEHTryStmt(const SEHTryStmt &S) {
     ExceptBB = createBasicBlock("__except");
     Builder.CreateCatchRet(CPI, ExceptBB);
     EmitBlock(ExceptBB);
-  }
-
-  // On Win64, the exception pointer is the exception code. Copy it to the slot.
-  if (CGM.getTarget().getTriple().getArch() != llvm::Triple::x86) {
-    llvm::Value *Code =
-        Builder.CreatePtrToInt(getExceptionFromSlot(), IntPtrTy);
-    Code = Builder.CreateTrunc(Code, Int32Ty);
-    Builder.CreateStore(Code, SEHCodeSlotStack.back());
+    // On Win64, the exception code is returned in EAX. Copy it into the slot.
+    if (CGM.getTarget().getTriple().getArch() != llvm::Triple::x86) {
+      llvm::Function *SEHCodeIntrin =
+          CGM.getIntrinsic(llvm::Intrinsic::eh_exceptioncode);
+      llvm::Value *Code = Builder.CreateCall(SEHCodeIntrin, {CPI});
+      Builder.CreateStore(Code, SEHCodeSlotStack.back());
+    }
+  } else {
+    // On Win64, the exception pointer is the exception code. Copy it to the slot.
+    if (CGM.getTarget().getTriple().getArch() != llvm::Triple::x86) {
+      llvm::Value *Code =
+          Builder.CreatePtrToInt(getExceptionFromSlot(), IntPtrTy);
+      Code = Builder.CreateTrunc(Code, Int32Ty);
+      Builder.CreateStore(Code, SEHCodeSlotStack.back());
+    }
   }
 
   // Emit the __except body.
index 795277bd9b4784d9052b461b9f6992a490c77d22..1a50ed0deaef014d8d64ac361f358839e9bb225f 100644 (file)
@@ -265,4 +265,25 @@ void finally_capture_twice(int x) {
 // CHECK-NEXT:    store i32 [[T0]], i32* [[Z]], align 4
 // CHECK-NEXT:    ret void
 
+int exception_code_in_except(void) {
+  __try {
+    try_body(0, 0, 0);
+  } __except(1) {
+    return _exception_code();
+  }
+}
+
+// CHECK-LABEL: define i32 @exception_code_in_except()
+// CHECK: %[[ret_slot:[^ ]*]] = alloca i32
+// CHECK: %[[code_slot:[^ ]*]] = alloca i32
+// CHECK: invoke void @try_body(i32 0, i32 0, i32* null)
+// CHECK: %[[pad:[^ ]*]] = catchpad
+// CHECK: catchret %[[pad]]
+// X64: %[[code:[^ ]*]] = call i32 @llvm.eh.exceptioncode(token %[[pad]])
+// X64: store i32 %[[code]], i32* %[[code_slot]]
+// CHECK: %[[ret1:[^ ]*]] = load i32, i32* %[[code_slot]]
+// CHECK: store i32 %[[ret1]], i32* %[[ret_slot]]
+// CHECK: %[[ret2:[^ ]*]] = load i32, i32* %[[ret_slot]]
+// CHECK: ret i32 %[[ret2]]
+
 // CHECK: attributes #[[NOINLINE]] = { {{.*noinline.*}} }