@samitouri / QOS-React-1 / commits / 68de7d0535

[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) <img width="580" alt="image" src="https://github.com/facebook/react/assets/2883231/eeca5eda-6f61-4ac4-a113-1b04370533d9"> --------- Co-authored-by: Jan Kassens <jkassens@meta.com>

Herrington Darkholme committed May 20, 2024 at 03:10 UTC 68de7d0535bae14cb31cca4a9d4fa078a855c7e4
1 file changed +1 -1
compiler/packages/babel-plugin-react-compiler/src/SSA/EliminateRedundantPhi.ts
+1 -1
@@ -21,7 +21,7 @@ import {
21 * In both these cases, the phi is eliminated and all usages of the phi identifier
22 * are replaced with the other operand (ie in both cases above, all usages of `x2` are replaced with `x1` .
23 *
24 - * The algorithm is inspired by that in https://pp.info.uni-karlsruhe.de/uploads/publikationen/braun13cc.pdf
24 + * The algorithm is inspired by that in https://pp.ipd.kit.edu/uploads/publikationen/braun13cc.pdf
25 * but modified to reduce passes over the CFG. We visit the blocks in reverse postorder. Each time a redundant
26 * phi is encountered we add a mapping (eg x2 -> x1) to a rewrite table. Subsequent instructions, terminals,
27 * and phis rewrite all their identifiers based on this table. The algorithm loops over the CFG repeatedly