i965: Remove four and a half year old TODO comments about samplers.
authorKenneth Graunke <kenneth@whitecape.org>
Wed, 22 Aug 2012 06:25:50 +0000 (23:25 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Sat, 25 Aug 2012 19:01:09 +0000 (12:01 -0700)
I can't actually understand what these mean, and they seem to
essentially say "we should simplify things", which is a nice goal but
not very specific.

Presumably things got cleaned up at some point.

Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Paul Berry <stereotype441@gmail.com>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Eric Anholt <eric@anholt.net>
src/mesa/drivers/dri/i965/brw_wm_sampler_state.c
src/mesa/drivers/dri/i965/gen7_sampler_state.c

index 1c61bd0816f09e188718df1022668614efccb91a..27f0d706e6a3e6ae3a90c00903bbb00cc9591c5c 100644 (file)
@@ -328,10 +328,6 @@ static void brw_update_sampler_state(struct brw_context *brw,
 }
 
 
-/* All samplers must be uploaded in a single contiguous array, which
- * complicates various things.  However, this is still too confusing -
- * FIXME: simplify all the different new texture state flags.
- */
 static void
 brw_upload_samplers(struct brw_context *brw)
 {
index 0cf62298412a8da9162c148f4a18916ccb8f637c..3c8daf9a1538e45ef7a10656d89bb79885d3fd38 100644 (file)
@@ -183,10 +183,6 @@ gen7_update_sampler_state(struct brw_context *brw, int unit,
 }
 
 
-/* All samplers must be uploaded in a single contiguous array, which
- * complicates various things.  However, this is still too confusing -
- * FIXME: simplify all the different new texture state flags.
- */
 static void
 gen7_upload_samplers(struct brw_context *brw)
 {