[Mesa-dev] [PATCH] nir: fix ir_binop_gequal glsl_to_nir conversion
Jason Ekstrand
jason at jlekstrand.net
Sat Apr 14 19:26:15 UTC 2018
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