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

[Mehdi AMINI via llvm-commits]

mehdi_amini added a comment.

Why a bit mask and not an integer with the number of arguments that are constant? 
Do you plan to benefit from knowing which argument is constant?


================
Comment at: lib/Bitcode/Reader/BitcodeReader.cpp:6369
@@ -6364,1 +6368,3 @@
+        CallsiteInfo Info;
+        Info.ConstArgumentsBitmask = Record[++I];
         uint64_t ProfileCount = HasProfile ? Record[++I] : 0;
----------------
You need to handle existing bitcode.

`if (Record.size() >= ...)`


https://reviews.llvm.org/D23885