[llvm] [Attributor] Do not optimize away externally_initialized loads. (PR #128170)

[Pierre van Houtryve via llvm-commits]

================
@@ -259,11 +259,16 @@ AA::getInitialValueForObj(Attributor &A, const AbstractAttribute &QueryingAA,
     if (!Initializer)
       return nullptr;
   } else {
-    if (!GV->hasLocalLinkage() &&
-        (GV->isInterposable() || !(GV->isConstant() && GV->hasInitializer())))
-      return nullptr;
-    if (!GV->hasInitializer())
-      return UndefValue::get(&Ty);
+    if (GV->hasLocalLinkage()) {
+      // Uninitialized global with local linkage.
+      if (!GV->hasInitializer())
----------------
Pierre-vh wrote:

@nikic Can I land the patch ? It'd fix a crash we've seen in the LTO pipeline.


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