[PATCH] D11708: AMDGPU/SI: Fix missing implicit def/use of exec

[Matt Arsenault]

arsenm created this revision.
arsenm added a reviewer: tstellarAMD.
arsenm added a subscriber: llvm-commits.

Not sure why this wasn't caught by the machineinstr verifier.

http://reviews.llvm.org/D11708

Files:
  lib/Target/AMDGPU/SILowerControlFlow.cpp

Index: lib/Target/AMDGPU/SILowerControlFlow.cpp
===================================================================
--- lib/Target/AMDGPU/SILowerControlFlow.cpp
+++ lib/Target/AMDGPU/SILowerControlFlow.cpp
@@ -317,8 +317,10 @@
     }
   } else {
     BuildMI(MBB, &MI, DL, TII->get(AMDGPU::V_CMPX_LE_F32_e32), AMDGPU::VCC)
-           .addImm(0)
-           .addOperand(Op);
+      .addImm(0)
+      .addOperand(Op)
+      .addReg(AMDGPU::EXEC, RegState::Implicit | RegState::Define)
+      .addReg(AMDGPU::EXEC, RegState::Implicit);
   }
 
   MI.eraseFromParent();


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D11708.31201.patch
Type: text/x-patch
Size: 566 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150802/997a6d24/attachment.bin>