arsenm accepted this revision.
arsenm added a comment.
This revision is now accepted and ready to land.
LGTM. It may not be the end step and may be annoying to make further changes
from this point, but on its own this isn't worse than before
================
Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:998
// TODO: Should images get their own address space?
+ Info.fallbackAddressSpace = AMDGPUAS::BUFFER_RESOURCE;
----------------
Should this todo be dropped?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D145441/new/
https://reviews.llvm.org/D145441
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
