[PATCH] D32254: Revert earlier change that flagged ds permute operations as not affecting lgkm counter.

Kannan Narayanan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 19 16:02:35 PDT 2017


kanarayan created this revision.
Herald added subscribers: nhaehnle, wdng.

https://bugs.llvm.org/show_bug.cgi?id=32666 found a case with memory operands. Need to study this better and correct the new pass.


https://reviews.llvm.org/D32254

Files:
  lib/Target/AMDGPU/DSInstructions.td


Index: lib/Target/AMDGPU/DSInstructions.td
===================================================================
--- lib/Target/AMDGPU/DSInstructions.td
+++ lib/Target/AMDGPU/DSInstructions.td
@@ -255,8 +255,6 @@
   [(set i32:$vdst,
    (node (DS1Addr1Offset i32:$addr, i16:$offset), i32:$data0))] > {
 
-  let LGKM_CNT = 0;
-
   let mayLoad = 0;
   let mayStore = 0;
   let isConvergent = 1;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D32254.95845.patch
Type: text/x-patch
Size: 391 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170419/ce19e2fc/attachment.bin>


More information about the llvm-commits mailing list