[PATCH] D133840: AMDGPU: Add a pass to rewrite certain undef in PHI

[Sameer Sahasrabuddhe via Phabricator via llvm-commits]

sameerds added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPURewriteUndefForPHI.cpp:116
+            HasBackedgeUndef = true;
+        } else if (Incoming != &PHI) {
+          if (!UniqueDefinedIncoming) {
----------------



================
Comment at: llvm/lib/Target/AMDGPU/AMDGPURewriteUndefForPHI.cpp:122
+            // Update DominateBB if necessary.
+            if (DT->dominates(IncomingBB, DominateBB))
+              DominateBB = IncomingBB;
----------------
If this is not true, then does DominateBB dominate IncomingBB?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D133840/new/

https://reviews.llvm.org/D133840