diff options
author | Matt Turner <mattst88@gmail.com> | 2015-07-12 00:13:45 -0700 |
---|---|---|
committer | Matt Turner <mattst88@gmail.com> | 2015-07-29 09:34:51 -0700 |
commit | c1da15709a0c0c2775bd9e534f67c60f7dc95ce8 (patch) | |
tree | 67fa5a5d7ad58315d8ca012239f5a83e74fd668d /src/mesa/drivers/dri/i965/gen6_multisample_state.c | |
parent | c67ce2bd3b27a26d7f5665f296d307c0de39b720 (diff) | |
download | external_mesa3d-c1da15709a0c0c2775bd9e534f67c60f7dc95ce8.zip external_mesa3d-c1da15709a0c0c2775bd9e534f67c60f7dc95ce8.tar.gz external_mesa3d-c1da15709a0c0c2775bd9e534f67c60f7dc95ce8.tar.bz2 |
i965: Use float calculations when double is unnecessary.
Literals without an f/F suffix are of type double, and implicit
conversion rules specify that the float in (float op double) be
converted to a double before the operation is performed. I believe float
execution was intended (in nearly all cases) or is sufficient (in the
case of gen7_urb.c).
Removes a lot of float <-> double conversion instructions and replaces
many double instructions with float instructions which are cheaper.
text data bss dec hex filename
4928659 195160 26192 5150011 4e953b i965_dri.so before
4928315 195152 26192 5149659 4e93db i965_dri.so after
Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>
Diffstat (limited to 'src/mesa/drivers/dri/i965/gen6_multisample_state.c')
-rw-r--r-- | src/mesa/drivers/dri/i965/gen6_multisample_state.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/mesa/drivers/dri/i965/gen6_multisample_state.c b/src/mesa/drivers/dri/i965/gen6_multisample_state.c index 36734f5..cf1421e 100644 --- a/src/mesa/drivers/dri/i965/gen6_multisample_state.c +++ b/src/mesa/drivers/dri/i965/gen6_multisample_state.c @@ -148,7 +148,7 @@ unsigned gen6_determine_sample_mask(struct brw_context *brw) { struct gl_context *ctx = &brw->ctx; - float coverage = 1.0; + float coverage = 1.0f; float coverage_invert = false; unsigned sample_mask = ~0u; @@ -166,7 +166,7 @@ gen6_determine_sample_mask(struct brw_context *brw) } if (num_samples > 1) { - int coverage_int = (int) (num_samples * coverage + 0.5); + int coverage_int = (int) (num_samples * coverage + 0.5f); uint32_t coverage_bits = (1 << coverage_int) - 1; if (coverage_invert) coverage_bits ^= (1 << num_samples) - 1; |