================ @@ -1333,10 +1377,10 @@ bool DevirtModule::trySingleImplDevirt( if (!IsExported) return false; - // If the only implementation has local linkage, we must promote to external - // to make it visible to thin LTO objects. We can only get here during the - // ThinLTO export phase. - if (TheFn->hasLocalLinkage()) { + // In LTO mode, if the only implementation has local linkage, we must promote + // to external to make it visible to thin LTO objects. We can only get here + // during the ThinLTO export phase. ---------------- teresajohnson wrote:
Is the last sentence still true? https://github.com/llvm/llvm-project/pull/145031 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits