<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_ASSIGNED "
title="ASSIGNED - [bisected] [HSW] Regression in clipping.user_defined.clip_* vulkancts tests"
href="https://bugs.freedesktop.org/show_bug.cgi?id=109081#c9">Comment # 9</a>
on <a class="bz_bug_link
bz_status_ASSIGNED "
title="ASSIGNED - [bisected] [HSW] Regression in clipping.user_defined.clip_* vulkancts tests"
href="https://bugs.freedesktop.org/show_bug.cgi?id=109081">bug 109081</a>
from <span class="vcard"><a class="email" href="mailto:t_arceri@yahoo.com.au" title="Timothy Arceri <t_arceri@yahoo.com.au>"> <span class="fn">Timothy Arceri</span></a>
</span></b>
<pre>(In reply to Kenneth Graunke from <a href="show_bug.cgi?id=109081#c8">comment #8</a>)
<span class="quote">> (In reply to Timothy Arceri from <a href="show_bug.cgi?id=109081#c7">comment #7</a>)
> > > I suspected it was something like that. I was planning to go through and
> > > beef up the places that handle if-statements to also handle bcsel. That
> > > doesn't (conceptually) seem to hard. What do you think?
> >
> > It should be fine for opt_peel_loop_initial_if() but for things like
> > opt_if_evaluate_condition_use() and
> > <a href="https://patchwork.freedesktop.org/patch/263442/">https://patchwork.freedesktop.org/patch/263442/</a> it's not really possible to
> > do.
>
> If that's the case, then maybe we ought to be doing peephole select
> optimizations (or at least the fancier versions?) later in the optimization
> process, after we do those other passes to clean up control flow?</span >
I tried that but last time I checked it resulted in more hurt then help, like
most optimisations its a bit of a trade off when and where its called.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the QA Contact for the bug.</li>
</ul>
</body>
</html>