[PATCH] D112635: [AMDGPU] Disable d16 loads/stores to high halves on gfx90a

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 27 10:34:50 PDT 2021


rampitec added a comment.

Needs test.



================
Comment at: llvm/lib/Target/AMDGPU/GCNSubtarget.h:594
   bool d16PreservesUnusedBits() const {
-    return hasD16LoadStore() && !TargetID.isSramEccOnOrAny();
+    // gff90a's d16 loads don't preserve unused bits
+    return hasD16LoadStore() && !TargetID.isSramEccOnOrAny() &&
----------------
Typo gff90a.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112635



More information about the llvm-commits mailing list