Mesa (master): nir: Allow nir_format conversions to work on 32-bit values
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Fri Feb 28 12:11:47 UTC 2020
Module: Mesa
Branch: master
Commit: 574b03eebfba1a4b9de1a497857f0bbdca76ef19
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=574b03eebfba1a4b9de1a497857f0bbdca76ef19
Author: Icecream95 <ixn at keemail.me>
Date: Wed Feb 26 19:29:03 2020 +1300
nir: Allow nir_format conversions to work on 32-bit values
The constant has to changed to unsigned long long, as shifting a
32-bit value by 32 is undefined behaviour.
Reviewed-by: Alyssa Rosenzweig <alyssa.rosenzweig at collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/3943>
---
src/compiler/nir/nir_format_convert.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/compiler/nir/nir_format_convert.h b/src/compiler/nir/nir_format_convert.h
index 23f85ada8d8..a9de69e695f 100644
--- a/src/compiler/nir/nir_format_convert.h
+++ b/src/compiler/nir/nir_format_convert.h
@@ -214,8 +214,8 @@ _nir_format_norm_factor(nir_builder *b, const unsigned *bits,
nir_const_value factor[NIR_MAX_VEC_COMPONENTS];
memset(factor, 0, sizeof(factor));
for (unsigned i = 0; i < num_components; i++) {
- assert(bits[i] < 32);
- factor[i].f32 = (1ul << (bits[i] - is_signed)) - 1;
+ assert(bits[i] <= 32);
+ factor[i].f32 = (1ull << (bits[i] - is_signed)) - 1;
}
return nir_build_imm(b, num_components, 32, factor);
}
More information about the mesa-commit
mailing list