i965: Move some /* CACHE_NEW_SAMPLER */ comments.
authorKenneth Graunke <kenneth@whitecape.org>
Tue, 25 Nov 2014 06:20:29 +0000 (22:20 -0800)
committerKenneth Graunke <kenneth@whitecape.org>
Sat, 29 Nov 2014 10:18:44 +0000 (02:18 -0800)
commit324368b500fc83fc8aa801ac2ab4db4427ebece1
treead340ca669adea9d6f213603ddfed6beb8ff8b37
parent66ebfad3cd123b6e23e28bdf17cda203b7c99a30
i965: Move some /* CACHE_NEW_SAMPLER */ comments.

Marking brw_stage_state::sampler_count as CACHE_NEW_SAMPLER is wrong.

The number of samplers used by each program is actually computed at
draw time (brw_try_draw_prims), based purely on the currently bound
shader programs (gl_program::SamplersUsed).

CACHE_NEW_SAMPLER means that we've emitted a new SAMPLER_STATE table.
Although this could indicate that the number of samplers has changed,
it could also simply mean that the contents of the table has changed
(i.e. we've bound different textures).

The real reason these atoms depend on CACHE_NEW_SAMPLER is because they
include a pointer to the SAMPLER_STATE table.  This was not commented.

So, move the comments to the appropriate place.

Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Kristian Høgsberg <krh@bitplanet.net>
Reviewed-by: Matt Turner <mattst88@gmail.com>
src/mesa/drivers/dri/i965/brw_vs_state.c
src/mesa/drivers/dri/i965/brw_wm_state.c