<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>