<div dir="ltr">Looks good with one more fix:<div><br></div><div>It should have it's own header file, much like we have include/llvm/Transforms/Scalar.h and the declarations of the free functions should be moved from Scalar.h to its new header. The Scalar.h include should also be replaced.</div>
<div><br></div><div style>Feel free to submit once you've made that fixe</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Jan 26, 2013 at 4:53 PM, Michael Gottesman <span dir="ltr"><<a href="mailto:mgottesman@apple.com" target="_blank">mgottesman@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The following patch extracts ObjCARC into its own library. This is because I am going to in the near future start refactoring its multiple passes into separate files. This implies that it must be its own library since it will have multiple source files (if my understanding of the LLVM library criteria is correct).<br>

<br>
I have successfully built llvm/clang with my changes on Darwin with both cmake/ninja and autoconf/make.<br>
<br>
Please Review,<br>
<br>
Michael<br>
<br>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>