================ @@ -49,9 +49,8 @@ class ClauseProcessor { public: ClauseProcessor(Fortran::lower::AbstractConverter &converter, Fortran::semantics::SemanticsContext &semaCtx, - const Fortran::parser::OmpClauseList &clauses) - : converter(converter), semaCtx(semaCtx), - clauses(makeClauses(clauses, semaCtx)) {} + const List<Clause> &clauses) + : converter(converter), semaCtx(semaCtx), clauses(clauses) {} ---------------- skatrak wrote:
Could we make the `clauses` field an `ArrayRef<Clause>`? In the regular use pattern of this class the clause list of the caller does not go out of scope before the `ClauseProcessor` instance does. Maybe we can document this and avoid creating a local copy. In any case this is a discussion to consider creating another patch for, not something to be addressed here. https://github.com/llvm/llvm-project/pull/87086 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits