[Bug 94744] Merge similar if-statements

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=94744

--- Comment #3 from Ian Romanick <idr at freedesktop.org> ---
(In reply to Timothy Arceri from comment #2)
> (In reply to Eero Tamminen from comment #1)
> > > Existing optimization passes should turn this into a pair of bcsel instructions.
> > 
> > Ian, is this already done or do you have patches for it?
> 
> This is done however i965 doesn't make use of it. If you pass a non-zero
> limit to nir_opt_peephole_select() (i965 currently has a limit of zero, all
> other nir drivers seem to pass a value) these if-statments should get turned
> into bcsel.

nip_opt_peephole_select would help with this specific case, but I have seen
other cases where the bodies of the then-blocks and the else-blocks are quite
large.  I noticed one or two cases of this recently, but I don't recall which
shaders.  I will probably work on this fairly soon (sometime this year) if
someone doesn't beat me to it.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/intel-3d-bugs/attachments/20180718/90bac0f1/attachment.html>