[llvm] [WebAssembly] Add assembly support for final EH proposal (PR #107917)
Heejin Ahn via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 10 14:28:18 PDT 2024
================
@@ -147,6 +178,8 @@ let isTerminator = 1, hasSideEffects = 1, isBarrier = 1, hasCtrlDep = 1,
// usage gets low enough.
// Rethrowing an exception: rethrow
+// The new exnref proposal also uses this instruction as an interim pseudo
----------------
aheejin wrote:
If we want to be strict maybe we can make the old rethrow `RETHROW_LEGACY` or something too and add a new pseudo `RETHROW`, but I didn't feel it very necessary given that this is not very confusing. (In `CATCH`/`CATCH_ALL`'s case it was confusing because in CFGStackify or MCInstLower we have to check `WebAssembly::WasmEnableExnref` many times to check whether the current instruction is the legacy `CATCH` or the new pseudo `CATCH`)
https://github.com/llvm/llvm-project/pull/107917
More information about the llvm-commits
mailing list