mesa: Silence narrowing warnings in ff_fragment_shader's emit_texenv().
authorKenneth Graunke <kenneth@whitecape.org>
Mon, 24 Sep 2012 05:38:58 +0000 (22:38 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Mon, 24 Sep 2012 05:38:58 +0000 (22:38 -0700)
Recent version of GCC report a warning for the implicit conversion from
int to float:

  ff_fragment_shader.cpp:897:3: warning: narrowing conversion of '(1 << ((int)rgb_shift))' from 'int' to 'float' inside { } is ill-formed in C++11 [-Wnarrowing]

This is because floats cannot precisely represent all possible 32-bit
integer values.  However, texenv code is all expected to be floating
point, so this should not be a problem.

Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/main/ff_fragment_shader.cpp

index e850d47dd80bdfcc81c848d896840f4058397677..f21cf80ae3d7d01880626c8cdb5eabbd49a3b80d 100644 (file)
@@ -890,10 +890,10 @@ emit_texenv(struct texenv_fragment_program *p, GLuint unit)
       }
       else {
         float const_data[4] = {
-           1 << rgb_shift,
-           1 << rgb_shift,
-           1 << rgb_shift,
-           1 << alpha_shift
+           float(1 << rgb_shift),
+           float(1 << rgb_shift),
+           float(1 << rgb_shift),
+           float(1 << alpha_shift)
         };
         shift = new(p->mem_ctx) ir_constant(glsl_type::vec4_type,
                                             (ir_constant_data *)const_data);