diff options
author | Son Ho | 2022-01-20 19:22:38 +0100 |
---|---|---|
committer | Son Ho | 2022-01-20 19:22:38 +0100 |
commit | 6eeb6256b598bffa2a8b636748e3c73ecb473da0 (patch) | |
tree | 2d7930873b855bba7f6d7069f0ebabc696e0ea99 | |
parent | 7fcd02817f4201e81ed35bf3356095997966c8d5 (diff) |
Make various style modifications
-rw-r--r-- | src/Cps.ml | 14 | ||||
-rw-r--r-- | src/InterpreterStatements.ml | 18 |
2 files changed, 25 insertions, 7 deletions
@@ -142,3 +142,17 @@ let () = let cc = comp_transmit return3 do_nothing in let cc = cc consume3 in cc () + +let comp_check_value (f : ('v -> 'ctx -> 'a) -> 'ctx -> 'b) + (g : 'v -> 'ctx -> unit) : ('v -> 'ctx -> 'a) -> 'ctx -> 'b = + fun cf -> + f (fun v ctx -> + g v ctx; + cf v ctx) + +let comp_check_ctx (f : ('ctx -> 'a) -> 'ctx -> 'b) (g : 'ctx -> unit) : + ('ctx -> 'a) -> 'ctx -> 'b = + fun cf -> + f (fun ctx -> + g ctx; + cf ctx) diff --git a/src/InterpreterStatements.ml b/src/InterpreterStatements.ml index 17cac599..f6223fa8 100644 --- a/src/InterpreterStatements.ml +++ b/src/InterpreterStatements.ml @@ -284,9 +284,8 @@ let ctx_pop_frame (config : C.config) (cf : V.typed_value -> m_fun) : m_fun = let cc = eval_operand config (E.Move (mk_place_from_var_id ret_vid)) in (* Sanity check *) let cc = - comp cc (fun cf ret_value -> - assert (not (bottom_in_value ctx.ended_regions ret_value)); - cf ret_value) + comp_check_value cc (fun ret_value ctx -> + assert (not (bottom_in_value ctx.ended_regions ret_value))) in (* Drop the outer *loans* we find in the local variables *) @@ -307,12 +306,11 @@ let ctx_pop_frame (config : C.config) (cf : V.typed_value -> m_fun) : m_fun = let cc = comp cc cf_drop_loans_in_locals in (* Debug *) let cc = - comp cc (fun cf ret_value ctx -> + comp_check_value cc (fun ret_value ctx -> log#ldebug (lazy ("ctx_pop_frame: after dropping outer loans in local variables:\n" - ^ eval_ctx_to_string ctx)); - cf ret_value ctx) + ^ eval_ctx_to_string ctx))) in (* Pop the frame - we remove the `Frame` delimiter, and reintroduce all @@ -556,7 +554,13 @@ let eval_non_local_function_call_concrete (config : C.config) (* let ctx, args_vl = eval_operands config ctx args in *) let cf_eval_ops = eval_operands config args in - (* Evaluate the call *) + (* Evaluate the call + * + * Style note: at some point we used [comp_transmit] to + * transmit the result of [eval_operands] above down to [push_vars] + * below, without having to introduce an intermediary function call, + * but it made it less clear where the computed values came from, + * so we reversed the modifications. *) let cf_eval_call cf (args_vl : V.typed_value list) : m_fun = (* Push the stack frame: we initialize the frame with the return variable, and one variable per input argument *) |