<html>
<head>
<base href="https://bugs.llvm.org/">
</head>
<body><span class="vcard"><a class="email" href="mailto:suc-daniil@yandex.ru" title="Daniil Suchkov <suc-daniil@yandex.ru>"> <span class="fn">Daniil Suchkov</span></a>
</span> changed
<a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - SSAUpdater corrupts SCEV"
href="https://bugs.llvm.org/show_bug.cgi?id=44320">bug 44320</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;">Resolution</td>
<td>---
</td>
<td>FIXED
</td>
</tr>
<tr>
<td style="text-align:right;">Status</td>
<td>CONFIRMED
</td>
<td>RESOLVED
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - SSAUpdater corrupts SCEV"
href="https://bugs.llvm.org/show_bug.cgi?id=44320#c5">Comment # 5</a>
on <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - SSAUpdater corrupts SCEV"
href="https://bugs.llvm.org/show_bug.cgi?id=44320">bug 44320</a>
from <span class="vcard"><a class="email" href="mailto:suc-daniil@yandex.ru" title="Daniil Suchkov <suc-daniil@yandex.ru>"> <span class="fn">Daniil Suchkov</span></a>
</span></b>
<pre>Fixed by:
commit 4a8dbc617d0b28ce2ce3e6856376b3802409554e
Author: Daniil Suchkov <<a href="mailto:suc-daniil@yandex.ru">suc-daniil@yandex.ru</a>>
Date: Tue Jan 21 12:21:53 2020 +0700
[SSAUpdater] Don't call ValueIsRAUWd upon single use replacement
It is incorrect to call ValueHandleBase::ValueIsRAUWd when only one use
is replaced since it simply violates semantics of the callback and leads
to bugs like PR44320.
Previously this call was used specifically to keep LICM's cache of
AliasSetTrackers up to date across passes (as PR36801 showed, even for
that purpose it didn't work properly), but since LICM doesn't have that
cache anymore, we can safely remove this incorrect call with no
repercussions.
This patch fixes <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - SSAUpdater corrupts SCEV"
href="show_bug.cgi?id=44320">https://bugs.llvm.org/show_bug.cgi?id=44320</a>
Reviewers: asbirlea, fhahn, efriedma, reames
Reviewed-By: asbirlea
Differential Revision: <a href="https://reviews.llvm.org/D73089">https://reviews.llvm.org/D73089</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>