[PATCH] D53596: [ThinLTO] Fix a crash in lazy loading of Metadata

[Duncan P. N. Exon Smith via Phabricator via llvm-commits]

dexonsmith added a comment.

In https://reviews.llvm.org/D53596#1287633, @dexonsmith wrote:

> Hmm.  I remember writing this code.  I have a faint recollection of proving at the time that these could never be forward refs, and that it was important for bitcode reading performance.


Sorry, I neglected to say anything actionable.

- If any bitcode writer in a released Clang has written forward refs for scope/inline-at, then clearly we need to handle this in the reader.  But if that's not the case, perhaps we can instead change the writer to make it impossible again.  (I think I might have intentionally used `dyn_cast` instead of `dyn_cast_or_null` as an assertion in the reader.)
- Either way, you might look at a bitcode-reading profile and see if this is still something to be careful about for compile-time.
- Note: I wrote that code way back before we had lazy loading of metadata and I don't remember if/how things changed there.


Repository:
  rL LLVM

https://reviews.llvm.org/D53596