More CR ocaml 5 comments (#2108)

Author: mshinwell

ocaml/bytecomp/emitcode.ml

@@ -421,7 +421,8 @@ let to_file outchan unit_name objfile ~required_globals code =
         (Filename.dirname (Location.absolute_path objfile))
         !debug_dirs;
       let p = pos_out outchan in
-      (* CR mshinwell: Compression not supported in the OCaml 4 runtime
+      (* CR ocaml 5 compressed-marshal mshinwell:
+         Compression not supported in the OCaml 4 runtime
       Marshal.(to_channel outchan !events [Compression]);
       Marshal.(to_channel outchan (String.Set.elements !debug_dirs)
                           [Compression]);

ocaml/file_formats/cmi_format.ml

@@ -167,7 +167,8 @@ let output_cmi filename oc cmi =
   output_int64 oc len;
   Out_channel.seek oc val_pos;
   (* BACKPORT BEGIN *)
-  (* mshinwell: upstream uses [Compression] here *)
+  (* CR ocaml 5 compressed-marshal mshinwell:
+     upstream uses [Compression] here *)
   output_value oc ((cmi.cmi_name, sign) : header);
   (* BACKPORT END *)
   flush oc;

ocaml/file_formats/cmt_format.ml

@@ -110,7 +110,8 @@ let input_cmt ic = (input_value ic : cmt_infos)
 let output_cmt oc cmt =
   output_string oc Config.cmt_magic_number;
   (* BACKPORT BEGIN *)
-  (* mshinwell: upstream uses [Compression] here *)
+  (* CR ocaml 5 compressed-marshal mshinwell:
+     upstream uses [Compression] here *)
   Marshal.(to_channel oc (cmt : cmt_infos) [])
   (* BACKPORT END *)
 

ocaml/testsuite/tests/backtrace/backtrace_effects_nested.ml

@@ -3,7 +3,7 @@
 
 flags = "-g"
 * skip
-  reason = "OCaml 5 only"
+  reason = "CR ocaml 5 effects: re-enable this test"
 ** bytecode
 ** no-flambda
 *** native

ocaml/testsuite/tests/callback/callback_effects_gc.ml

@@ -1,7 +1,7 @@
 (* TEST
    ocamlrunparam += ",s=512"
    * skip
-   reason = "OCaml 5 only"
+   reason = "CR ocaml 5 effects: re-enable this test"
    ** native
 *)
 

ocaml/testsuite/tests/callback/nested_fiber.ml

@@ -2,7 +2,7 @@
    include unix
    modules = "nested_fiber_.c"
    * skip
-     reason = "OCaml 5 only"
+     reason = "CR ocaml 5 effects: re-enable this test"
    ** libunix
    *** bytecode
    *** native

ocaml/testsuite/tests/callback/stack_overflow.ml

@@ -2,7 +2,7 @@
    include unix
    modules = "stack_overflow_.c"
    * skip
-     reason = "OCaml 5 only"
+     reason = "CR ocaml 5 effects: re-enable this test"
    ** libunix
    *** bytecode
    *** native

ocaml/testsuite/tests/callback/test7.ml

@@ -3,7 +3,7 @@
    include unix
    modules = "test7_.c"
    * skip
-     reason = "OCaml 5 only"
+     reason = "CR ocaml 5 effects: re-enable this test"
    ** libunix
    *** bytecode
    *** native

ocaml/testsuite/tests/effects/marshal.ml

@@ -1,6 +1,6 @@
 (* TEST
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 effects: re-enable this test"
  *)
 
 open Effect

ocaml/testsuite/tests/effects/unhandled_unlinked.ml

@@ -3,7 +3,7 @@
 
      exit_status= "2"
      * skip
-     reason = "OCaml 5 only"
+     reason = "CR ocaml 5 effects: re-enable this test"
 *)
 
 open Effect

ocaml/testsuite/tests/frame-pointers/effects.ml

@@ -3,7 +3,7 @@
 
 
 * skip
-reason - "OCaml 5 only"
+reason = "CR ocaml 5 effects: re-enable this test"
 ** frame_pointers
 *** native
 readonly_files = "fp_backtrace.c"

ocaml/testsuite/tests/frame-pointers/reperform.ml

@@ -3,7 +3,7 @@
 
 
 * skip
-reason - "OCaml 5 only"
+reason = "CR ocaml 5 effects: re-enable this test"
 ** frame_pointers
 *** native
 

ocaml/testsuite/tests/frame-pointers/stack_realloc.ml

@@ -3,7 +3,7 @@
 
 
 * skip
-reason - "OCaml 5 only"
+reason = "CR ocaml 5 effects: re-enable this test"
 ** frame_pointers
 *** native
 

ocaml/testsuite/tests/frame-pointers/stack_realloc2.ml

@@ -3,7 +3,7 @@
 
 
 * skip
-reason - "OCaml 5 only"
+reason = "CR ocaml 5 effects: re-enable this test"
 ** frame_pointers
 *** native
 

ocaml/testsuite/tests/gc-roots/globroots_parallel.ml

@@ -3,7 +3,7 @@
    modules = "globrootsprim.c globroots.ml"
 
    * skip
-     reason = "OCaml 5 only"
+     reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 open Globroots

ocaml/testsuite/tests/gc-roots/globroots_parallel_spawn_burn.ml

@@ -3,7 +3,7 @@
    modules = "globrootsprim.c globroots.ml"
 
    * skip
-     reason = "OCaml 5 only"
+     reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 open Globroots

ocaml/testsuite/tests/lazy/lazy2.ml

@@ -1,7 +1,7 @@
 (* TEST
    ocamlopt_flags += " -O3 "
    * skip
-     reason = "OCaml 5 only"
+     reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 open Domain

ocaml/testsuite/tests/lazy/lazy3.ml

@@ -1,7 +1,7 @@
 (* TEST
    ocamlopt_flags += " -O3 "
    * skip
-     reason = "OCaml 5 only"
+     reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 let f count =

ocaml/testsuite/tests/lazy/lazy5.ml

@@ -1,7 +1,7 @@
 (* TEST
    ocamlopt_flags += " -O3 "
    * skip
-     reason = "OCaml 5 only"
+     reason = "CR ocaml 5 domains: re-enable this test"
 *)
 let rec safe_force l =
   try Lazy.force l with

ocaml/testsuite/tests/lazy/lazy6.ml

@@ -1,7 +1,7 @@
 (* TEST
    ocamlopt_flags += " -O3 "
    * skip
-     reason = "OCaml 5 only"
+     reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 let flag1 = Atomic.make false

ocaml/testsuite/tests/lazy/lazy7.ml

@@ -1,7 +1,7 @@
 (* TEST
    ocamlopt_flags += " -O3 "
    * skip
-     reason = "OCaml 5 only"
+     reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 let num_domains = 4

ocaml/testsuite/tests/lazy/lazy8.ml

@@ -1,7 +1,7 @@
 (* TEST
    ocamlopt_flags += " -O3 "
    * skip
-     reason = "OCaml 5 only"
+     reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 exception E

ocaml/testsuite/tests/lf_skiplist/test_parallel.ml

@@ -2,7 +2,7 @@
    modules = "stubs.c"
 
    * skip
-     reason = "OCaml 5 only"
+     reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 external init_skiplist : unit -> unit = "init_skiplist"

ocaml/testsuite/tests/lib-channels/refcounting.ml

@@ -1,6 +1,6 @@
 (* TEST
    * skip
-   reason = "OCaml 5 only"
+   reason = "CR ocaml 5 domains: re-enable this test"
    ** expect
 *)
 

ocaml/testsuite/tests/lib-dynlink-domains/main.ml

@@ -1,6 +1,6 @@
 (* TEST
  * skip
-   reason = "OCaml 5 only"
+   reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 (* BACKPORT

ocaml/testsuite/tests/lib-format/domains.ml

@@ -1,6 +1,6 @@
 (* TEST
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 (** Test that domains stdout and stderr are flushed at domain exit *)

ocaml/testsuite/tests/lib-format/mc_pr586_par.ml

@@ -1,6 +1,6 @@
 (* TEST
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 let () =

ocaml/testsuite/tests/lib-format/mc_pr586_par2.ml

@@ -1,6 +1,6 @@
 (* TEST
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 let () =

ocaml/testsuite/tests/lib-marshal/intext_par.ml

@@ -1,6 +1,6 @@
 (* TEST
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 domains: re-enable this test"
    modules = "intextaux_par.c"
 *)
 

ocaml/testsuite/tests/lib-random/parallel.ml

@@ -1,7 +1,7 @@
 (* TEST
    include unix
    * skip
-   reason = "OCaml 5 only"
+   reason = "CR ocaml 5 domains: re-enable this test"
    ** libunix
    *** bytecode
    *** native

ocaml/testsuite/tests/lib-str/parallel.ml

@@ -1,6 +1,6 @@
 (* TEST
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 domains: re-enable this test"
 ** hasstr
 include str
 *** bytecode

ocaml/testsuite/tests/lib-sync/prodcons.ml

@@ -1,6 +1,6 @@
 (* TEST
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 (* Classic producer-consumer *)

ocaml/testsuite/tests/lib-systhreads/multicore_lifecycle.ml

@@ -1,6 +1,6 @@
 (* TEST
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 domains: re-enable this test"
 ** hassysthreads
 include systhreads
 *** bytecode

ocaml/testsuite/tests/lib-unix/common/multicore_fork_domain_alone.ml

@@ -1,7 +1,7 @@
 (* TEST
 include unix
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 domains: re-enable this test"
 ** hasunix
 *** not-windows
 **** bytecode

ocaml/testsuite/tests/lib-unix/common/multicore_fork_domain_alone2.ml

@@ -1,7 +1,7 @@
 (* TEST
 include unix
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 domains: re-enable this test"
 ** hasunix
 *** not-windows
 **** bytecode

ocaml/testsuite/tests/memory-model/publish.ml

@@ -1,7 +1,7 @@
 (* TEST
   modules="opt.ml barrier.ml hist.ml shared.ml run.ml outcome.ml"
   * skip
-  reason = "OCaml 5 only"
+  reason = "CR ocaml 5 domains: re-enable this test"
   ** not-bsd
   *** not-windows
   **** bytecode

ocaml/testsuite/tests/parallel/atomics.ml

@@ -1,6 +1,6 @@
 (* TEST
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 type u = U of unit

ocaml/testsuite/tests/parallel/backup_thread.ml

@@ -1,6 +1,6 @@
 (* TEST
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 domains: re-enable this test"
 ** hasunix
 include unix
 *** bytecode

ocaml/testsuite/tests/parallel/backup_thread_pipe.ml

@@ -1,6 +1,6 @@
 (* TEST
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 domains: re-enable this test"
 ** hasunix
 include unix
 *** bytecode

ocaml/testsuite/tests/parallel/constpromote.ml

@@ -1,6 +1,6 @@
 (* TEST
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 (* when run with the bytecode debug runtime, this test

ocaml/testsuite/tests/parallel/deadcont.ml

@@ -1,6 +1,6 @@
 (* TEST
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 effects: re-enable this test"
 *)
 
 (*

ocaml/testsuite/tests/parallel/domain_dls.ml

@@ -1,6 +1,6 @@
 (* TEST
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 let check_dls () =

ocaml/testsuite/tests/parallel/domain_dls2.ml

@@ -1,6 +1,6 @@
 (* TEST
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 let _ =

ocaml/testsuite/tests/parallel/domain_id.ml

@@ -1,6 +1,6 @@
 (* TEST
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 open Domain

ocaml/testsuite/tests/parallel/domain_parallel_spawn_burn.ml

@@ -1,6 +1,6 @@
 (* TEST
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 open Domain

ocaml/testsuite/tests/parallel/domain_parallel_spawn_burn_gc_set.ml

@@ -1,6 +1,6 @@
 (* TEST
 * skip
-reason = "OCaml 5 only"
+reason = "CR ocaml 5 domains: re-enable this test"
 *)
 
 open Domain