[PATCH] D118109: [clang] Replace `std::vector` use in Syntax

2022-01-25 Thread Duncan P. N. Exon Smith via Phabricator via cfe-commits
dexonsmith accepted this revision. dexonsmith added a comment. LGTM too. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118109/new/ https://reviews.llvm.org/D118109 ___ cfe-commits mailing list cfe-commit

[PATCH] D118109: [clang] Replace `std::vector` use in Syntax

2022-01-25 Thread David Blaikie via Phabricator via cfe-commits
dblaikie accepted this revision. dblaikie added a comment. This revision is now accepted and ready to land. Looks good, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118109/new/ https://reviews.llvm.org/D118109

[PATCH] D118109: [clang] Replace `std::vector` use in Syntax

2022-01-25 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 created this revision. jansvoboda11 added a reviewer: dexonsmith. jansvoboda11 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. LLVM Programmer’s Manual strongly discourages the use of `std::vector` and suggests `llvm::BitVecto