Skip to content

Commit 2cdc54e

Browse files
committed
handleLongjmpableCallsForWasmSjLj: simplify a bit
1 parent 93bde20 commit 2cdc54e

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

llvm/lib/Target/WebAssembly/WebAssemblyLowerEmscriptenEHSjLj.cpp

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -355,7 +355,6 @@ class WebAssemblyLowerEmscriptenEHSjLj final : public ModulePass {
355355
SmallVectorImpl<PHINode *> &SetjmpRetPHIs);
356356
void
357357
handleLongjmpableCallsForWasmSjLj(Function &F, InstVector &SetjmpTableInsts,
358-
InstVector &SetjmpTableSizeInsts,
359358
SmallVectorImpl<PHINode *> &SetjmpRetPHIs);
360359
Function *getFindMatchingCatch(Module &M, unsigned NumClauses);
361360

@@ -1417,8 +1416,7 @@ bool WebAssemblyLowerEmscriptenEHSjLj::runSjLjOnFunction(Function &F) {
14171416
handleLongjmpableCallsForEmscriptenSjLj(
14181417
F, SetjmpTableInsts, SetjmpTableSizeInsts, SetjmpRetPHIs);
14191418
else // EnableWasmSjLj
1420-
handleLongjmpableCallsForWasmSjLj(F, SetjmpTableInsts, SetjmpTableSizeInsts,
1421-
SetjmpRetPHIs);
1419+
handleLongjmpableCallsForWasmSjLj(F, SetjmpTableInsts, SetjmpRetPHIs);
14221420

14231421
// Erase everything we no longer need in this function
14241422
for (Instruction *I : ToErase)
@@ -1712,7 +1710,7 @@ static BasicBlock *getCleanupRetUnwindDest(const CleanupPadInst *CPI) {
17121710
// BBs. Refer to 4) of "Wasm setjmp/longjmp handling" section in the comments at
17131711
// top of the file for details.
17141712
void WebAssemblyLowerEmscriptenEHSjLj::handleLongjmpableCallsForWasmSjLj(
1715-
Function &F, InstVector &SetjmpTableInsts, InstVector &SetjmpTableSizeInsts,
1713+
Function &F, InstVector &SetjmpTableInsts,
17161714
SmallVectorImpl<PHINode *> &SetjmpRetPHIs) {
17171715
Module &M = *F.getParent();
17181716
LLVMContext &C = F.getContext();
@@ -1739,7 +1737,6 @@ void WebAssemblyLowerEmscriptenEHSjLj::handleLongjmpableCallsForWasmSjLj(
17391737
// Arbitrarily use the ones defined in the beginning of the function.
17401738
// SSAUpdater will later update them to the correct values.
17411739
Instruction *SetjmpTable = *SetjmpTableInsts.begin();
1742-
Instruction *SetjmpTableSize = *SetjmpTableSizeInsts.begin();
17431740

17441741
// Add setjmp.dispatch BB right after the entry block. Because we have
17451742
// initialized setjmpTable/setjmpTableSize in the entry block and split the

0 commit comments

Comments
 (0)