================ @@ -506,24 +507,15 @@ void NVPTXAsmPrinter::emitKernelFunctionDirectives(const Function &F, // If the NVVM IR has some of reqntid* specified, then output // the reqntid directive, and set the unspecified ones to 1. // If none of Reqntid* is specified, don't output reqntid directive. - std::optional<unsigned> Reqntidx = getReqNTIDx(F); - std::optional<unsigned> Reqntidy = getReqNTIDy(F); - std::optional<unsigned> Reqntidz = getReqNTIDz(F); + const auto ReqNTID = getReqNTID(F); + if (!ReqNTID.empty()) + O << formatv(".reqntid {0:$[, ]}\n", + make_range(ReqNTID.begin(), ReqNTID.end())); ---------------- Artem-B wrote:
`getReqNTID()` already returns a small vector which should be accepted as a range input. Do we really need an explicit range creation here? https://github.com/llvm/llvm-project/pull/127736 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits