<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body><span class="vcard"><a class="email" href="mailto:sroland@vmware.com" title="Roland Scheidegger <sroland@vmware.com>"> <span class="fn">Roland Scheidegger</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW - llvmpipe blends produce bad code after llvm patch https://reviews.llvm.org/D44785"
href="https://bugs.freedesktop.org/show_bug.cgi?id=106231">bug 106231</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;">CC</td>
<td>
</td>
<td>jfonseca@vmware.com, sroland@vmware.com
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - llvmpipe blends produce bad code after llvm patch https://reviews.llvm.org/D44785"
href="https://bugs.freedesktop.org/show_bug.cgi?id=106231#c1">Comment # 1</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - llvmpipe blends produce bad code after llvm patch https://reviews.llvm.org/D44785"
href="https://bugs.freedesktop.org/show_bug.cgi?id=106231">bug 106231</a>
from <span class="vcard"><a class="email" href="mailto:sroland@vmware.com" title="Roland Scheidegger <sroland@vmware.com>"> <span class="fn">Roland Scheidegger</span></a>
</span></b>
<pre>(In reply to Tom Hudson from <a href="show_bug.cgi?id=106231#c0">comment #0</a>)
<span class="quote">> <a href="https://reviews.llvm.org/D44785">https://reviews.llvm.org/D44785</a> changed the way adds, addus, subs, subus are
> handled.
>
> llvmpipe issues llvm.x86.sse2.padds and llvm.x86.sse2.psubs in
> src/gallium/auxiliary/gallivm/lp_bld_arit.c:lp_build_add() and
> lp_build_sub().
>
> After D44785 landed, lp_test_blend.c started crash every time it entered
> LLVM-compiled code for type=u8nx16.
>
> Commenting out the issues of padds/psubs avoids this crash. The LLVM
> project, in discussing the bug at <a href="https://reviews.llvm.org/D44785">https://reviews.llvm.org/D44785</a>, suspects
> that the cause may be because llvmpipe is "missing the autoupgrade stage"?</span >
Autoupgrade doesn't work for jit code (at least I wouldn't know how, and it
never has in the past), so the way we handled disappearing of intrinsics in the
past was to just not use them any more (for newer llvm versions) and do
essentially the same as what autoupgrade would do, and we'll have to do the
same here.
I am however sure that in the past when intrinsics disappeared, it would
complain when compiling the IR, rather than just call 0 function in the
compiled code. Which is of course much nicer...
For instance when the min/max (integer) intrinsics disappared:
<a href="https://bugs.llvm.org/show_bug.cgi?id=28176">https://bugs.llvm.org/show_bug.cgi?id=28176</a>
But I'm not sure if just calling 0 function now is expected due to some llvm
changes, but if it is that's definitely making everybody's life harder...</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>