Remove mov where arg and dst are the same

voltrevo · Jul 5, 2023 · 5b76267 · 5b76267
1 parent 7010296
commit 5b76267
Showing 1 changed file with 15 additions and 2 deletions.
diff --git a/valuescript_compiler/src/optimization/reduce_instructions.rs b/valuescript_compiler/src/optimization/reduce_instructions.rs
@@ -29,8 +29,21 @@ fn reduce(instr: Instruction) -> Option<Instruction> {
     | Throw(..) | SetCatch(..) | UnsetCatch | ConstSubCall(..) | RequireMutableThis
     | ThisSubCall(..) | Next(..) | Yield(..) | YieldStar(..) => Some(instr),
 
-    Mov(_, dst)
-    | OpPlus(_, _, dst)
+    Mov(arg, dst) => 'b: {
+      if dst.is_ignore() {
+        break 'b None;
+      }
+
+      if let Value::Register(arg) = arg {
+        if arg.name == dst.name {
+          break 'b None;
+        }
+      }
+
+      Some(instr)
+    }
+
+    OpPlus(_, _, dst)
     | OpMinus(_, _, dst)
     | OpMul(_, _, dst)
     | OpDiv(_, _, dst)