intel/defines: Explicitly cast to uint32_t in SET_FIELD and SET_BITS
authorJason Ekstrand <jason.ekstrand@intel.com>
Mon, 29 Oct 2018 22:17:43 +0000 (17:17 -0500)
committerJason Ekstrand <jason@jlekstrand.net>
Tue, 29 Jan 2019 18:43:55 +0000 (18:43 +0000)
commit009c0bd84020589c21ce25e5a5317abdd63cdb3a
tree2adcdf5fa85f9443351bc6264978508a8ea7fd1f
parent077b9557a4e18e267f2c7966d643a457f26644e2
intel/defines: Explicitly cast to uint32_t in SET_FIELD and SET_BITS

If you pass a bool in as the value to set, the C standard says that it
gets converted to an int prior to shifting.  If you try to set a bool to
bit 31, this lands you in undefined behavior.  It's better just to add
the explicit cast and let the compiler delete it for us.

Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>
src/intel/compiler/brw_eu_defines.h
src/mesa/drivers/dri/i965/brw_defines.h