[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