<p dir="ltr">Ping?</p>
<div class="gmail_extra"><br><div class="gmail_quote">On Sep 30, 2016 9:43 AM, "Chuck Atkins" <<a href="mailto:chuck.atkins@kitware.com">chuck.atkins@kitware.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">chuckatkins updated this revision to Diff 73043.<br>
chuckatkins added a comment.<br>
<br>
Adjusted code style with clang-format<br>
<br>
<br>
<a href="https://reviews.llvm.org/D24670" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D24670</a><br>
<br>
Files:<br>
  lib/Transforms/Utils/<wbr>LoopUtils.cpp<br>
<br>
<br>
Index: lib/Transforms/Utils/<wbr>LoopUtils.cpp<br>
==============================<wbr>==============================<wbr>=======<br>
--- lib/Transforms/Utils/<wbr>LoopUtils.cpp<br>
+++ lib/Transforms/Utils/<wbr>LoopUtils.cpp<br>
@@ -929,6 +929,13 @@<br>
   return UsedOutside;<br>
 }<br>
<br>
+// We locally access their IDs for LoopSimplify and LCSSA here because<br>
+// users shouldn't directly get them from this header.<br>
+namespace llvm {<br>
+extern char &LoopSimplifyID;<br>
+extern char &LCSSAID;<br>
+}<br>
+<br>
 void llvm::getLoopAnalysisUsage(<wbr>AnalysisUsage &AU) {<br>
   // By definition, all loop passes need the LoopInfo analysis and the<br>
   // Dominator tree it depends on. Because they all participate in the loop<br>
@@ -938,10 +945,7 @@<br>
   AU.addRequired<<wbr>LoopInfoWrapperPass>();<br>
   AU.addPreserved<<wbr>LoopInfoWrapperPass>();<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>
+  // We must also preserve LoopSimplify and LCSSA.<br>
   AU.addRequiredID(<wbr>LoopSimplifyID);<br>
   AU.addPreservedID(<wbr>LoopSimplifyID);<br>
   AU.addRequiredID(LCSSAID);<br>
<br>
<br>
</blockquote></div></div>