diff options
author | Ville Syrjälä <[email protected]> | 2017-06-05 14:01:58 +0300 |
---|---|---|
committer | Ville Syrjälä <[email protected]> | 2017-06-14 18:22:52 +0300 |
commit | c1eedb43f32f6a3733f26e7918eb028f68bd60a4 (patch) | |
tree | 20a52a806676c0e1644a7345635d5262ee169cfe /m4/ax_gcc_builtin.m4 | |
parent | 5f8b654b47af12f8e583df7911581379eba549e2 (diff) |
i915: Fix wpos_tex vs. -1 comparison
wpos_tex used to be a GLuint so assigning -1 to it and
later comparing with -1 worked correctly, but commit
c349031c27b7 ("i915: Fix texcoord vs. varying collision in
fragment programs") changed wpos_tex to uint8_t and hence
broke the comparison. To fix this define a more explicit
invalid value for wpos_tex.
gcc warns us:
i915_fragprog.c:1255:57: warning: comparison is always true due to limited range of data type [-Wtype-limits]
if (inputsRead & VARYING_BITS_TEX_ANY || p->wpos_tex != -1) {
^
And clang says:
i915_fragprog.c:1255:57: warning: comparison of constant -1 with expression of type 'uint8_t' (aka 'unsigned char') is always true [-Wtautological-constant-out-of-range-compare]
if (inputsRead & VARYING_BITS_TEX_ANY || p->wpos_tex != -1) {
~~~~~~~~~~~ ^ ~~
Cc: Chih-Wei Huang <[email protected]>
Cc: Eric Anholt <[email protected]>
Cc: Ian Romanick <[email protected]>
Cc: [email protected]
Fixes: c349031c27b7 ("i915: Fix texcoord vs. varying collision in fragment programs")
Signed-off-by: Ville Syrjälä <[email protected]>
Reviewed-by: Emil Velikov <[email protected]>
Diffstat (limited to 'm4/ax_gcc_builtin.m4')
0 files changed, 0 insertions, 0 deletions