================
@@ -54,6 +54,9 @@ cl::opt<bool>
// setjmp/longjmp handling using wasm EH instrutions
cl::opt<bool> WebAssembly::WasmEnableSjLj(
"wasm-enable-sjlj", cl::desc("WebAssembly setjmp/longjmp handling"));
+cl::opt<bool> WebAssembly::WasmEnableAltSjLj(
+ "experimental-wasm-enable-alt-sjlj",
+ cl::desc("Use experimental alternate ABI for --wasm-enable-sjlj"));
----------------
aheejin wrote:
```suggestion
cl::opt<bool> WebAssembly::WasmEnableExperimentalNewSjLj(
"wasm-enable-experimental-new-sjlj",
cl::desc("Use experimental new ABI for -wasm-enable-sjlj"));
```
I've been trying to make all Wasm related option variable names start with
`Wasm`. Also, if we are to replace the current impl with this, how about just
'new' than 'alt', which sounds like we are planning to keep both?
If we are to change this, we have to fix other places that includes either of
`WasmEnableAltSjLj` and `experimental-wasm-enable-alt-sjlj` as well.
https://github.com/llvm/llvm-project/pull/84137
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
