[PATCH] D12452: AMDGPU/SI: Add support for llvm.r600.local.size.* instrics when targeting HSA

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 28 15:44:41 PDT 2015


arsenm added inline comments.

================
Comment at: lib/Target/AMDGPU/SIISelLowering.cpp:1041
@@ +1040,3 @@
+    Offset = SI::DispatchPacketOffset::LOCAL_SIZE_X + (Dim * 4);
+    MemVT = MVT::i16;
+  } else {
----------------
Why is this an i16? We really don't want to have to do an argument extload, although from the tests it looks like that doesn't happen.


http://reviews.llvm.org/D12452





More information about the llvm-commits mailing list