[PATCH] D36233: [ThinLTO] Add FunctionAttrs to ThinLTO index

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added inline comments.


================
Comment at: lib/Bitcode/Reader/BitcodeReader.cpp:5118
+        RefListStartIndex = 5;
+      }
+
----------------
mehdi_amini wrote:
> Note: for these we don't usually use a version, we just look at the size of the Record.
I don't think that will work since we have a variable number of reference and call edge records following the flags. We have the number of references (NumRefs) but not the number of calls. That is deduced from the length of the record.


https://reviews.llvm.org/D36233