<div dir="ltr">Sorry for the typos in the first commit message<br><div class="gmail_extra"><br clear="all"><div><div data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div>- Chuck<a href="tel:%28518%29%20881-1183" value="+15188811183" target="_blank"></a><br></div></div></div></div></div></div></div>
<br><div class="gmail_quote">On Fri, Sep 9, 2016 at 11:05 AM, Chuck Atkins <span dir="ltr"><<a href="mailto:chuck.atkins@kitware.com" target="_blank">chuck.atkins@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>When LCSSAID and LoopSimplifyID are declared extern inside a function body,<br>
some linkers will fail to resolve the actual symbnols defined in the llvm<br>
</span>namespace.  By removing the function local extern declarations and relying<br>
<div><div>on the header definitions in Scalar.h, the symbols can be properly reolved.<br>
This is specifically an issue when building with the Intel compiler.<br>
---<br>
 lib/Transforms/Utils/LoopUtil<wbr>s.cpp | 3 +--<br>
 1 file changed, 1 insertion(+), 2 deletions(-)<br>
<br>
diff --git a/lib/Transforms/Utils/LoopUti<wbr>ls.cpp b/lib/Transforms/Utils/LoopUti<wbr>ls.cpp<br>
index 8878689..9a7813f 100644<br>
--- a/lib/Transforms/Utils/LoopUti<wbr>ls.cpp<br>
+++ b/lib/Transforms/Utils/LoopUti<wbr>ls.cpp<br>
@@ -11,6 +11,7 @@<br>
 //<br>
 //===------------------------<wbr>------------------------------<wbr>----------------===//<br>
<br>
+#include "llvm/Transforms/Scalar.h"<br>
 #include "llvm/Transforms/Utils/LoopUti<wbr>ls.h"<br>
 #include "llvm/Analysis/AliasAnalysis.h<wbr>"<br>
 #include "llvm/Analysis/BasicAliasAnaly<wbr>sis.h"<br>
@@ -940,8 +941,6 @@ void llvm::getLoopAnalysisUsage(Ana<wbr>lysisUsage &AU) {<br>
<br>
   // We must also preserve LoopSimplify and LCSSA. We locally access their IDs<br>
   // here because users shouldn't directly get them from this header.<br>
-  extern char &LoopSimplifyID;<br>
-  extern char &LCSSAID;<br>
   AU.addRequiredID(LoopSimplify<wbr>ID);<br>
   AU.addPreservedID(LoopSimplif<wbr>yID);<br>
   AU.addRequiredID(LCSSAID);<br>
--<br>
2.7.4<br>
<br>
</div></div></blockquote></div><br></div></div>