[Mesa-dev] [PATCH] nir/opcodes: Fix constant-folding of ufind_msb
Bas Nieuwenhuizen
bas at basnieuwenhuizen.nl
Mon Aug 28 23:06:53 UTC 2017
Reviewed-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
On Tue, Aug 29, 2017, at 00:10, Jason Ekstrand wrote:
> We didn't fold correctly in the case of 0x1 because we never let the
> loop counter hit 0. Switching it to bit >= 0 solves this problem.
>
> Cc: mesa-stable at lists.freedesktop.org
> ---
> src/compiler/nir/nir_opcodes.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/compiler/nir/nir_opcodes.py
> b/src/compiler/nir/nir_opcodes.py
> index 39c01a7..a64a28e 100644
> --- a/src/compiler/nir/nir_opcodes.py
> +++ b/src/compiler/nir/nir_opcodes.py
> @@ -308,7 +308,7 @@ for (unsigned bit = 0; bit < 32; bit++) {
>
> unop_convert("ufind_msb", tint32, tuint32, """
> dst = -1;
> -for (int bit = 31; bit > 0; bit--) {
> +for (int bit = 31; bit >= 0; bit--) {
> if ((src0 >> bit) & 1) {
> dst = bit;
> break;
> --
> 2.5.0.400.gff86faf
>
> _______________________________________________
> 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