[PATCH] D149651: [UnreachableBlockElim] Don't remove LCSSA phi nodes

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 2 08:13:47 PDT 2023


foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SILowerControlFlow.cpp:430-431
           .add(MI.getOperand(0));
+  if (LV)
+    LV->replaceKillInstruction(MI.getOperand(0).getReg(), MI, *AndN2);
 
----------------
I can submit this separately, but I guess I would have to come up with a MIR test case. I am not sure why the change to UnreachableBlockElim exposed this problem.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149651



More information about the llvm-commits mailing list