evgeny777 added a comment. This needs to be rebased
================ Comment at: llvm/lib/Transforms/IPO/WholeProgramDevirt.cpp:1784 + ImportSummary->getTypeIdSummary(cast<MDString>(TypeId)->getString()); + if (!TidSummary) + RemoveTypeTestAssumes(); ---------------- tejohnson wrote: > Here is the fix for the issue with the multi-stage clang bootstrap test > failures described in D75201. I also restructured this code to try to make it > clearer (rather than an early continue when we don't need to remove type test > assumes, make it an explicit removal when needed). > > Added new test llvm/test/ThinLTO/X86/type_test_noindircall.ll for this. Can you please also test the case when LTO unit is split? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73242/new/ https://reviews.llvm.org/D73242 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
