<html>
<head>
<base href="https://bugs.llvm.org/">
</head>
<body><span class="vcard"><a class="email" href="mailto:max.kazantsev@azul.com" title="Max Kazantsev <max.kazantsev@azul.com>"> <span class="fn">Max Kazantsev</span></a>
</span> changed
<a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - ScalarEvolution::getGEPExpr assertion failure "GEP should not change type mid-flight.""
href="https://bugs.llvm.org/show_bug.cgi?id=46786">bug 46786</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>CONFIRMED
</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 - ScalarEvolution::getGEPExpr assertion failure "GEP should not change type mid-flight.""
href="https://bugs.llvm.org/show_bug.cgi?id=46786#c15">Comment # 15</a>
on <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - ScalarEvolution::getGEPExpr assertion failure "GEP should not change type mid-flight.""
href="https://bugs.llvm.org/show_bug.cgi?id=46786">bug 46786</a>
from <span class="vcard"><a class="email" href="mailto:max.kazantsev@azul.com" title="Max Kazantsev <max.kazantsev@azul.com>"> <span class="fn">Max Kazantsev</span></a>
</span></b>
<pre>Assert disabled by:
commit b96114c1e1fc4448ea966bce013706359aee3fa9
Author: Max Kazantsev <<a href="mailto:mkazantsev@azul.com">mkazantsev@azul.com</a>>
Date: Wed Jul 22 15:32:13 2020 +0700
[SCEV] Remove premature assert. PR46786
This assert was added to verify assumption that GEP's SCEV will be of
pointer type,
basing on fact that it should be a SCEVAddExpr with (at least) last operand
being
pointer. Two notes:
- GEP's SCEV does not have to be a SCEVAddExpr after all simplifications;
- In current state, GEP's SCEV does not have to have at least one pointer
operands
(all of them can become int during the transforms).
However, we might want to be at a point where it is true. We are currently
removing
this assert and will try to enumerate the cases where "is pointer" notion
might be
lost during the transforms. When all of them are fixed, we can return it.
Differential Revision: <a href="https://reviews.llvm.org/D84294">https://reviews.llvm.org/D84294</a>
Reviewed By: lebedev.ri
I'll try to help to find cases that we are still missing to have this assert,
because generally preserving a notion of something being a pointer seems
useful. Unfortunately, we are not yet in point to say we preserve it.</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>