[PATCH] D72931: AMDGPU: Add a16 feature to gfx10

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 27 06:58:30 PST 2020


arsenm marked an inline comment as done.
arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:5628
                                           : AMDGPU::MIMGEncGfx10Default,
-                                   NumVDataDwords, NumVAddrDwords);
+                                   NumVDataDwords, VAddrs.size());
   } else {
----------------
sebastian-ne wrote:
> Are you sure that’s right?
> The test CodeGen/AMDGPU/llvm.amdgcn.image.nsa.ll fails for me with this change.
No, it's probably wrong


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

https://reviews.llvm.org/D72931





More information about the llvm-commits mailing list