[PATCH] D41651: AMDGPU: Add 32-bit constant address space

[Tony Tye via Phabricator via llvm-commits]

t-tye added inline comments.


================
Comment at: lib/Target/AMDGPU/SIISelLowering.cpp:153
+  // TODO: This breaks unaligned loads, so don't do it on AmdHsa.
+  if (!Subtarget->isAmdHsaOS())
+    setOperationAction(ISD::LOAD, MVT::i32, Custom);
----------------
Would it be better to add a query for whether unaligned accesses were supported? Currently it would only be for AmdHsaOS but would allow it to be easily changed in the future.


https://reviews.llvm.org/D41651