fix(ssa refactor): Do not remove enable_side_effects instructions in …

…die pass (#1673) Do not remove enable_side_effects instructions in die pass
noir-lang · Jun 14, 2023 · cbee4c0 · cbee4c0
1 parent 377ac5c
commit cbee4c0
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/crates/noirc_evaluator/src/ssa_refactor/opt/die.rs b/crates/noirc_evaluator/src/ssa_refactor/opt/die.rs
@@ -93,7 +93,10 @@ impl Context {
         let instruction = &function.dfg[instruction_id];
 
         // These instruction types cannot be removed
-        if matches!(instruction, Constrain(_) | Call { .. } | Store { .. }) {
+        if matches!(
+            instruction,
+            Constrain(_) | Call { .. } | Store { .. } | EnableSideEffects { .. }
+        ) {
             return false;
         }