================ @@ -958,13 +931,14 @@ Parser::OpenACCClauseParseResult Parser::ParseOpenACCClauseParams( case OpenACCClauseKind::Link: case OpenACCClauseKind::NoCreate: case OpenACCClauseKind::Present: - case OpenACCClauseKind::Private: case OpenACCClauseKind::UseDevice: - if (ParseOpenACCClauseVarList(ClauseKind)) { - Parens.skipToEnd(); - return OpenACCCanContinue(); - } + ParseOpenACCVarList(); + break; + case OpenACCClauseKind::Private: { + llvm::SmallVector<Expr *> Vars = ParseOpenACCVarList(); + ParsedClause.setVarListDetails(std::move(Vars)); ---------------- erichkeane wrote:
Right, `Vars` was immediately going out of scope, so it has no need for its elements anymore, so passing as a 'copy' is not necessary. BUT I've changed it to be an inlined call, so THIS std::move is gone. https://github.com/llvm/llvm-project/pull/90521 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits