minor tweaks for pass pipeline

Author: MaxGraey

src/module.ts

@@ -2860,11 +2860,12 @@ export class Module {
         }
 
         passes.push("simplify-globals-optimizing");
-        passes.push("reorder-globals");
-        passes.push("reorder-functions");
         passes.push("remove-unused-brs");
         passes.push("optimize-instructions");
+        passes.push("vacuum");
       }
+      passes.push("reorder-globals");
+      passes.push("reorder-functions");
       // clean up
       if (optimizeLevel >= 2 || shrinkLevel >= 1) {
         passes.push("duplicate-function-elimination");

tests/compiler/class-overloading.release.wat

@@ -3162,15 +3162,6 @@
     call $~lib/builtins/abort
     unreachable
    end
-   i32.const 0
-   if
-    i32.const 0
-    i32.const 1552
-    i32.const 209
-    i32.const 1
-    call $~lib/builtins/abort
-    unreachable
-   end
    global.get $~lib/memory/__stack_pointer
    i32.const 4
    i32.add

tests/compiler/class-static-function.release.wat

@@ -1,7 +1,7 @@
 (module
  (type $0 (func (result i32)))
- (type $1 (func (param i32 i32 i32 i32)))
- (type $2 (func))
+ (type $1 (func))
+ (type $2 (func (param i32 i32 i32 i32)))
  (import "env" "abort" (func $~lib/builtins/abort (param i32 i32 i32 i32)))
  (global $~lib/memory/__stack_pointer (mut i32) (i32.const 33916))
  (memory $0 1)
@@ -39,15 +39,6 @@
   i32.load
   call_indirect (type $0)
   drop
-  i32.const 0
-  if
-   i32.const 0
-   i32.const 1088
-   i32.const 11
-   i32.const 1
-   call $~lib/builtins/abort
-   unreachable
-  end
   global.get $~lib/memory/__stack_pointer
   i32.const 4
   i32.add

tests/compiler/inlining.release.wat

@@ -1740,15 +1740,6 @@
    i32.load
    call_indirect (type $5)
    drop
-   i32.const 0
-   if
-    i32.const 0
-    i32.const 1056
-    i32.const 69
-    i32.const 3
-    call $~lib/builtins/abort
-    unreachable
-   end
    global.get $~lib/memory/__stack_pointer
    i32.const 123
    i32.store