diff options
author | Heejin Ahn <aheejin@gmail.com> | 2024-02-06 12:22:56 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-02-06 12:22:56 -0800 |
commit | 3db60dfeb4cb07b239e7f3210b20af547a6554fc (patch) | |
tree | d86755829e8c35c591f9772843dda08363c7f4ca /test | |
parent | d490318d64a0de809c19333c4b1c5ddfdfa65d18 (diff) | |
download | binaryen-3db60dfeb4cb07b239e7f3210b20af547a6554fc.tar.gz binaryen-3db60dfeb4cb07b239e7f3210b20af547a6554fc.tar.bz2 binaryen-3db60dfeb4cb07b239e7f3210b20af547a6554fc.zip |
[EH] Add --experimental-new-eh option to wasm-opt (#6270)
This adds `--experimental-new-eh` option to `wasm-opt`. The difference
between this and `--translate-to-new-eh` is, `--translate-to-new-eh`
just runs `TranslateToNewEH` pass, while `--experimental-new-eh`
attaches `TranslateToNewEH` pass at the end of the whole optimization
pipeline. So if no other passes or optimization options (`-On`) are
specified, it is equivalent to `--translate-to-new-eh`. If other
optimization passes are specified, it runs them and at the end run the
translator to ensure the new EH instructions are emitted. The reason we
are doing this this way is that the optimization pipeline as a whole
does not support the new EH instruction yet, but we would like to
provide an option to emit a reasonably OK code with the new EH
instructions.
This also means when the optimization level > 3, it will also run
the StackIR + local2stack optimization after the translation.
Not sure how to test the output of this option, given that there is not
much point in testing the default optimization passes, and it is also
not clear how to print the stack IR if the stack ir generation and
optimization runs as a part of the pipeline and not the explicit command
line options.
This is created in favor of #6267, which added the option to
`optimization-options.h`. It had a problem of running the translator
multiple times when `-On` was given multiple times in the command line,
which I learned was rather a common usage. This adds the option directly
to `wasm-opt.cpp`, which avoids the problem. With this, it is still
possible to create and optimize Stack IR unnecessarily, but that feels a
better alternative.
Diffstat (limited to 'test')
-rw-r--r-- | test/lit/help/wasm-opt.test | 9 | ||||
-rw-r--r-- | test/lit/passes/experimental-new_eh.wast | 28 |
2 files changed, 37 insertions, 0 deletions
diff --git a/test/lit/help/wasm-opt.test b/test/lit/help/wasm-opt.test index 5a207a1b0..88fc2448a 100644 --- a/test/lit/help/wasm-opt.test +++ b/test/lit/help/wasm-opt.test @@ -79,6 +79,15 @@ ;; CHECK-NEXT: --new-wat-parser Use the experimental new WAT ;; CHECK-NEXT: parser ;; CHECK-NEXT: +;; CHECK-NEXT: --experimental-new-eh After running all requested +;; CHECK-NEXT: transformations / optimizations, +;; CHECK-NEXT: translate the instruction to use +;; CHECK-NEXT: the new EH instructions at the +;; CHECK-NEXT: end. Depending on the +;; CHECK-NEXT: optimization level specified, +;; CHECK-NEXT: this may do some more +;; CHECK-NEXT: post-translation optimizations. +;; CHECK-NEXT: ;; CHECK-NEXT: ;; CHECK-NEXT: Optimization passes: ;; CHECK-NEXT: -------------------- diff --git a/test/lit/passes/experimental-new_eh.wast b/test/lit/passes/experimental-new_eh.wast new file mode 100644 index 000000000..ee6624676 --- /dev/null +++ b/test/lit/passes/experimental-new_eh.wast @@ -0,0 +1,28 @@ +;; When given alone, --experimental-new-eh just runs --translate-to-new-eh +;; RUN: wasm-opt %s -all --translate-to-new-eh -S -o %a.wasm +;; RUN: wasm-opt %s -all --experimental-new-eh -S -o %b.wasm +;; RUN: diff %a.wasm %b.wasm + +;; When given with other flags, --experimental-new-eh runs the translator after +;; running other passes. If --optimize-level >=3, --experimenal-new-eh also runs +;; StackIR (+ local2stack) optimization. So running '-O --experimental-new-eh' +;; should be the same as running all these passes separately. +;; RUN: wasm-opt %s -all -O --translate-to-new-eh --optimize-level=3 --generate-stack-ir --optimize-stack-ir -o %a.wasm +;; RUN: wasm-opt %s -all -O --experimental-new-eh -o %b.wasm +;; RUN: diff %a.wasm %b.wasm + +(module + (import "env" "foo" (func $foo)) + (start $test) + (func $test + (try $l + (do + (call $foo) + ) + (catch_all + (call $foo) + (rethrow $l) + ) + ) + ) +) |