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

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 3 08:32:30 PDT 2017


mehdi_amini added inline comments.


================
Comment at: lib/Bitcode/Reader/BitcodeReader.cpp:5118
+        RefListStartIndex = 5;
+      }
+
----------------
Note: for these we don't usually use a version, we just look at the size of the Record.


================
Comment at: lib/Bitcode/Reader/BitcodeReader.cpp:5215
+        RefListStartIndex = 6;
+      }
+
----------------
Same here, the Record size seems enough for a discriminant.


https://reviews.llvm.org/D36233





More information about the llvm-commits mailing list