nir: Allow nir_format conversions to work on 32-bit values
authorIcecream95 <ixn@keemail.me>
Wed, 26 Feb 2020 06:29:03 +0000 (19:29 +1300)
committerMarge Bot <eric+marge@anholt.net>
Fri, 28 Feb 2020 11:52:40 +0000 (11:52 +0000)
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@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/3943>

src/compiler/nir/nir_format_convert.h

index 23f85ada8d8fc0abbc1ae79ef5103fe2a6a5e5b3..a9de69e695fcb416acefc142c5175c0ccf195c38 100644 (file)
@@ -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);
 }