From adfae63f208b1e8e98c1923a2ac8e26686173253 Mon Sep 17 00:00:00 2001 From: Sathya Gunasekaran Date: Mon, 7 Aug 2023 13:41:59 -0400 Subject: [PATCH] [hir] Remove unnecessary rewrite of instr.lvalue The for loop over eachInstructionLValue already rewrites instr.lvalue: ``` for (const place of eachInstructionLValue(instr)) { rewritePlace(place, rewrites); } ``` --- .../babel-plugin-react-forget/src/SSA/EliminateRedundantPhi.ts | 3 --- 1 file changed, 3 deletions(-) diff --git a/compiler/forget/packages/babel-plugin-react-forget/src/SSA/EliminateRedundantPhi.ts b/compiler/forget/packages/babel-plugin-react-forget/src/SSA/EliminateRedundantPhi.ts index 07559fd8e6..9da0bba5d9 100644 --- a/compiler/forget/packages/babel-plugin-react-forget/src/SSA/EliminateRedundantPhi.ts +++ b/compiler/forget/packages/babel-plugin-react-forget/src/SSA/EliminateRedundantPhi.ts @@ -112,9 +112,6 @@ export function eliminateRedundantPhi( rewritePlace(place, rewrites); } } - - rewritePlace(instr.lvalue, rewrites); - // visit function expressions on first iteration of each block if ( isFirstIteration &&