[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
Fri May 17 19:30:04 PDT 2019
beanz added a comment.
This is pretty cool. Since these are usually allocated as globals, this is saving pages of dirtied memory.
One implementation thought. What if we group the 16-bit fields together and make them uint16_t then use a uint16_t as the bitfield type for the remaining 15 bits. The performance of the 16-bit fields should be better if they are aligned (not that cl::opt perf really matters).
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