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

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added a comment.

In https://reviews.llvm.org/D36233#829464, @ncharlie wrote:

> In https://reviews.llvm.org/D36233#829454, @mehdi_amini wrote:
>
> > What makes them fail?
>
>
> They're missing the new field I added to the bitcode, so they fail a bounds-check assert.


I'm glad we have these tests then :)
That's a real bug in your code: the bitcode has to be designed so that you can read back older version.


https://reviews.llvm.org/D36233