<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 30, 2015 at 4:04 PM, Hal Finkel <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">----- Original Message -----<br>
> From: "David Blaikie" <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>><br>
> To: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> Sent: Friday, January 30, 2015 5:52:19 PM<br>
> Subject: [llvm] r227645 - Add PPC test for r227481,   but XFAIL because this is actually more work than it appeared to<br>
> be.<br>
><br>
> Author: dblaikie<br>
> Date: Fri Jan 30 17:52:19 2015<br>
> New Revision: 227645<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=227645&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=227645&view=rev</a><br>
> Log:<br>
> Add PPC test for r227481, but XFAIL because this is actually more<br>
> work than it appeared to be.<br>
><br>
> Same sort of bug as on ARM where the cmp+branch are lowered to br_cc<br>
> (choosing the branch's debugloc for the br_cc's debugloc) then<br>
> expanded<br>
> out to a cmp and a br, but both using the debug loc of the br_cc,<br>
> thus<br>
> losing fidelity.<br>
<br>
</span>Do we need to have two debug locations on br_cc? (and maybe select_cc?)<br></blockquote><div><br>That would be one way to solve it, I'd imagine (echristo & grosbach were speculating about that on IRC earlier when I brought this up - but it's not enough of a priority for me to bother investigating that right now). I don't know enough about SelectionDAG to know if that's the right path forward, or if we should avoid coalescing those options if we can, etc.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
 -Hal<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
> Added:<br>
>     llvm/trunk/test/DebugInfo/PowerPC/line.test<br>
><br>
> Added: llvm/trunk/test/DebugInfo/PowerPC/line.test<br>
> URL:<br>
> <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/PowerPC/line.test?rev=227645&view=auto" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/PowerPC/line.test?rev=227645&view=auto</a><br>
> ==============================================================================<br>
> --- llvm/trunk/test/DebugInfo/PowerPC/line.test (added)<br>
> +++ llvm/trunk/test/DebugInfo/PowerPC/line.test Fri Jan 30 17:52:19<br>
> 2015<br>
> @@ -0,0 +1,7 @@<br>
> +; RUN: llc -mtriple=powerpc-unknown-linux -O0 -filetype=asm <<br>
> %S/../Inputs/line.ll | FileCheck %S/../Inputs/line.ll<br>
> +<br>
> +; This is more complex than it looked. It's mixed up somewhere in<br>
> SelectionDAG<br>
> +; (legalized as br_cc, losing the separation between the comparison<br>
> and the<br>
> +; branch, then further lowered to cmplwi + brcc but without the<br>
> fidelity that<br>
> +; those two instructions are on separate lines)<br>
> +; XFAIL: *<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>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Hal Finkel<br>
Assistant Computational Scientist<br>
Leadership Computing Facility<br>
Argonne National Laboratory<br>
</font></span></blockquote></div><br></div></div>