[PATCH] LowerBitSets: Use byte arrays instead of bit sets to represent in-memory bit sets.
JF Bastien
jfb at chromium.org
Mon Mar 2 16:11:01 PST 2015
================
Comment at: include/llvm/Transforms/IPO/LowerBitSets.h:179
@@ +178,3 @@
+ /// The number of bytes allocated so far for each of the bits.
+ uint64_t BitAllocs[8];
+
----------------
I'd still rather have to 8 pulled out as its own symbolic constant, instead of being used here and below as a magic number.
http://reviews.llvm.org/D7954
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list