[PATCH] D122091: [amdgpu] Elide module lds allocation in kernels with no callees

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 4 11:30:05 PDT 2022


arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUMachineFunction.h:60
+  // This kernel calls no functions that require the module lds struct
+  bool CanElideModuleLDS = false;
+
----------------
arsenm wrote:
> Probably not much value in caching this in MFI
I guess might as well leave it since we have the others (although I'm not a huge fan of these cached fields we don't serialize)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122091



More information about the llvm-commits mailing list