Valentin Clement =?utf-8?b?KOODkOODrOODsw=?Message-ID: In-Reply-To: <llvm.org/llvm/llvm-project/pull/77...@github.com>
================ @@ -124,21 +125,41 @@ class OpenACCRoutineInfo { void set_isGang(bool value = true) { isGang_ = value; } unsigned gangDim() const { return gangDim_; } void set_gangDim(unsigned value) { gangDim_ = value; } - bool isNohost() const { return isNohost_; } - void set_isNohost(bool value = true) { isNohost_ = value; } const std::string *bindName() const { return bindName_ ? &*bindName_ : nullptr; } void set_bindName(std::string &&name) { bindName_ = std::move(name); } + void set_dType(Fortran::common::OpenACCDeviceType dType) { dType_ = dType; } + Fortran::common::OpenACCDeviceType dType() const { return dType_; } private: bool isSeq_{false}; bool isVector_{false}; bool isWorker_{false}; bool isGang_{false}; unsigned gangDim_{0}; - bool isNohost_{false}; std::optional<std::string> bindName_; + Fortran::common::OpenACCDeviceType dType_{ ---------------- razvanlupusoru wrote: nit: Can you name it deviceType_ instead of dType_? Although the name "dtype" is specifically noted as acceptable abbreviation in OpenACC spec, I personally find it confusing. https://github.com/llvm/llvm-project/pull/77804 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits