Skip to content

Commit

Permalink
Don't store closures in summary (reduction effects)
Browse files Browse the repository at this point in the history
We already have indirection (constant -> effect name, effect name ->
function) so we only need to stop using summary.ref for the second map.
  • Loading branch information
SkySkimmer committed Apr 8, 2019
1 parent 70a00b7 commit e9738ee
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pretyping/reductionops.ml
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ type effect_name = string
let constant_effect_table = Summary.ref ~name:"reduction-side-effect" Cmap.empty

(* Table bindings function key to effective functions *)
let effect_table = Summary.ref ~name:"reduction-function-effect" String.Map.empty
let effect_table = ref String.Map.empty

(** a test to know whether a constant is actually the effect function *)
let reduction_effect_hook env sigma con c =
Expand Down

0 comments on commit e9738ee

Please sign in to comment.