[PATCH] D44046: [AMDGPU] Fixed incorrect break from loop

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added inline comments.


================
Comment at: lib/Target/AMDGPU/SILowerControlFlow.cpp:354
+  if (MI.getOperand(1).isReg())
+    if (MachineInstr *Def = MRI->getUniqueVRegDef(MI.getOperand(1).getReg()))
+      if (Def->getParent() == MI.getParent())
----------------
arsenm wrote:
> Can you add a testcase where this will fail if there is no defining instruction (e.g. the source value was a function argument)
Can this be a partial def?


Repository:
  rL LLVM

https://reviews.llvm.org/D44046