Skip to content

Commit 518d1f7

Browse files
Swap order of Apron and ARG hashcons checks
Check Apron first (fail) before ARG (warn) since fail is more severe Co-authored-by: michael-schwarz <13812333+michael-schwarz@users.noreply.github.com>
1 parent 0b677c6 commit 518d1f7

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/maingoblint.ml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -137,10 +137,10 @@ let check_arguments () =
137137
if get_bool "exp.basic-blocks" && not (get_bool "justcil") && List.mem "assert" @@ get_string_list "trans.activated" then (set_bool "exp.basic-blocks" false; warn "The option exp.basic-blocks implicitly disabled by activating the \"assert\" transformation.");
138138
if (not @@ get_bool "witness.invariant.all-locals") && (not @@ get_bool "cil.addNestedScopeAttr") then (set_bool "cil.addNestedScopeAttr" true; warn "Disabling witness.invariant.all-locals implicitly enables cil.addNestedScopeAttr.");
139139
if not (get_bool "ana.opt.hashcons") then (
140-
if get_bool "exp.arg.enabled" then
141-
warn "Disabling ana.opt.hashcons has no effect because hashconsing is implicitly enabled by exp.arg.enabled";
142140
if MCPRegistry.any_activated_uses_apron () then
143141
fail "Disabling ana.opt.hashcons is not supported when using Apron domains";
142+
if get_bool "exp.arg.enabled" then
143+
warn "Disabling ana.opt.hashcons has no effect because hashconsing is implicitly enabled by exp.arg.enabled";
144144
);
145145
if List.mem "remove_dead_code" @@ get_string_list "trans.activated" then (
146146
(* 'assert' transform happens before 'remove_dead_code' transform *)

0 commit comments

Comments
 (0)