================ @@ -94,9 +94,10 @@ StmtResult SemaOpenACC::ActOnEndStmtDirective(OpenACCDirectiveKind K, case OpenACCDirectiveKind::Parallel: case OpenACCDirectiveKind::Serial: case OpenACCDirectiveKind::Kernels: + // TODO OpenACC: Add clauses to the construct here. return OpenACCComputeConstruct::Create( getASTContext(), K, StartLoc, EndLoc, - AssocStmt.isUsable() ? AssocStmt.get() : nullptr); + /*Clauses=*/{}, AssocStmt.isUsable() ? AssocStmt.get() : nullptr); ---------------- erichkeane wrote:
Making the change (assuming it builds:) ) But curious as to why the nullopt preference here? Both create an 'empty' `ArrayRef`, right? https://github.com/llvm/llvm-project/pull/87675 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits