fix infinite recursion if a global's initializer references the global

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@51617 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/tools/lto2/LTOModule.cpp b/tools/lto2/LTOModule.cpp
index c4bee714c6c8b1faa1d238ae60c214a391bc7498..eb0a094934f3892dd5089897249d8d69a0f9c2f9 100644 (file)
--- a/tools/lto2/LTOModule.cpp
+++ b/tools/lto2/LTOModule.cpp
@@ -161,7 +161,7 @@ void LTOModule::addDefinedDataSymbol(GlobalValue* v, Mangler &mangler)
     addDefinedSymbol(v, mangler, false); 
 
     // add external symbols referenced by this data.
-    for (unsigned count = 0, total = v->getNumOperands();\
+    for (unsigned count = 0, total = v->getNumOperands();
                                                 count != total; ++count) {
         findExternalRefs(v->getOperand(count), mangler);
     }
@@ -234,8 +234,13 @@ void LTOModule::findExternalRefs(Value* value, Mangler &mangler) {
     if (GlobalValue* gv = dyn_cast<GlobalValue>(value)) {
         if ( !gv->hasExternalLinkage() )
             addPotentialUndefinedSymbol(gv, mangler);
+               // If this is a variable definition, do not recursively process
+               // initializer.  It might contain a reference to this variable
+               // and cause an infinite loop.  The initializer will be
+               // processed in addDefinedDataSymbol(). 
+           return;
     }
-
+       
     // GlobalValue, even with InternalLinkage type, may have operands with 
     // ExternalLinkage type. Do not ignore these operands.
     if (Constant* c = dyn_cast<Constant>(value)) {