[PATCH] D80562: AMDGPU/GlobalISel: Start rewriting load/store legality rules

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jun 6 07:26:55 PDT 2020


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


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp:188
+
 // Any combination of 32 or 64-bit elements up to 1024 bits, and multiples of
 // v2s16.
----------------
kerbowa wrote:
> Or 128 or 256-bit elements right?
I think loads and some extracts of those should be legal, but they aren't really naturally legal for any operation


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

https://reviews.llvm.org/D80562





More information about the llvm-commits mailing list