<html>
<head>
<base href="https://bugs.llvm.org/">
</head>
<body><span class="vcard"><a class="email" href="mailto:spatel+llvm@rotateright.com" title="Sanjay Patel <spatel+llvm@rotateright.com>"> <span class="fn">Sanjay Patel</span></a>
</span> changed
<a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - Missing copysign(1., x) < 0. to std::signbit(x) fold"
href="https://bugs.llvm.org/show_bug.cgi?id=49179">bug 49179</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">Status</td>
<td>NEW
</td>
<td>RESOLVED
</td>
</tr>
<tr>
<td style="text-align:right;">Resolution</td>
<td>---
</td>
<td>FIXED
</td>
</tr>
<tr>
<td style="text-align:right;">Fixed By Commit(s)</td>
<td>
</td>
<td>85294703a
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - Missing copysign(1., x) < 0. to std::signbit(x) fold"
href="https://bugs.llvm.org/show_bug.cgi?id=49179#c6">Comment # 6</a>
on <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - Missing copysign(1., x) < 0. to std::signbit(x) fold"
href="https://bugs.llvm.org/show_bug.cgi?id=49179">bug 49179</a>
from <span class="vcard"><a class="email" href="mailto:spatel+llvm@rotateright.com" title="Sanjay Patel <spatel+llvm@rotateright.com>"> <span class="fn">Sanjay Patel</span></a>
</span></b>
<pre>Should be fixed in IR with:
<a href="https://reviews.llvm.org/rG85294703a74a">https://reviews.llvm.org/rG85294703a74a</a>
We have seen complications with these kinds of transforms because LLVM supports
non-IEEE754-compliant targets, but I think this is always ok. If not, we'll
have to move the transform to the backend.
As noted in the comments, if variations of the specified pattern are found,
please file other bugs.
And finally (it's not clear from the description if we care about a particular
target), if the performance is still not optimal, we may need to adjust codegen
too. For x86, I'm seeing something like this currently:
movmskpd %xmm0, %eax
andl $1, %eax</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are on the CC list for the bug.</li>
</ul>
</body>
</html>