<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Unigine Heaven 4.0 logging vm faults since radeon/llvm: Run LLVM's instruction combining pass"
href="https://bugs.freedesktop.org/show_bug.cgi?id=90266#c5">Comment # 5</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Unigine Heaven 4.0 logging vm faults since radeon/llvm: Run LLVM's instruction combining pass"
href="https://bugs.freedesktop.org/show_bug.cgi?id=90266">bug 90266</a>
from <span class="vcard"><a class="email" href="mailto:adf.lists@gmail.com" title="Andy Furniss <adf.lists@gmail.com>"> <span class="fn">Andy Furniss</span></a>
</span></b>
<pre>(In reply to Tom Stellard from <a href="show_bug.cgi?id=90266#c4">comment #4</a>)
<span class="quote">> (In reply to Andy Furniss from <a href="show_bug.cgi?id=90266#c0">comment #0</a>)
> > Related to
> >
> > <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - Unigine Valley regression since radeon/llvm: Run LLVM's instruction combining pass"
href="show_bug.cgi?id=90056">https://bugs.freedesktop.org/show_bug.cgi?id=90056</a>
> >
> > Unlike valley I thought heaven 4.0 wasn't affected by above as it runs and
> > looks OK.
> >
> > There is however an issue, again "fixed" by reverting mesa commit
> >
>
> Thanks for the report, I just have a few qustions:
>
> Are there any problems with Valley after the fix for <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - Unigine Valley regression since radeon/llvm: Run LLVM's instruction combining pass"
href="show_bug.cgi?id=90056">Bug 90056</a> was committed?</span >
No, Valley is OK
<span class="quote">> Does the problem reported in this bug for Heaven still happen if you keep
> the InsCombine pass enabled, but revert the fix from <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - Unigine Valley regression since radeon/llvm: Run LLVM's instruction combining pass"
href="show_bug.cgi?id=90056">Bug 90056</a>?</span >
Yes with or without the fix Heaven logs the errors.
Initially tested on an older llvm with and without the patch from -
<a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - Unigine Valley regression since radeon/llvm: Run LLVM's instruction combining pass"
href="show_bug.cgi?id=90056#c14">https://bugs.freedesktop.org/show_bug.cgi?id=90056#c14</a>
and again just now by reverting "R600/SI: Fix verifier errors from the
SIAnnotateControlFlow" on a more recent llvm.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>