[PATCH] D61574: [CommandLine] Allow Options to specify multiple OptionCategory's.
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun May 5 21:31:23 PDT 2019
MaskRay added inline comments.
================
Comment at: llvm/lib/Support/CommandLine.cpp:2421
+ for (auto &Cat : I.second->Categories) {
+ if (std::find(CategoriesBegin, CategoriesEnd, Cat) ==
+ CategoriesEnd &&
----------------
Use `llvm::find` (or `llvm::count`) and delete the otherwise unused `CategoriesBegin` `CategoriesEnd`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61574/new/
https://reviews.llvm.org/D61574
More information about the llvm-commits
mailing list