diff options
| author | Brian Paul <[email protected]> | 2015-10-14 09:10:19 -0600 |
|---|---|---|
| committer | Brian Paul <[email protected]> | 2015-10-15 07:21:07 -0600 |
| commit | 083b3f5cb4c5bd701d6a371282d7dc8c4f5fcaa8 (patch) | |
| tree | cd0f1d4b1c522fbd3aba99c349a4703542ed52c6 /src/mesa | |
| parent | 0de5e0f3fb0f3671a3ecec6ab4473f9131ecd0ae (diff) | |
mesa: short-cut new_state == _NEW_LINE in _mesa_update_state_locked()
We can skip to the end of _mesa_update_state_locked() if only the
_NEW_LINE flag is set since none of the derived state depends on it
(just like _NEW_CURRENT_ATTRIB). Note that we still call the
ctx->Driver.UpdateState() function, of course.
v2: use bitmask-based test, per Eric.
Reviewed-by: Eric Anholt <[email protected]>
Diffstat (limited to 'src/mesa')
| -rw-r--r-- | src/mesa/main/state.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/mesa/main/state.c b/src/mesa/main/state.c index d3b1c72b08d..4043c4f2057 100644 --- a/src/mesa/main/state.c +++ b/src/mesa/main/state.c @@ -391,8 +391,12 @@ _mesa_update_state_locked( struct gl_context *ctx ) GLbitfield new_state = ctx->NewState; GLbitfield prog_flags = _NEW_PROGRAM; GLbitfield new_prog_state = 0x0; + const GLbitfield computed_states = ~(_NEW_CURRENT_ATTRIB | _NEW_LINE); - if (new_state == _NEW_CURRENT_ATTRIB) + /* we can skip a bunch of state validation checks if the dirty + * state matches one or more bits in 'computed_states'. + */ + if ((new_state & computed_states) == 0) goto out; if (MESA_VERBOSE & VERBOSE_STATE) |