[PATCH] D112731: [AMDGPU] Really preserve LiveVariables in SILowerControlFlow

[Jay Foad via Phabricator via llvm-commits]

foad added inline comments.


================
Comment at: llvm/lib/CodeGen/LiveVariables.cpp:732
+    MachineBasicBlock &UseBB = *MF->getBlockNumbered(UseBBNum);
+    // Only add kills in DefBB if all uses are in DefBB.
+    if (&UseBB == &DefBB && UseBlocks.count() != 1)
----------------
ruiling wrote:
> why do we need this?
This is the cheapest way I could think of to test whether Reg is live-to-end of DefBB. Note that DefBB never appears in AliveBlocks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112731