Facebook/react
1
0
Fork 0
mirror of https://github.com/facebook/react.git synced 2025-11-01 09:12:30 +00:00
Code Issues Packages Projects Releases Wiki Activity

[be] Remove dead code in ConstantPropagation

Browse Source
This commit is contained in:
Joe Savona
2023-10-10 13:15:39 -07:00
parent 5a8c7594c5
commit ba23b990ac
1 changed files with 0 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files
compiler/packages/babel-plugin-react-forget/src/Optimization/ConstantPropagation.ts
-20
View File
@@ -97,26 +97,6 @@ function applyConstantPropagation(
fn: HIRFunction,
constants: Constants
): boolean {
// Track the set of identifiers which are used as dependencies for function expressions
// in order to avoid propagating these constants. This is necessary because the function
// itself will still reference the original value. If the dependency is propagated but the
// function still refers to the original value, this can create a situation where DCE
// will think the original expression is unused. Until we are able to propagate constants
// into function expression bodies, we disable propagation of function deps.
const functionDependencies = new Set<IdentifierId>();
for (const [, block] of fn.body.blocks) {
for (const instr of block.instructions) {
if (
instr.value.kind === "FunctionExpression" ||
instr.value.kind === "ObjectMethod"
) {
for (const operand of instr.value.loweredFunc.dependencies) {
functionDependencies.add(operand.identifier.id);
}
}
}
}
let hasChanges = false;
for (const [, block] of fn.body.blocks) {
// Initialize phi values if all operands have the same known constant value.