[llvm] [WPD] Reduce ThinLTO link time by avoiding unnecessary summary analysis (PR #164046)

[Mingming Liu via llvm-commits]

================
@@ -1184,6 +1180,7 @@ bool DevirtIndex::tryFindVirtualCallTargets(
         // with public LTO visibility.
         if (VS->getVCallVisibility() == GlobalObject::VCallVisibilityPublic)
           return false;
+        break;
----------------
mingmingl-llvm wrote:

Before this change, loop iterates all `VS` and returns false if any `VS` has public visibility.

After this change, the visibility check happens on the first `VS`. Is my understanding correct that we assumes one-definition-rule and all vtables with the same GUID has the same visibility?



https://github.com/llvm/llvm-project/pull/164046