[PATCH] D102401: [AMDGPU] Allocate LDS globals in sorted order of their size and alignment.

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 13 07:59:51 PDT 2021


foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUMachineFunction.cpp:31
+    // We might want to use a b96 or b128 load/store
+    Alignment = Alignment >= Align(16) ? Alignment : Align(16);
+  } else if (GVSize > 4) {
----------------
Isn't there a suitable Alignment::max?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102401



More information about the llvm-commits mailing list