[PATCH] D62105: [CommandLine] Remove OptionCategory and SubCommand caches from the Option class.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 3 23:05:55 PDT 2019


MaskRay added inline comments.


================
Comment at: llvm/include/llvm/Support/CommandLine.h:290
+  // Return the set of OptionCategories that this Option belongs to.
+  SmallPtrSet<OptionCategory *, 16> getCategories() const;
+
----------------
16. Does it have to be so large?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62105





More information about the llvm-commits mailing list