On Wednesday, May 4, 2016 5:41:37 PM PDT Kenneth Graunke wrote:
> On Wednesday, May 4, 2016 3:54:14 PM PDT Matt Turner wrote:
> > Ken suggested instead of a big and complicated optimization pass, to
> > just recognize the operations here. It's certainly less code and a lot
> > prettier, but it seems to actually perform worse for currently unknown
> > reasons.
> 
> One potential pitfall is that is_zero() currently returns false for
> -0.0f.  The fp64 series that's supposedly landing fixes this.
> 

That's fixed now.  With get_nir_src_imm simplified, this series is:

Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>

I'm not sure why the other one does better, but this is better than
nothing, and it's really simple.  We may as well land it...

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to