[clang] [llvm] [mlir] [NVPTX] Add support for Shared Cluster Memory address space. (PR #135444)
Alex MacLean via cfe-commits
cfe-commits at lists.llvm.org
Sat Apr 12 10:53:33 PDT 2025
https://github.com/AlexMaclean commented:
Backend changes look reasonable so far. One concern I have with this change is that until now we've assumed specific address-spaces are non-overlapping. You've addressed some of the places where this assumption is encoded but I think there are others you have not. For example, addrspacecasts between shared::cta and shared::cluster seem like they should be valid and should be expanded via a pair of cvta instructions through generic space, I think this would produce an error right now. I also wonder if InferAS or other places have made this assumption as well.
https://github.com/llvm/llvm-project/pull/135444
More information about the cfe-commits
mailing list