[PATCH] D39036: AMDGPU: Fix default range in non-kernel functions

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 23 09:22:31 PDT 2017


arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/AMDGPUSubtarget.cpp:199
+  case CallingConv::SPIR_KERNEL:
+    return std::make_pair(getWavefrontSize() * 2, getWavefrontSize() * 4);
+  case CallingConv::AMDGPU_VS:
----------------
t-tye wrote:
> Should there be a target property that returns the maximum size supported? 4 * wavefront size may not match every target.
It's the same for every target now, so there's no real point in adding one.


https://reviews.llvm.org/D39036





More information about the llvm-commits mailing list