<html>
<head>
<base href="https://llvm.org/bugs/" />
</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 - missed opportunity for associative FP math in summation reduction (pairwise summation)" href="https://urldefense.proofpoint.com/v2/url?u=https-3A__llvm.org_bugs_show-5Fbug.cgi-3Fid-3D17305&d=AwMBaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=pF93YEPyB-J_PERP4DUZOJDzFVX5ZQ57vQk33wu0vio&m=rk4BixZMqytKc0zzd8HDLpN_OdyN6usepk_Mq4waE48&s=KprorWIEB-5g88mpaTv7up23MSzUA8RO_S7J6dLs9NI&e=">bug 17305</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></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_RESOLVED bz_closed" title="RESOLVED FIXED - missed opportunity for associative FP math in summation reduction (pairwise summation)" href="https://urldefense.proofpoint.com/v2/url?u=https-3A__llvm.org_bugs_show-5Fbug.cgi-3Fid-3D17305-23c4&d=AwMBaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=pF93YEPyB-J_PERP4DUZOJDzFVX5ZQ57vQk33wu0vio&m=rk4BixZMqytKc0zzd8HDLpN_OdyN6usepk_Mq4waE48&s=PvixbilshXnRAV_-oKSsB8d5O4YiZ2hOBSyDTsXPCr0&e=">Comment # 4</a>
on <a class="bz_bug_link
bz_status_RESOLVED bz_closed" title="RESOLVED FIXED - missed opportunity for associative FP math in summation reduction (pairwise summation)" href="https://urldefense.proofpoint.com/v2/url?u=https-3A__llvm.org_bugs_show-5Fbug.cgi-3Fid-3D17305&d=AwMBaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=pF93YEPyB-J_PERP4DUZOJDzFVX5ZQ57vQk33wu0vio&m=rk4BixZMqytKc0zzd8HDLpN_OdyN6usepk_Mq4waE48&s=KprorWIEB-5g88mpaTv7up23MSzUA8RO_S7J6dLs9NI&e=">bug 17305</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>Reassociation is now done in the machine combiner pass so it can be
register-pressure-aware.
The code produced after:
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject-3Fview-3Drevision-26revision-3D241871&d=AwMBaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=pF93YEPyB-J_PERP4DUZOJDzFVX5ZQ57vQk33wu0vio&m=rk4BixZMqytKc0zzd8HDLpN_OdyN6usepk_Mq4waE48&s=gG7Iufmubt5NGcpd4xZWQkBaAeWierGH_t9ZmNVdiEk&e=">http://llvm.org/viewvc/llvm-project?view=revision&revision=241871</a>
...is equivalent to that in <a href="show_bug.cgi?id=17305#c2">comment 2</a>.
It's still not the perfect binary tree of ops because that would require a
potentially big leap in compile time, but this is probably as good as it gets.
Other relevant commits:
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject-3Fview-3Drevision-26revision-3D239486&d=AwMBaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=pF93YEPyB-J_PERP4DUZOJDzFVX5ZQ57vQk33wu0vio&m=rk4BixZMqytKc0zzd8HDLpN_OdyN6usepk_Mq4waE48&s=7CcQ7Y8VxWlGUJDQqrj078kw_1veC3AcT3xXg39l7Jw&e=">http://llvm.org/viewvc/llvm-project?view=revision&revision=239486</a>
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject-3Fview-3Drevision-26revision-3D240361&d=AwMBaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=pF93YEPyB-J_PERP4DUZOJDzFVX5ZQ57vQk33wu0vio&m=rk4BixZMqytKc0zzd8HDLpN_OdyN6usepk_Mq4waE48&s=JgOAx58vIqPAwEx4nJiByK20ii74B896OplKY_ikQJA&e=">http://llvm.org/viewvc/llvm-project?view=revision&revision=240361</a>
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject-3Fview-3Drevision-26revision-3D241515&d=AwMBaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=pF93YEPyB-J_PERP4DUZOJDzFVX5ZQ57vQk33wu0vio&m=rk4BixZMqytKc0zzd8HDLpN_OdyN6usepk_Mq4waE48&s=5PCAuAji3TgdBYFAziSi8jexDbOGAr_BECDytMXaoEo&e=">http://llvm.org/viewvc/llvm-project?view=revision&revision=241515</a></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>