From 68de7d0535bae14cb31cca4a9d4fa078a855c7e4 Mon Sep 17 00:00:00 2001 From: Herrington Darkholme <2883231+HerringtonDarkholme@users.noreply.github.com> Date: Mon, 20 May 2024 03:10:10 -0700 Subject: [PATCH] [compiler:chore] fix SSA pdf reference link (#29162) ## Summary This PR fixes the deadlink in the SSA comment. Previously the attached link is down now. ## How did you test this change? 1. I can confirm the PDF in the new link is the same as the old now. Reference: https://www.recompiled.dev/blog/ssa/ 2. I can confirm the old link is down, and it is [not just me](https://downforeveryoneorjustme.com/pp.info.uni-karlsruhe.de?proto=https) image --------- Co-authored-by: Jan Kassens --- .../src/SSA/EliminateRedundantPhi.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/compiler/packages/babel-plugin-react-compiler/src/SSA/EliminateRedundantPhi.ts b/compiler/packages/babel-plugin-react-compiler/src/SSA/EliminateRedundantPhi.ts index 2631147889884..50d8022f5fa28 100644 --- a/compiler/packages/babel-plugin-react-compiler/src/SSA/EliminateRedundantPhi.ts +++ b/compiler/packages/babel-plugin-react-compiler/src/SSA/EliminateRedundantPhi.ts @@ -21,7 +21,7 @@ import { * In both these cases, the phi is eliminated and all usages of the phi identifier * are replaced with the other operand (ie in both cases above, all usages of `x2` are replaced with `x1` . * - * The algorithm is inspired by that in https://pp.info.uni-karlsruhe.de/uploads/publikationen/braun13cc.pdf + * The algorithm is inspired by that in https://pp.ipd.kit.edu/uploads/publikationen/braun13cc.pdf * but modified to reduce passes over the CFG. We visit the blocks in reverse postorder. Each time a redundant * phi is encountered we add a mapping (eg x2 -> x1) to a rewrite table. Subsequent instructions, terminals, * and phis rewrite all their identifiers based on this table. The algorithm loops over the CFG repeatedly