nir: coverity sign-extension fix
authorRob Clark <robclark@freedesktop.org>
Wed, 18 May 2016 14:17:02 +0000 (10:17 -0400)
committerRob Clark <robclark@freedesktop.org>
Fri, 20 May 2016 15:13:50 +0000 (11:13 -0400)
commit53c48feae086bbf5b17798fc1cd98ab14a17c7a8
tree7c8e8663024acda5fe7c1b92371eb2f4b2959362
parentbb993da7959da07cbbbd0f8d41e07652263260e6
nir: coverity sign-extension fix

Not 100% sure, but I think being an unsigned literal will help:

CID 1358505 (#1 of 1): Unintended sign extension
(SIGN_EXTENSION)sign_extension: Suspicious implicit sign extension:
load1->def.num_components with type unsigned char (8 bits, unsigned) is
promoted in load1->def.num_components * (load1->def.bit_size / 8) to
type int (32 bits, signed), then sign-extended to type unsigned long (64
bits, unsigned). If load1->def.num_components * (load1->def.bit_size /
8) is greater than 0x7FFFFFFF, the upper bits of the result will all be
1.

Signed-off-by: Rob Clark <robclark@freedesktop.org>
Reviewed-by: Matt Turner <mattst88@gmail.com>
src/compiler/nir/nir_instr_set.c