[StackIR] Run StackIR during binary writing and not as a pass (#6568)

Previously we had passes --generate-stack-ir, --optimize-stack-ir, --print-stack-ir
that could be run like any other passes. After generating StackIR it was stashed on
the function and invalidated if we modified BinaryenIR. If it wasn't invalidated then
it was used during binary writing. This PR switches things so that we optionally
generate, optimize, and print StackIR only during binary writing. It also removes
all traces of StackIR from wasm.h - after this, StackIR is a feature of binary writing
(and printing) logic only.

This is almost NFC, but there are some minor noticeable differences:

1.  We no longer print has StackIR in the text format when we see it is there. It
    will not be there during normal printing, as it is only present during binary writing.
    (but --print-stack-ir still works as before; as mentioned above it runs during writing).
2.  --generate/optimize/print-stack-ir change from being passes to being flags that
    control that behavior instead. As passes, their order on the commandline mattered,
    while now it does not, and they only "globally" affect things during writing.
3.  The C API changes slightly, as there is no need to pass it an option "optimize" to
    the StackIR APIs. Whether we optimize is handled by --optimize-stack-ir which is
    set like other optimization flags on the PassOptions object, so we don't need the
    old option to those C APIs.

The main benefit here is simplifying the code, so we don't need to think about
StackIR in more places than just binary writing. That may also allow future
improvements to our usage of StackIR.

1 files changed, 7 insertions, 7 deletions

diff --git a/test/lit/passes/O_fast-math.wast b/test/lit/passes/O_fast-math.wast
index 722f44a05..a964edc99 100644
--- a/test/lit/passes/O_fast-math.wast
+++ b/test/lit/passes/O_fast-math.wast
@@ -37,7 +37,7 @@
 
  ;; CHECK:      (export "abs_sub_zero2" (func $abs_sub_zero2))
 
- ;; CHECK:      (func $div (; has Stack IR ;) (result f32)
+ ;; CHECK:      (func $div (result f32)
  ;; CHECK-NEXT:  (f32.const -nan:0x23017a)
  ;; CHECK-NEXT: )
  (func $div (export "div") (result f32)
@@ -46,7 +46,7 @@
    (f32.const 1)
   )
  )
- ;; CHECK:      (func $mul1 (; has Stack IR ;) (result f32)
+ ;; CHECK:      (func $mul1 (result f32)
  ;; CHECK-NEXT:  (f32.const -nan:0x34546d)
  ;; CHECK-NEXT: )
  (func $mul1 (export "mul1") (result f32)
@@ -55,7 +55,7 @@
    (f32.const 1)
   )
  )
- ;; CHECK:      (func $mul2 (; has Stack IR ;) (result f32)
+ ;; CHECK:      (func $mul2 (result f32)
  ;; CHECK-NEXT:  (f32.const nan:0x400000)
  ;; CHECK-NEXT: )
  (func $mul2 (export "mul2") (result f32)
@@ -100,7 +100,7 @@
    (f32.const -0)
   )
  )
- ;; CHECK:      (func $mul_neg_one1 (; has Stack IR ;) (param $0 f32) (result f32)
+ ;; CHECK:      (func $mul_neg_one1 (param $0 f32) (result f32)
  ;; CHECK-NEXT:  (f32.neg
  ;; CHECK-NEXT:   (local.get $0)
  ;; CHECK-NEXT:  )
@@ -111,7 +111,7 @@
    (f32.const -1)
   )
  )
- ;; CHECK:      (func $mul_neg_one2 (; has Stack IR ;) (param $0 f64) (result f64)
+ ;; CHECK:      (func $mul_neg_one2 (param $0 f64) (result f64)
  ;; CHECK-NEXT:  (f64.neg
  ;; CHECK-NEXT:   (local.get $0)
  ;; CHECK-NEXT:  )
@@ -122,7 +122,7 @@
    (f64.const -1)
   )
  )
- ;; CHECK:      (func $abs_sub_zero1 (; has Stack IR ;) (param $0 f32) (result f32)
+ ;; CHECK:      (func $abs_sub_zero1 (param $0 f32) (result f32)
  ;; CHECK-NEXT:  (f32.abs
  ;; CHECK-NEXT:   (local.get $0)
  ;; CHECK-NEXT:  )
@@ -136,7 +136,7 @@
    )
   )
  )
- ;; CHECK:      (func $abs_sub_zero2 (; has Stack IR ;) (param $0 f64) (result f64)
+ ;; CHECK:      (func $abs_sub_zero2 (param $0 f64) (result f64)
  ;; CHECK-NEXT:  (f64.abs
  ;; CHECK-NEXT:   (local.get $0)
  ;; CHECK-NEXT:  )