[PATCH] D26840: [ThinLTO] Fix crash when importing an opaque type

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 18 09:37:02 PST 2016


mehdi_amini added a comment.

The thing that makes me worried is that this only about the initial types: I wonder if you couldn't start with an empty module, and then import an opaque type, and only then import the definition for this type. I'll try to forge an example.

Also there are calls to addOpaque() during the import, what you suggest seems that some cleanup would be needed.


https://reviews.llvm.org/D26840





More information about the llvm-commits mailing list