================
@@ -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

Reply via email to