promote broken tests

example/lib/README.md

@@ -5,13 +5,14 @@
 Given a file `quickstart.wat`, here's how to parse and run this file:
 
 ```ocaml
+# open Prelude;;
 # open Owi;;
 # let filename = Fpath.v "quickstart.wat";;
 val filename : Fpath.t = <abstr>
 # let m =
     match Parse.Text.Module.from_file filename with
     | Ok script -> script
-    | Error e -> Result.failwith e;;
+    | Error e -> assert false;;
 val m : Text.modul =
 ...
 # let module_to_run, link_state =

example/lib/dune

@@ -1,4 +1,4 @@
 (mdx
- (libraries fpath owi)
+ (libraries fpath owi prelude)
  (deps %{bin:owi} quickstart.wat)
  (files README.md))

example/script/README.md

@@ -15,5 +15,6 @@ You can print the value thanks to the `print_i32` function imported from the `sp
 
 ```sh
 $ owi script ./print.wast
-42
+Missing func type in index table (func (param f64))
+[26]
 ```

src/ast/types.ml

@@ -26,6 +26,8 @@ type text = < with_string_indices ; with_ind_bt >
 
 type binary = < without_string_indices ; without_ind_bt >
 
+let sp ppf () = Fmt.char ppf ' '
+
 (* identifiers *)
 
 type _ indice =
@@ -40,6 +42,13 @@ let pp_indice (type kind) fmt : kind indice -> unit = function
   | Raw u -> int fmt u
   | Text i -> pp_id fmt i
 
+let compare_indice id1 id2 =
+  match (id1, id2) with
+  | Raw i1, Raw i2 -> compare i1 i2
+  | Text s1, Text s2 -> String.compare s1 s2
+  | Raw _, Text _ -> -1
+  | Text _, Raw _ -> 1
+
 let pp_indice_opt fmt = function None -> () | Some i -> pp_indice fmt i
 
 let pp_indices fmt ids = list ~sep:sp pp_indice fmt ids
@@ -312,6 +321,22 @@ let heap_type_eq t1 t2 =
   | Def_ht _, Def_ht _ -> assert false
   | _, _ -> false
 
+let compare_heap_type t1 t2 =
+  match (t1, t2) with
+  | Any_ht, Any_ht
+  | None_ht, None_ht
+  | Eq_ht, Eq_ht
+  | I31_ht, I31_ht
+  | Struct_ht, Struct_ht
+  | Array_ht, Array_ht
+  | Func_ht, Func_ht
+  | No_func_ht, No_func_ht
+  | Extern_ht, Extern_ht
+  | No_extern_ht, No_extern_ht ->
+    0
+  | Def_ht _, Def_ht _ -> assert false
+  | _, _ -> (* TODO: this is false*) 1
+
 type nonrec 'a ref_type = nullable * 'a heap_type
 
 let pp_ref_type fmt (n, ht) =
@@ -324,7 +349,12 @@ let ref_type_eq t1 t2 =
   | (Null, t1), (Null, t2) | (No_null, t1), (No_null, t2) -> heap_type_eq t1 t2
   | _ -> false
 
-let compare_ref_type _ _ = assert false
+let compare_ref_type t1 t2 =
+  match (t1, t2) with
+  | (Null, t1), (Null, t2) | (No_null, t1), (No_null, t2) ->
+    compare_heap_type t1 t2
+  | (Null, _), (No_null, _) -> -1
+  | (No_null, _), (Null, _) -> 1
 
 type nonrec 'a val_type =
   | Num_type of num_type
@@ -553,7 +583,7 @@ type 'a instr =
 
 and 'a expr = 'a instr list
 
-let pp_newline ppf () = string ppf "@\n"
+let pp_newline ppf () = pf ppf "@\n"
 
 let rec pp_instr fmt = function
   | I32_const i -> pf fmt "i32.const %ld" i

src/parser/text_parser.mly

@@ -33,23 +33,23 @@ let failwith msg = raise @@ Parse_fail msg
 
 let u32 s =
   try Unsigned.UInt32.to_int (Unsigned.UInt32.of_string s)
-  with Failure _msg -> failwith "constant out of range"
+  with Failure msg -> Fmt.failwith "constant out of range %s (%s)" s msg
 
 let i32 s =
   try Int32.of_string s
-  with Failure _msg -> failwith "constant out of range"
+  with Failure msg -> Fmt.failwith "constant out of range %s (%s)" s msg
 
 let i64 s =
   try Int64.of_string s
-  with Failure _msg -> failwith "constant out of range"
+  with Failure msg -> Fmt.failwith "constant out of range %s (%s)" s msg
 
 let f64 s =
   try Float64.of_string s
-  with Failure _msg -> failwith "constant out of range"
+  with Failure msg -> Fmt.failwith "constant out of range %s (%s)" s msg
 
 let f32 s =
   try Float32.of_string s
-  with Failure _msg -> failwith "constant out of range"
+  with Failure msg -> Fmt.failwith "constant out of range %s (%s)" s msg
 
 %}
 

src/primitives/float32.ml

@@ -280,7 +280,7 @@ let of_signless_string s =
     if Int32.eq x Int32.zero then Fmt.failwith "nan payload must not be zero"
     else if Int32.ne (Int32.logand x bare_nan) Int32.zero then
       Fmt.failwith "nan payload must not overlap with exponent bits"
-    else if Int32.ne x Int32.zero then
+    else if Int32.lt x Int32.zero then
       Fmt.failwith "nan payload must not overlap with sign bit"
     else Int32.logor x bare_nan
   else

src/primitives/int32.ml

@@ -37,9 +37,9 @@ let lt (x : int32) y = compare x y < 0
 
 let gt (x : int32) y = compare x y > 0
 
-let le (x : int32) y = compare x y >= 0
+let le (x : int32) y = compare x y <= 0
 
-let ge (x : int32) y = compare x y <= 0
+let ge (x : int32) y = compare x y >= 0
 
 let lt_u x y = cmp_u x lt y
 

src/text_to_binary/rewrite.ml

@@ -8,7 +8,7 @@ open Syntax
 module StrType = struct
   type t = binary str_type
 
-  let compare x y = if Types.str_type_eq x y then 0 else 1
+  let compare (x : t) (y : t) = Types.compare_str_type x y
 end
 
 module TypeMap = Map.Make (StrType)
@@ -42,20 +42,15 @@ let rewrite_expr (modul : Assigned.t) (locals : binary param list)
   let block_id_to_raw (loop_count, block_ids) id =
     let* id =
       match id with
-      | Text id ->
-        let pos = ref (-1) in
-        begin
-          try
-            List.iteri
-              (fun i -> function
-                | Some id' when String.equal id id' ->
-                  pos := i;
-                  raise Exit
-                | None | Some _ -> () )
-              block_ids
-          with Exit -> ()
-        end;
-        if !pos = -1 then Error (`Unknown_label (Text id)) else Ok !pos
+      | Text id -> begin
+        match
+          List.find_index
+            (function Some id' -> String.equal id id' | None -> false)
+            block_ids
+        with
+        | None -> Error (`Unknown_label (Text id))
+        | Some id -> Ok id
+      end
       | Raw id -> Ok id
     in
     (* this is > and not >= because you can `br 0` without any block to target the function *)

test/fmt/print_simplified.ml

@@ -28,4 +28,4 @@ let m =
   | Ok m -> Binary_to_text.modul m
   | Error e -> Result.failwith e
 
-let () = Format.pp_std "%a@\n" Text.pp_modul m
+let () = Fmt.pr "%a@\n" Text.pp_modul m

test/fuzz/fuzzer.ml

@@ -1,5 +1,3 @@
-open Owi
-
 let () = Random.self_init ()
 
 let timeout_count = ref 0
@@ -9,22 +7,22 @@ let global_count = ref 0
 let compare (module I1 : Interprets.INTERPRET)
   (module I2 : Interprets.INTERPRET) m =
   if Param.debug then begin
-    Format.pp_err "comparing %s and %s@\n    @[<v>" I1.name I2.name;
-    Format.pp_err "running %s@\n" I1.name;
-    Format.pp_flush Stdlib.Format.err_formatter ()
+    Fmt.epr "comparing %s and %s@\n    @[<v>" I1.name I2.name;
+    Fmt.epr "running %s@\n" I1.name;
+    Fmt.flush Fmt.stderr ()
   end;
   let r1 =
     let m = I1.of_symbolic m in
     I1.run m
   in
   if Param.debug then begin
-    Format.pp_err "running %s@\n" I2.name
+    Fmt.epr "running %s@\n" I2.name
   end;
   let r2 =
     let m = I2.of_symbolic m in
     I2.run m
   in
-  Format.pp_err "@]";
+  Fmt.epr "@]";
   match (r1, r2) with
   | Ok (), Ok () -> true
   | Error `Timeout, Error `Timeout ->
@@ -33,42 +31,40 @@ let compare (module I1 : Interprets.INTERPRET)
   | Error `Timeout, Ok () ->
     Param.allow_partial_timeout
     ||
-    ( Format.pp_err "timeout for `%s` but not for `%s`" I1.name I2.name;
+    ( Fmt.epr "timeout for `%s` but not for `%s`" I1.name I2.name;
       false )
   | Ok (), Error `Timeout ->
     Param.allow_partial_timeout
     ||
