================
@@ -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
