================ @@ -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