summaryrefslogtreecommitdiffstats
path: root/src/gallium/drivers/freedreno
diff options
context:
space:
mode:
authorEric Engestrom <eric@engestrom.ch>2016-07-30 01:33:56 +0100
committerRob Clark <robdclark@gmail.com>2016-07-30 09:23:42 -0400
commita63bac9271cf4984b4b8c0158e99fffa7f78c8bb (patch)
tree784931e643ef950e0836783f9950c10241fe022e /src/gallium/drivers/freedreno
parent3563c4d16138360fd8756576e0c9c825ed6cec86 (diff)
downloadexternal_mesa3d-a63bac9271cf4984b4b8c0158e99fffa7f78c8bb.zip
external_mesa3d-a63bac9271cf4984b4b8c0158e99fffa7f78c8bb.tar.gz
external_mesa3d-a63bac9271cf4984b4b8c0158e99fffa7f78c8bb.tar.bz2
gallium/freedreno: move cast to avoid integer overflow
Previously, the bitshift would be performed on a simple int (32 bits on most systems), overflow, and then be cast to 64 bits. CovID: 1362461 Signed-off-by: Eric Engestrom <eric@engestrom.ch> Signed-off-by: Rob Clark <robdclark@gmail.com>
Diffstat (limited to 'src/gallium/drivers/freedreno')
-rw-r--r--src/gallium/drivers/freedreno/a2xx/fd2_compiler.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/gallium/drivers/freedreno/a2xx/fd2_compiler.c b/src/gallium/drivers/freedreno/a2xx/fd2_compiler.c
index 6829544..39418fc 100644
--- a/src/gallium/drivers/freedreno/a2xx/fd2_compiler.c
+++ b/src/gallium/drivers/freedreno/a2xx/fd2_compiler.c
@@ -402,10 +402,10 @@ add_src_reg(struct fd2_compile_context *ctx, struct ir2_instruction *alu,
swiz[3] = swiz_vals[src->SwizzleW];
swiz[4] = '\0';
- if ((ctx->need_sync & (uint64_t)(1 << num)) &&
+ if ((ctx->need_sync & ((uint64_t)1 << num)) &&
!(flags & IR2_REG_CONST)) {
alu->sync = true;
- ctx->need_sync &= ~(uint64_t)(1 << num);
+ ctx->need_sync &= ~((uint64_t)1 << num);
}
return ir2_reg_create(alu, num, swiz, flags);