Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[WebAssembly] Make Emscripten EH work with Emscripten SjLj
When Emscripten EH mixes with Emscripten SjLj, we are not currently handling some of them correctly. There are three cases: 1. The current function calls `setjmp` and there is an `invoke` to a function that can either throw or longjmp. In this case, we have to check both for exception and longjmp. We are currently handling this case correctly: https://github.com/llvm/llvm-project/blob/0c0eb76782d5224b8d81a5afbb9a152bcf7c94c7/llvm/lib/Target/WebAssembly/WebAssemblyLowerEmscriptenEHSjLj.cpp#L1058-L1090 When inserting routines for functions that can longjmp, which we do only for setjmp-calling functions, we check if the function was previously an `invoke` and handle it correctly. 2. The current function does NOT call `setjmp` and there is an `invoke` to a function that can either throw or longjmp. Because there is no `setjmp` call, we haven't been doing any check for functions that can longjmp. But in that case, for `invoke`, we only check for an exception and if it is not an exception we reset `__THREW__` to 0, which can silently swallow the longjmp: https://github.com/llvm/llvm-project/blob/0c0eb76782d5224b8d81a5afbb9a152bcf7c94c7/llvm/lib/Target/WebAssembly/WebAssemblyLowerEmscriptenEHSjLj.cpp#L70-L80 This CL fixes this. 3. The current function calls `setjmp` and there is no `invoke`. Because it is not an `invoke`, we haven't been doing any check for functions that can throw, and only insert longjmp-checking routines for functions that can longjmp. But in that case, if a longjmpable function throws, we only check for a longjmp so if it is not a longjmp we reset `__THREW__` to 0, which can silently swallow the exception: https://github.com/llvm/llvm-project/blob/0c0eb76782d5224b8d81a5afbb9a152bcf7c94c7/llvm/lib/Target/WebAssembly/WebAssemblyLowerEmscriptenEHSjLj.cpp#L156-L169 This CL fixes this. To do that, this moves around some code, so we register necessary functions for both EH and SjLj and precompute some data (the set of functions that contains `setjmp`) before doing actual EH or SjLj transformation. This CL makes 2nd and 3rd tests in emscripten-core/emscripten#14732 work. Reviewed By: dschuff Differential Revision: https://reviews.llvm.org/D106525
- Loading branch information
Showing
4 changed files
with
249 additions
and
63 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,132 @@ | ||
; RUN: opt < %s -wasm-lower-em-ehsjlj -S | FileCheck %s | ||
; RUN: llc < %s | ||
|
||
; Tests for cases when exception handling and setjmp/longjmp handling are mixed. | ||
|
||
target datalayout = "e-m:e-p:32:32-i64:64-n32:64-S128" | ||
target triple = "wasm32-unknown-unknown" | ||
|
||
%struct.__jmp_buf_tag = type { [6 x i32], i32, [32 x i32] } | ||
|
||
; There is a function call (@foo) that can either throw an exception or longjmp | ||
; and there is also a setjmp call. When @foo throws, we have to check both for | ||
; exception and longjmp and jump to exception or longjmp handling BB depending | ||
; on the result. | ||
define void @setjmp_longjmp_exception() personality i8* bitcast (i32 (...)* @__gxx_personality_v0 to i8*) { | ||
; CHECK-LABEL: @setjmp_longjmp_exception | ||
entry: | ||
%buf = alloca [1 x %struct.__jmp_buf_tag], align 16 | ||
%arraydecay = getelementptr inbounds [1 x %struct.__jmp_buf_tag], [1 x %struct.__jmp_buf_tag]* %buf, i32 0, i32 0 | ||
%call = call i32 @setjmp(%struct.__jmp_buf_tag* %arraydecay) #0 | ||
invoke void @foo() | ||
to label %try.cont unwind label %lpad | ||
|
||
; CHECK: entry.split: | ||
; CHECK: %[[CMP0:.*]] = icmp ne i32 %__THREW__.val, 0 | ||
; CHECK-NEXT: %__threwValue.val = load i32, i32* @__threwValue | ||
; CHECK-NEXT: %[[CMP1:.*]] = icmp ne i32 %__threwValue.val, 0 | ||
; CHECK-NEXT: %[[CMP:.*]] = and i1 %[[CMP0]], %[[CMP1]] | ||
; CHECK-NEXT: br i1 %[[CMP]], label %if.then1, label %if.else1 | ||
|
||
; This is exception checking part. %if.else1 leads here | ||
; CHECK: entry.split.split: | ||
; CHECK-NEXT: %[[CMP:.*]] = icmp eq i32 %__THREW__.val, 1 | ||
; CHECK-NEXT: br i1 %[[CMP]], label %lpad, label %try.cont | ||
|
||
; longjmp checking part | ||
; CHECK: if.then1: | ||
; CHECK: call i32 @testSetjmp | ||
|
||
lpad: ; preds = %entry | ||
%0 = landingpad { i8*, i32 } | ||
catch i8* null | ||
%1 = extractvalue { i8*, i32 } %0, 0 | ||
%2 = extractvalue { i8*, i32 } %0, 1 | ||
%3 = call i8* @__cxa_begin_catch(i8* %1) #2 | ||
call void @__cxa_end_catch() | ||
br label %try.cont | ||
|
||
try.cont: ; preds = %entry, %lpad | ||
ret void | ||
} | ||
|
||
; @foo can either throw an exception or longjmp. Because this function doesn't | ||
; have any setjmp calls, we only handle exceptions in this function. But because | ||
; sjlj is enabled, we check if the thrown value is longjmp and if so rethrow it | ||
; by calling @emscripten_longjmp. | ||
define void @rethrow_longjmp() personality i8* bitcast (i32 (...)* @__gxx_personality_v0 to i8*) { | ||
; CHECK-LABEL: @rethrow_longjmp | ||
entry: | ||
invoke void @foo() | ||
to label %try.cont unwind label %lpad | ||
; CHECK: entry: | ||
; CHECK: %cmp.eq.one = icmp eq i32 %__THREW__.val, 1 | ||
; CHECK-NEXT: %cmp.eq.zero = icmp eq i32 %__THREW__.val, 0 | ||
; CHECK-NEXT: %or = or i1 %cmp.eq.zero, %cmp.eq.one | ||
; CHECK-NEXT: br i1 %or, label %tail, label %longjmp.rethrow | ||
|
||
; CHECK: tail: | ||
; CHECK-NEXT: %cmp = icmp eq i32 %__THREW__.val, 1 | ||
; CHECK-NEXT: br i1 %cmp, label %lpad, label %try.cont | ||
|
||
; CHECK: longjmp.rethrow: | ||
; CHECK-NEXT: %__threwValue.val = load i32, i32* @__threwValue, align 4 | ||
; CHECK-NEXT: call void @emscripten_longjmp(i32 %__THREW__.val, i32 %__threwValue.val) | ||
; CHECK-NEXT: unreachable | ||
|
||
lpad: ; preds = %entry | ||
%0 = landingpad { i8*, i32 } | ||
catch i8* null | ||
%1 = extractvalue { i8*, i32 } %0, 0 | ||
%2 = extractvalue { i8*, i32 } %0, 1 | ||
%3 = call i8* @__cxa_begin_catch(i8* %1) #5 | ||
call void @__cxa_end_catch() | ||
br label %try.cont | ||
|
||
try.cont: ; preds = %entry, %lpad | ||
ret void | ||
} | ||
|
||
; This function contains a setjmp call and no invoke, so we only handle longjmp | ||
; here. But @foo can also throw an exception, so we check if an exception is | ||
; thrown and if so rethrow it by calling @__resumeException. | ||
define void @rethrow_exception() { | ||
; CHECK-LABEL: @rethrow_exception | ||
entry: | ||
%buf = alloca [1 x %struct.__jmp_buf_tag], align 16 | ||
%arraydecay = getelementptr inbounds [1 x %struct.__jmp_buf_tag], [1 x %struct.__jmp_buf_tag]* %buf, i32 0, i32 0 | ||
%call = call i32 @setjmp(%struct.__jmp_buf_tag* %arraydecay) #0 | ||
%cmp = icmp ne i32 %call, 0 | ||
br i1 %cmp, label %return, label %if.end | ||
|
||
if.end: ; preds = %entry | ||
call void @foo() | ||
br label %return | ||
|
||
; CHECK: if.end: | ||
; CHECK: %cmp.eq.one = icmp eq i32 %__THREW__.val, 1 | ||
; CHECK-NEXT: br i1 %cmp.eq.one, label %eh.rethrow, label %normal | ||
|
||
; CHECK: normal: | ||
; CHECK-NEXT: icmp ne i32 %__THREW__.val, 0 | ||
|
||
; CHECK: eh.rethrow: | ||
; CHECK-NEXT: %exn = call i8* @__cxa_find_matching_catch_2() | ||
; CHECK-NEXT: call void @__resumeException(i8* %exn) | ||
; CHECK-NEXT: unreachable | ||
|
||
return: ; preds = %entry, %if.end | ||
ret void | ||
} | ||
|
||
declare void @foo() | ||
; Function Attrs: returns_twice | ||
declare i32 @setjmp(%struct.__jmp_buf_tag*) | ||
; Function Attrs: noreturn | ||
declare void @longjmp(%struct.__jmp_buf_tag*, i32) | ||
declare i32 @__gxx_personality_v0(...) | ||
declare i8* @__cxa_begin_catch(i8*) | ||
declare void @__cxa_end_catch() | ||
|
||
attributes #0 = { returns_twice } | ||
attributes #1 = { noreturn } |
Oops, something went wrong.