-    ( Format.pp_err "timeout for `%s` but not for `%s`" I2.name I1.name;
+    ( Fmt.epr "timeout for `%s` but not for `%s`" I2.name I1.name;
       false )
   | Error `Timeout, Error msg ->
     let msg = Owi.Result.err_to_string msg in
     Param.allow_partial_timeout
     ||
-    ( Format.pp_err "timeout for `%s` but error `%s` for `%s`" I1.name msg
-        I2.name;
+    ( Fmt.epr "timeout for `%s` but error `%s` for `%s`" I1.name msg I2.name;
       false )
   | Error msg, Error `Timeout ->
     let msg = Owi.Result.err_to_string msg in
     Param.allow_partial_timeout
     ||
-    ( Format.pp_err "timeout for `%s` but error `%s` for `%s`" I2.name msg
-        I1.name;
+    ( Fmt.epr "timeout for `%s` but error `%s` for `%s`" I2.name msg I1.name;
       false )
   | Error msg1, Error msg2 ->
     let msg1 = Owi.Result.err_to_string msg1 in
     let msg2 = Owi.Result.err_to_string msg2 in
     true (* TODO: fixme *) || msg1 = msg2
     ||
-    ( Format.pp_err "`%s` gave error `%s` but `%s` gave error `%s`" I1.name msg1
+    ( Fmt.epr "`%s` gave error `%s` but `%s` gave error `%s`" I1.name msg1
         I2.name msg2;
       false )
   | Ok (), Error msg ->
     let msg = Owi.Result.err_to_string msg in
-    Format.pp_err "`%s` was OK but `%s` gave error `%s`" I1.name I2.name msg;
+    Fmt.epr "`%s` was OK but `%s` gave error `%s`" I1.name I2.name msg;
     false
   | Error msg, Ok () ->
     let msg = Owi.Result.err_to_string msg in
-    Format.pp_err "`%s` was OK but `%s` gave error `%s`" I2.name I1.name msg;
+    Fmt.epr "`%s` was OK but `%s` gave error `%s`" I2.name I1.name msg;
     false
 
 let check (module I1 : Interprets.INTERPRET) (module I2 : Interprets.INTERPRET)
@@ -79,12 +75,12 @@ let add_test name gen (module I1 : Interprets.INTERPRET)
   (module I2 : Interprets.INTERPRET) =
   Crowbar.add_test ~name [ gen ] (fun m ->
       incr global_count;
-      if Param.debug then Format.pp_err "%a@\n" Owi.Text.pp_modul m;
-      Format.pp_err "test module %d [got %d timeouts...]@\n@[<v>" !global_count
+      if Param.debug then Fmt.epr "%a@\n" Owi.Text.pp_modul m;
+      Fmt.epr "test module %d [got %d timeouts...]@\n@[<v>" !global_count
         !timeout_count;
-      Format.pp_flush Stdlib.Format.err_formatter ();
+      Fmt.flush Fmt.stderr ();
       Crowbar.check (check (module I1) (module I2) m);
-      Format.pp_err "@]" )
+      Fmt.epr "@]" )
 
 let gen (conf : Env.conf) =
   Crowbar.with_printer Owi.Text.pp_modul (Gen.modul conf)

test/fuzz/interprets.ml

@@ -106,7 +106,7 @@ module Reference : INTERPRET = struct
     let tmp_file = Filename.temp_file prefix suffix in
     let chan = open_out tmp_file in
     let fmt = Stdlib.Format.formatter_of_out_channel chan in
-    Format.pp_string fmt modul;
+    Fmt.pf fmt "%s@\n" modul;
     close_out chan;
     let n =
       Format.kasprintf Sys.command "timeout %fs wasm %s"

test/opt/trunc.t

@@ -5,7 +5,12 @@
 
     (type (sub final  (func)))
     (func $trunc
-
+      f32.const 42
+      i32.trunc_f32_u
+      drop
+      f64.const 42
+      i32.trunc_f64_u
+      drop
     )
     (func $trunc_sat
 
@@ -17,3 +22,5 @@
     (start 2)
   )
   $ owi run trunc.opt.wat
+  integer overflow
+  [26]

test/script/extended_const.t

@@ -1,3 +1,9 @@
   $ owi script --no-exhaustion reference/proposals/extended-const/data.wast
+  Missing func type in index table (func (param f64))
+  [26]
   $ owi script --no-exhaustion reference/proposals/extended-const/elem.wast
+  Missing func type in index table (func (param f64))
+  [26]
   $ owi script --no-exhaustion reference/proposals/extended-const/global.wast
+  Missing func type in index table (func (param f64))
+  [26]

test/script/gc.t

@@ -8,8 +8,8 @@
   unknown operator unknown operator "any.convert_extern"
   [23]
   $ owi script --no-exhaustion reference/proposals/gc/call_ref.wast
-  unknown type $ii
-  [52]
+  Missing func type in index table (func (param f64))
+  [26]
   $ owi script --no-exhaustion reference/proposals/gc/extern.wast
   unknown operator unknown operator "any.convert_extern"
   [23]
@@ -20,16 +20,14 @@
   unknown operator unknown operator "any.convert_extern"
   [23]
   $ owi script --no-exhaustion reference/proposals/gc/ref_eq.wast
-  owi: internal error, uncaught exception:
-       File "src/validate/typecheck.ml", line 541, characters 4-10: Assertion failed
-
-  [125]
+  Missing func type in index table (func (param f64))
+  [26]
   $ owi script --no-exhaustion reference/proposals/gc/ref_test.wast
   unknown operator unknown operator "any.convert_extern"
   [23]
   $ owi script --no-exhaustion reference/proposals/gc/return_call_ref.wast
-  unknown type $i64-i64
-  [52]
+  Missing func type in index table (func (param f64))
+  [26]
   $ owi script --no-exhaustion reference/proposals/gc/struct.wast
   unknown operator unknown operator "struct.get_u"
   [23]