[PATCH] D102211: [AMDGPU] Do not clause NSA instructions

[Jay Foad via Phabricator via llvm-commits]

foad added a comment.

As I understand it, this is only needed for GFX10.1, not for GFX10.3. So I think it probably deserves its own target feature which would be included in GFX10_1_Bugs.



================
Comment at: llvm/lib/Target/AMDGPU/SIInsertHardClauses.cpp:80
+    // Do not clause NSA instructions on GFX10.
+    if (AMDGPU::isGFX10(*ST) && ST->hasFeature(AMDGPU::FeatureNSAEncoding)) {
+      const AMDGPU::MIMGInfo *Info = AMDGPU::getMIMGInfo(MI.getOpcode());
----------------
Maybe sink this down to just before we return HARDCLAUSE_VMEM?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D102211/new/

https://reviews.llvm.org/D102211