[PATCH] D16725: AMDGPU/SI: Implement a work-around for smrd corrupting vccz bit

Tom Stellard via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 5 11:31:11 PST 2016


tstellarAMD marked an inline comment as done.

================
Comment at: lib/Target/AMDGPU/SIInsertWaits.cpp:523
@@ +522,3 @@
+          VCCZCorrupt = true;
+        } else if (!hasOutstandingLGKM() && I->modifiesRegister(AMDGPU::VCC, TRI)) {
+          // FIXME: We only care about SMRD instructions here, not LDS or GDS. 
----------------
arsenm wrote:
> Do we need to worry about modifications to vcc_lo/vcc_hi?
modifiesRegister() considers sub-regs too.


http://reviews.llvm.org/D16725





More information about the llvm-commits mailing list