[PATCH] D33513: [ThinLTO] Fix ThinLTO crash while destroying context

Charles Saternos via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 29 09:52:32 PDT 2017


ncharlie added a comment.

@pcc @mehdi_amini I've simplified the tests using your recommendations. Note that to get lazy loading to work with llvm-dis, I had to set the `IsImporting` bool to true - I'm not sure if this is correct, but I couldn't set the value to true otherwise. Should I add a flag to allow enabling/disabling it instead of hardcoding it?


https://reviews.llvm.org/D33513





More information about the llvm-commits mailing list