<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>Thanks again!</div><div><br></div><div>r173647.</div><br><div><div>On Jan 27, 2013, at 1:50 PM, Chandler Carruth <<a href="mailto:chandlerc@gmail.com">chandlerc@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr">On Sun, Jan 27, 2013 at 1:04 PM, Bill Wendling <span dir="ltr"><<a href="mailto:wendling@apple.com" target="_blank" class="cremed">wendling@apple.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Cool! Chandler, what do you think?<br></blockquote><div><br></div><div style="">I have no strong opinions. =] I'm happy with whatever organization makes sense to folks who actually work on the ARC optimizer.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
-bw<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Jan 27, 2013, at 2:14 AM, Michael Gottesman <<a href="mailto:mgottesman@apple.com" class="cremed">mgottesman@apple.com</a>> wrote:<br>
<br>
> (i.e. I agree and am going to do that).<br>
><br>
> On Jan 27, 2013, at 2:11 AM, Michael Gottesman <<a href="mailto:mgottesman@apple.com" class="cremed">mgottesman@apple.com</a>> wrote:<br>
><br>
>> I was actually thinking the same thing earlier this evening Bill. (you read my mind).<br>
>><br>
>> On Jan 27, 2013, at 2:02 AM, Bill Wendling <<a href="mailto:wendling@apple.com" class="cremed">wendling@apple.com</a>> wrote:<br>
>><br>
>>> I don't have a problem with splitting the file up into its own library (I think it's a good idea). I did have a question of whether it would be better to put it in its own directory (like the InstCombine stuff) instead of making it a subdirectory of Transforms/Scalar.<br>
>>><br>
>>> Thoughts?<br>
>>><br>
>>> -bw<br>
>>><br>
>>> On Jan 26, 2013, at 10:38 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com" class="cremed">chandlerc@google.com</a>> wrote:<br>
>>><br>
>>>> Looks good with one more fix:<br>
>>>><br>
>>>> 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.<br>
>>>><br>
>>>> Feel free to submit once you've made that fixe<br>
>>>><br>
>>>><br>
>>>> On Sat, Jan 26, 2013 at 4:53 PM, Michael Gottesman <<a href="mailto:mgottesman@apple.com" class="cremed">mgottesman@apple.com</a>> wrote:<br>
>>>> 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" class="cremed">llvm-commits@cs.uiuc.edu</a><br>
>>>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
>>>><br>
>>>><br>
>>>> _______________________________________________<br>
>>>> llvm-commits mailing list<br>
>>>> <a href="mailto:llvm-commits@cs.uiuc.edu" class="cremed">llvm-commits@cs.uiuc.edu</a><br>
>>>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
>>><br>
>><br>
>> _______________________________________________<br>
>> llvm-commits mailing list<br>
>> <a href="mailto:llvm-commits@cs.uiuc.edu" class="cremed">llvm-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
><br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" class="cremed">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>
</blockquote></div><br></body></html>