It's fine by me either way, but if you decide to commit make sure you close this bug 
<span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)"> </span><a href="http://llvm.org/bugs/show_bug.cgi?id=10348" target="_blank" style="color:rgb(17,85,204);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)">http://llvm.org/bugs/show_bug.cgi?id=10348</a> <br>

<br><div class="gmail_quote">On Fri, Jul 13, 2012 at 1:42 PM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

I can roll it in now if we'd like to see that.<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Fri, Jul 13, 2012 at 4:03 AM, Nikola Smiljanic <<a href="mailto:popizdeh@gmail.com">popizdeh@gmail.com</a>> wrote:<br>
> There was some talk on the list about solving all file handling<br>
> problems in a general way, but I don't think anything happened with<br>
> that. The thread is called "LLVM & Clang file management" so you can<br>
> find more information there. I just stopped pinging about this patch<br>
> assuming it wouldn't be needed.<br>
><br>
> On Fri, Jul 13, 2012 at 12:12 AM, Jason Haslam <<a href="mailto:jason.haslam@gmail.com">jason.haslam@gmail.com</a>> wrote:<br>
>> Forgive me for resurrecting this old thread. Does anybody know why this<br>
>> patch was never applied? Was there some objection to it or was it just<br>
>> forgotten? It works for me and I would like to see it applied.<br>
>><br>
>> Jason<br>
>><br>
>><br>
>> On Dec 13, 2011, at 1:13 AM, Nikola Smiljanic wrote:<br>
>><br>
>> Pong<br>
>><br>
>> On Thu, Dec 1, 2011 at 9:15 PM, Nikola Smiljanic <<a href="mailto:popizdeh@gmail.com">popizdeh@gmail.com</a>> wrote:<br>
>>><br>
>>> Ping<br>
>>><br>
>>><br>
>>> On Mon, Nov 21, 2011 at 4:09 PM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>><br>
>>> wrote:<br>
>>>><br>
>>>> On Mon, Nov 21, 2011 at 8:54 AM, Nikola Smiljanic <<a href="mailto:popizdeh@gmail.com">popizdeh@gmail.com</a>><br>
>>>> wrote:<br>
>>>> > Will somebody with commit access please review the patch? Attached<br>
>>>> > version<br>
>>>> > should cleanly apply to trunk.<br>
>>>><br>
>>>> Still looks good to me -- but I don't have commit access, so someone<br>
>>>> else will have to roll it in.<br>
>>>><br>
>>>> ~Aaron<br>
>>>><br>
>>>> > On Wed, Oct 12, 2011 at 11:34 AM, Nikola Smiljanic <<a href="mailto:popizdeh@gmail.com">popizdeh@gmail.com</a>><br>
>>>> > wrote:<br>
>>>> >><br>
>>>> >>  What is the procedure now? Do we just wait for someone else to look<br>
>>>> >> at it<br>
>>>> >>  and commit (I'm guessing that you don't have commit access)?<br>
>>>> >> cfe-commits<br>
>>>> >> is<br>
>>>> >>  in the CC.<br>
>>>> >><br>
>>>> >> > On Tue, Oct 11, 2011 at 6:55 PM, Aaron Ballman<br>
>>>> >> > <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>><br>
>>>> >> > wrote:<br>
>>>> >> ><br>
>>>> >> > Looks good to me!<br>
>>>> >> ><br>
>>>> >> > Thanks!<br>
>>>> >> ><br>
>>>> >> > ~Aaron<br>
>>>> >><br>
>>>> ><br>
>>><br>
>>><br>
>><br>
>> _______________________________________________<br>
>> cfe-commits mailing list<br>
>> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
>><br>
>><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br>