[Mesa-dev] [PATCH] nir: fix ir_binop_gequal glsl_to_nir conversion

Jason Ekstrand jason at jlekstrand.net
Mon Apr 16 19:19:20 UTC 2018


Pushed.  Thanks and welcome to Mesa!

On April 14, 2018 12:26:18 Jason Ekstrand <jason at jlekstrand.net> wrote:

> Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>
>
> What driver is hitting this path?  The !supports_ints path isn't used to my
> knowledge so if some driver has started using it, they're liable to find
> more bugs than just this one. :-)
>
> On April 14, 2018 12:16:48 Erico Nunes <nunes.erico at gmail.com> wrote:
>
> > ir_binop_gequal needs to be converted to nir_op_sge when native integers
> > are not supported in the driver.
> > Otherwise it becomes no different than ir_binop_less after the
> > conversion.
> >
> > Signed-off-by: Erico Nunes <nunes.erico at gmail.com>
> > ---
> > src/compiler/glsl/glsl_to_nir.cpp | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/src/compiler/glsl/glsl_to_nir.cpp
> > b/src/compiler/glsl/glsl_to_nir.cpp
> > index 17d58acc4c..8e5e9c3491 100644
> > --- a/src/compiler/glsl/glsl_to_nir.cpp
> > +++ b/src/compiler/glsl/glsl_to_nir.cpp
> > @@ -1832,7 +1832,7 @@ nir_visitor::visit(ir_expression *ir)
> >          else
> >             result = nir_uge(&b, srcs[0], srcs[1]);
> >       } else {
> > -         result = nir_slt(&b, srcs[0], srcs[1]);
> > +         result = nir_sge(&b, srcs[0], srcs[1]);
> >       }
> >       break;
> >    case ir_binop_equal:
> > --
> > 2.14.3
> >
> > _______________________________________________
> > mesa-dev mailing list
> > mesa-dev at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/mesa-dev





More information about the mesa-dev mailing list