[PATCH] D61574: [CommandLine] Allow Options to specify multiple OptionCategory's.

[Don Hinton via Phabricator via llvm-commits]

hintonda marked an inline comment as done.
hintonda added inline comments.


================
Comment at: llvm/lib/Support/CommandLine.cpp:2421
+    for (auto &Cat : I.second->Categories) {
+      if (std::find(CategoriesBegin, CategoriesEnd, Cat) ==
+          CategoriesEnd &&
----------------
MaskRay wrote:
> Use `llvm::find` (or `llvm::count`) and delete the otherwise unused `CategoriesBegin` `CategoriesEnd`.
I'll update them all.  Thanks for point that out.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61574/new/

https://reviews.llvm.org/D61574