diff options
author | Kenneth Graunke <[email protected]> | 2014-03-13 02:28:14 -0700 |
---|---|---|
committer | Kenneth Graunke <[email protected]> | 2014-04-08 13:09:46 -0700 |
commit | ec1baea95a4a654de5c80089b01e0e8f38f7129a (patch) | |
tree | 76fe9bea63c8949c326afc353bb842bb7d63a357 | |
parent | ee839cc6ef92d37ec6a44e6036e7a2c46172a16a (diff) |
i965: Delete "fast color clear unsupported" performance warning.
Applications frequently clear to colors other than 0.0 or 1.0, which
prevents us from doing fast color clears. In that case, we issue this
performance warning on basically every glClear call, resulting in so
much spam that it's nearly impossible to see any other messages.
Plus, I don't think it's useful. We aren't suggesting a better way to
do what the application developers want---we're just telling them it
would be faster to do something they don't want.
Driver developers have no control over the clear color, so this message
is totally useless to them.
A better alternative to get this sort of information is to use
INTEL_DEBUG=blorp, which tells you whether color clears were fast,
simd16 repdata, or slow.
v2: Rebase on has_color_component changes.
Signed-off-by: Kenneth Graunke <[email protected]>
Reviewed-by: Ian Romanick <[email protected]>
-rw-r--r-- | src/mesa/drivers/dri/i965/brw_blorp_clear.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_blorp_clear.cpp b/src/mesa/drivers/dri/i965/brw_blorp_clear.cpp index d16da97c3c0..d8aa53c854c 100644 --- a/src/mesa/drivers/dri/i965/brw_blorp_clear.cpp +++ b/src/mesa/drivers/dri/i965/brw_blorp_clear.cpp @@ -155,8 +155,6 @@ is_color_fast_clear_compatible(struct brw_context *brw, for (int i = 0; i < 4; i++) { if (color->f[i] != 0.0 && color->f[i] != 1.0 && _mesa_format_has_color_component(format, i)) { - perf_debug("Clear color unsupported by fast color clear. " - "Falling back to slow clear.\n"); return false; } } |