[PATCH] Introduce bitset metadata format and bitset lowering pass.

Kostya Serebryany kcc at google.com
Wed Feb 4 18:57:47 PST 2015


================
Comment at: lib/Transforms/IPO/LowerBitSets.cpp:44
@@ +43,3 @@
+
+}
+
----------------
}   // namespace

================
Comment at: lib/Transforms/IPO/LowerBitSets.cpp:54
@@ +53,3 @@
+
+bool LowerBitSets::buildBitSets(Module &M, const DataLayout *DL) {
+  Function *BitSetTestFunc =
----------------
I wonder if you could split this 300 LOC function into several smaller ones.

http://reviews.llvm.org/D7288

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list