On Monday, September 01, 2014 09:44:26 AM Jordan Justen wrote: > Signed-off-by: Jordan Justen <jordan.l.jus...@intel.com> > --- > src/mesa/drivers/dri/i965/Makefile.sources | 2 +- > src/mesa/drivers/dri/i965/brw_blorp_blit_eu.h | 2 +- > src/mesa/drivers/dri/i965/brw_fs.cpp | 4 +- > src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 2009 ------------------- > src/mesa/drivers/dri/i965/brw_scalar.h | 20 +- > src/mesa/drivers/dri/i965/brw_scalar_generator.cpp | 2011 > ++++++++++++++++++++ > 6 files changed, 2025 insertions(+), 2023 deletions(-) > delete mode 100644 src/mesa/drivers/dri/i965/brw_fs_generator.cpp > create mode 100644 src/mesa/drivers/dri/i965/brw_scalar_generator.cpp
I think this patch makes a lot of sense. The generator is going to be used for FS, VS, HS, DS, and compute programs, and having all of that in a single class is not a crazy plan - it's pretty much dumb/straightforward code generation without a lot of specific stage knowledge. Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev