[AMDGPU] Ask subtarget if waitcnt instruction is needed before barrier instruction
authorKonstantin Zhuravlyov <kzhuravl_dev@outlook.com>
Fri, 30 Sep 2016 16:50:36 +0000 (16:50 +0000)
committerKonstantin Zhuravlyov <kzhuravl_dev@outlook.com>
Fri, 30 Sep 2016 16:50:36 +0000 (16:50 +0000)
Differential Revision: https://reviews.llvm.org/D24985

llvm-svn: 282875

llvm/lib/Target/AMDGPU/AMDGPUSubtarget.h
llvm/lib/Target/AMDGPU/SIInsertWaits.cpp

index c278cc5..3298a4b 100644 (file)
@@ -540,6 +540,12 @@ public:
 
   /// Return the maximum number of waves per SIMD for kernels using \p VGPRs VGPRs
   unsigned getOccupancyWithNumVGPRs(unsigned VGPRs) const;
+
+  /// \returns True if waitcnt instruction is needed before barrier instruction,
+  /// false otherwise.
+  bool needWaitcntBeforeBarrier() const {
+    return true;
+  }
 };
 
 } // End namespace llvm
index d24588d..b9551be 100644 (file)
@@ -590,8 +590,9 @@ bool SIInsertWaits::runOnMachineFunction(MachineFunction &MF) {
       // S_SENDMSG implicitly waits for all outstanding LGKM transfers to finish,
       // but we also want to wait for any other outstanding transfers before
       // signalling other hardware blocks
-      if (I->getOpcode() == AMDGPU::S_BARRIER ||
-          I->getOpcode() == AMDGPU::S_SENDMSG)
+      if ((I->getOpcode() == AMDGPU::S_BARRIER &&
+               ST->needWaitcntBeforeBarrier()) ||
+           I->getOpcode() == AMDGPU::S_SENDMSG)
         Required = LastIssued;
       else
         Required = handleOperands(*I);