<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body><span class="vcard"><a class="email" href="mailto:samuel.pitoiset@gmail.com" title="Samuel Pitoiset <samuel.pitoiset@gmail.com>"> <span class="fn">Samuel Pitoiset</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW - [LLVM Regression,bisected] Witcher 3 Corrupted Terrain"
href="https://bugs.freedesktop.org/show_bug.cgi?id=108935">bug 108935</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;">Summary</td>
<td>[LLVM Regression] Witcher 3 Corrupted Terrain
</td>
<td>[LLVM Regression,bisected] Witcher 3 Corrupted Terrain
</td>
</tr>
<tr>
<td style="text-align:right;">CC</td>
<td>
</td>
<td>nhaehnle@gmail.com
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [LLVM Regression,bisected] Witcher 3 Corrupted Terrain"
href="https://bugs.freedesktop.org/show_bug.cgi?id=108935#c1">Comment # 1</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [LLVM Regression,bisected] Witcher 3 Corrupted Terrain"
href="https://bugs.freedesktop.org/show_bug.cgi?id=108935">bug 108935</a>
from <span class="vcard"><a class="email" href="mailto:samuel.pitoiset@gmail.com" title="Samuel Pitoiset <samuel.pitoiset@gmail.com>"> <span class="fn">Samuel Pitoiset</span></a>
</span></b>
<pre>The problem can be reproduced on Polaris using the attached RenderDoc capture.
Bisected to:
41f457ff4f2e1571126d778f897efd274d1993ba is the first bad commit
commit 41f457ff4f2e1571126d778f897efd274d1993ba
Author: Nicolai Haehnle <<a href="mailto:nhaehnle@gmail.com">nhaehnle@gmail.com</a>>
Date: Thu Nov 29 11:06:26 2018 +0000
AMDGPU/InsertWaitcnts: Remove the dependence on MachineLoopInfo
Summary:
MachineLoopInfo cannot be relied on for correctness, because it cannot
properly recognize loops in irreducible control flow which can be
introduced by late machine basic block optimization passes. See the new
test case for the reduced form of an example that occurred in practice.
Use a simple fixpoint iteration instead.
In order to facilitate this change, refactor WaitcntBrackets so that it
only tracks pending events and registers, rather than also maintaining
state that is relevant for the high-level algorithm. Various accessor
methods can be removed or made private as a consequence.
Affects (in radv):
-
dEQP-VK.glsl.loops.special.{for,while}_uniform_iterations.select_iteration_count_{fragment,vertex}
Fixes: r345719 ("AMDGPU: Rewrite SILowerI1Copies to always stay on SALU")
Reviewers: msearles, rampitec, scott.linder, kanarayan
Subscribers: arsenm, kzhuravl, jvesely, wdng, yaxunl, dstuttard, tpr,
t-tye, llvm-commits, hakzsam
Differential Revision: <a href="https://reviews.llvm.org/D54231">https://reviews.llvm.org/D54231</a>
git-svn-id: <a href="https://llvm.org/svn/llvm-project/llvm/trunk@347853">https://llvm.org/svn/llvm-project/llvm/trunk@347853</a>
91177308-0d34-0410-b5e6-96231b3b80d8</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the QA Contact for the bug.</li>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>