[PATCH] D23885: [ThinLTO] add constArgumentsBitmask to caller summary

Piotr Padlewski via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 25 11:21:04 PDT 2016


Prazek added inline comments.

================
Comment at: include/llvm/IR/ModuleSummaryIndex.h:246
@@ +245,3 @@
+  /// if n_th bit is set, it means that n_th argument is constant.
+  /// Only 16 bits, because functions with more than that are very rare.
+  typedef uint16_t ConstArgumentsBitmaskTy;
----------------
I wasn't sure about the size here, maybe it is even enough to have bitmask here of 8 or less bits.
I assume that this struct will be used to add some extra information that is usefull


https://reviews.llvm.org/D23885





More information about the llvm-commits mailing list