Get more tests working with the new text parser (#6284)

The new parser enforces the rule that imports must come before declarations
(except for type declarations). The old parser does not enforce this rule, so
many of our tests did not follow it. Fix them to follow that rule and fix other
invalid syntax. Also add missing finalization of Load expressions in
wasm-builder.h that was causing a test to fail under the new parser and guard
against an error case in wasm-ir-builder.cpp that used to cause a segfault.

1 files changed, 4 insertions, 4 deletions

diff --git a/test/passes/duplicate-function-elimination_optimize-level=2.wast b/test/passes/duplicate-function-elimination_optimize-level=2.wast
index 1097f6715..c66cfba5d 100644
--- a/test/passes/duplicate-function-elimination_optimize-level=2.wast
+++ b/test/passes/duplicate-function-elimination_optimize-level=2.wast
@@ -435,10 +435,10 @@
   )
 )
 (module
-  (memory 0)
-  (type $FUNCSIG$v (func))
   (import "env" "i" (func $i))
   (import "env" "j" (func $j))
+  (memory 0)
+  (type $FUNCSIG$v (func))
   (func $erase (type $FUNCSIG$v)
     (call $i)
   )
@@ -447,10 +447,10 @@
   )
 )
 (module
-  (memory 0)
-  (type $FUNCSIG$v (func))
   (import "env" "i" (func $i))
   (import "env" "j" (func $j))
+  (memory 0)
+  (type $FUNCSIG$v (func))
   (func $keep2 (type $FUNCSIG$v)
     (call $i)
   )