[PATCH] D65238: AMDGPU/GlobalISel: Select llvm.amdgcn.raw.buffer.store

[Matt Arsenault via Phabricator via llvm-commits]

arsenm marked 2 inline comments as done.
arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp:745-755
+static unsigned extractGLC(unsigned CachePolicy) {
+  return CachePolicy & 1;
+}
+
+static unsigned extractSLC(unsigned CachePolicy) {
+  return (CachePolicy >> 1) & 1;
+}
----------------
nhaehnle wrote:
> Could we rearrange these to have a signature analogous to `parseCachePolicy`, to sanity-check that no bits in CachePolicy are set unexpectedly?
I think that's a lot uglier. That kind of check belongs in the verifier


================
Comment at: lib/Target/AMDGPU/AMDGPURegisterBankInfo.cpp:1600-1611
+#if 0
+      // We directly select here to avoid pack/unpack and conversion
+      // instructions out of a possible waterfall loop, and to avoid unnecessary
+      // intermediate operations.
+
+      if (IntrID == Intrinsic::amdgcn_raw_buffer_store) {
+        MachineIRBuilder B(MI);
----------------
nhaehnle wrote:
> What's the intention here?
This is leftover junk from when I thought it was a good idea to select this here


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

https://reviews.llvm.org/D65238