[PATCH] D62091: [CommandLine] Reduce size of Option class by moving more members into bit field

Chris Bieneman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 18 11:04:08 PDT 2019


beanz added inline comments.


================
Comment at: llvm/include/llvm/Support/CommandLine.h:288
       Categories;                    // The Categories this option belongs to
   SmallPtrSet<SubCommand *, 4> Subs; // The subcommands this option belongs to.
 
----------------
Side note: I wonder how many members actually get put into this.  The common-case for most options is surely 0.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62091





More information about the llvm-commits mailing list