<br><br><div class="gmail_quote">On Wed, Oct 24, 2012 at 10:46 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Wed, Oct 24, 2012 at 4:15 AM, Alexey Samsonov <<a href="mailto:samsonov@google.com">samsonov@google.com</a>> wrote:<br>
> On Wed, Oct 17, 2012 at 9:48 PM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>><br>
> wrote:<br>
>><br>
>> I've gone ahead and reverted it here:<br>
>><br>
>>         D       test/CodeGenCXX/debug-info-user-def.cpp<br>
>>         M       test/CodeGenCXX/debug-lambda-expressions.cpp<br>
>>         M       test/CodeGenCXX/debug-info-template-quals.cpp<br>
>>         M       test/CodeGenCXX/debug-info-template-member.cpp<br>
>>         M       lib/CodeGen/CGDebugInfo.cpp<br>
>> W: -empty_dir: test/CodeGenCXX/debug-info-user-def.cpp<br>
>> r166109 = 7c70a2300fbd713858c31cc9cbbe074bdc233412<br>
>> (refs/remotes/origin/master)<br>
>><br>
>> Feel free to rebase some test suite changes on top of that.<br>
>><br>
>><br>
>> Alexey: If you need anything else let me know.<br>
><br>
><br>
> Yes, I think I still need a review for patch for PR13942.<br>
><br>
> <a href="https://codereview.appspot.com/6720044/" target="_blank">https://codereview.appspot.com/6720044/</a><br>
><br>
<br>
</div>LGTM. One possible cleanup for later: we have a lot of maybe... calls<br>
scattered all over the place. It'd be nice if we could get them<br>
consolidated somehow.<br>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div>r166676, thanks! </div></div><div><br></div>-- <br><div>Alexey Samsonov, MSK</div><br>