================ @@ -280,8 +282,10 @@ NVPTXTargetMachine::getPredicatedAddrSpace(const Value *V) const { case Intrinsic::nvvm_isspacep_local: return std::make_pair(II->getArgOperand(0), llvm::ADDRESS_SPACE_LOCAL); case Intrinsic::nvvm_isspacep_shared: - case Intrinsic::nvvm_isspacep_shared_cluster: return std::make_pair(II->getArgOperand(0), llvm::ADDRESS_SPACE_SHARED); + case Intrinsic::nvvm_isspacep_shared_cluster: ---------------- modiking wrote:
Technically this is conservative since `nvvm_isspacep_shared_cluster == true` implies that the pointer is also under `ADDRESS_SPACE_SHARED`. However the current interface only allows returning one address so that'll need to get updated. I'm planning on improving InferAddrSpaces to also detect from mapa/mapa_shared_cluster so I'll update this part as well during that. https://github.com/llvm/llvm-project/pull/135444 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits