Skip to content

Commit ac252ef

Browse files
committed
Fix some debugging leftovers
1 parent bd0bfc3 commit ac252ef

File tree

2 files changed

+13
-10
lines changed

2 files changed

+13
-10
lines changed

middle_end/flambda/tests/meet_test.ml

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -201,11 +201,11 @@ let () =
201201
(Linkage_name.create "meet_test")
202202
in
203203
Compilation_unit.set_current comp_unit;
204-
(* Format.eprintf "MEET CHAINS WITH TWO VARS\n\n%!";
205-
* test_meet_chains_two_vars ();
206-
* Format.eprintf "\nMEET CHAINS WITH THREE VARS\n\n%!";
207-
* test_meet_chains_three_vars ();
208-
* Format.eprintf "@.MEET VARIANT@.@.";
209-
* meet_variants_don't_lose_aliases (); *)
210-
Format.eprintf "@.Test bug ?@.";
204+
Format.eprintf "MEET CHAINS WITH TWO VARS@\n@.";
205+
test_meet_chains_two_vars ();
206+
Format.eprintf "@.MEET CHAINS WITH THREE VARS@\n@.";
207+
test_meet_chains_three_vars ();
208+
Format.eprintf "@.MEET VARIANT@\n@.";
209+
meet_variants_don't_lose_aliases ();
210+
Format.eprintf "@.MEET TWO BLOCKS@\n@.";
211211
test_meet_two_blocks ()

middle_end/flambda/types/env/typing_env.rec.ml

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -896,14 +896,17 @@ let invariant_for_alias (t:t) name ty =
896896
Name.print name Type_grammar.print ty
897897
end
898898

899+
(* This is too costly to check, but it can be useful for debugging problems with
900+
canonical aliases.
899901
let invariant_for_aliases (t:t) =
900902
Name.Map.iter (fun name (ty, _, _) ->
901903
invariant_for_alias t name ty
902-
) (names_to_types t)
904+
) (names_to_types t)
905+
*)
903906

904907
let invariant_for_new_equation (t:t) name ty =
905-
invariant_for_alias t name ty;
906908
if !Clflags.flambda_invariant_checks then begin
909+
invariant_for_alias t name ty;
907910
(* CR mshinwell: This should check that precision is not decreasing. *)
908911
let defined_names =
909912
Name_occurrences.create_names
@@ -978,7 +981,7 @@ let rec add_equation0 (t:t) name ty =
978981
One_level.create (current_scope t) level ~just_after_level
979982
in
980983
let res = with_current_level t ~current_level in
981-
invariant_for_aliases res;
984+
(* invariant_for_aliases res; *)
982985
res
983986

984987
and add_equation t name ty =

0 commit comments

Comments
 (0)