<p dir="ltr">Looks like we got 4 xpasses out of it so it might be already covered. </p>
<div class="gmail_quote">On Aug 29, 2013 10:50 PM, "David Blaikie" <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Thu, Aug 29, 2013 at 10:45 PM, Yunzhong Gao<br>
<<a href="mailto:Yunzhong_Gao@playstation.sony.com">Yunzhong_Gao@playstation.sony.com</a>> wrote:<br>
><br>
>   Committed the fix in r189649.<br>
>   <a href="http://llvm.org/viewvc/llvm-project?view=revision&revision=189649" target="_blank">http://llvm.org/viewvc/llvm-project?view=revision&revision=189649</a><br>
<br>
Thanks! (for both the patch & your patience in review)<br>
<br>
><br>
>   Should I add tests to the debuginfo-tests repository as well? I wrote<br>
>   some tests here:<br>
>   <a href="http://llvm-reviews.chandlerc.com/D1280" target="_blank">http://llvm-reviews.chandlerc.com/D1280</a><br>
<br>
Personally I don't much like these end-to-end tests. There's value in<br>
full feature tests, to be sure, but this isn't exactly a complete<br>
suite - you might see if you could get them into the GDB test suite if<br>
there's nothing like that in there already & you're feeling<br>
exceptionally diligent. (at some point I'll need to port my GDB 7.5<br>
xfails, etc, up to GDB ToT & setup a Clang ToT + GDB ToT builder, but<br>
I haven't gotten around to it yet)<br>
<br>
><br>
> <a href="http://llvm-reviews.chandlerc.com/D1281" target="_blank">http://llvm-reviews.chandlerc.com/D1281</a><br>
</blockquote></div>