[PATCH] D23885: [ThinLTO] add constArgumentsBitmask to caller summary
Mehdi AMINI via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 25 11:58:03 PDT 2016
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
More information about the llvm-commits
mailing list