[PATCH] D109337: [GlobalOpt][FIX] Do not embed initializers into AS!=0 globals
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 7 16:32:43 PDT 2021
arsenm added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUTargetTransformInfo.h:187
+ bool canHaveNonUndefGlobalInitializerInAddressSpace(unsigned AS) const {
+ return AS != AMDGPUAS::LOCAL_ADDRESS;
+ }
----------------
Should also disallow REGION_ADDRESS, and PRIVATE_ADDRESS (we don't even allow private globals)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109337/new/
https://reviews.llvm.org/D109337
More information about the llvm-commits
mailing list