This is a re-send of the series I did a week or two ago to remove the width field from the fs_reg class. I really didn't want to do a re-send but there have been enough fixes since then that I thought it was worth re-sending. Most of these patches have already been reviewed but not all.
02: New. Needs to be reviewed by someone familiar with SKL 03: Needs re-review. This one is affected by 02. 05: Needs re-review. This one went through a lot of changes to actually get it right. It should be the way we want now. 08: New. It's just moving code around so it should be trivial. 09: New. This is a complete replacement of patch 07 from the previous series. Cc: Topi Pohjolainen <topi.pohjolai...@intel.com> Cc: Iago Toral Quiroga <ito...@igalia.com> Cc: Francisco Jerez <curroje...@riseup.net> Cc: Neil Roberts <n...@linux.intel.com> Jason Ekstrand (19): i965/fs: Use a switch statement in fs_inst::regs_read() i965/fs: Actually set/use the mlen for gen7 uniform pull constant loads i965/fs: Fix fs_inst::regs_read() for uniform pull constant loads i965/fs: Report the right value in fs_inst::regs_read() for PIXEL_X/Y i965/fs: Explicitly set the exec_size on the add(32) in interpolation setup i965/fs: Set the builder group for emitting FB-write stencil/AA alpha i965/blorp: Explicitly set execution sizes for new'd instructions i965/fs: Move offset(fs_reg, unsigned) to brw_fs.h i965/fs: Add a builder argument to offset() i965/fs: Make better use of the builder in shader_time i965/fs: Remove fs_inst constructors that don't take an explicit exec_size i965/fs: Use exec_size for determining regs read/written and partial writes i965/fs_builder: Use the dispatch width for setting exec sizes i965/fs: Remove exec_size guessing from fs_inst::init() i965/fs: Use the builder dispatch width instead of dst.width for pull constants i965/fs: Use the builder dispatch_width for computing register offsets i965/fs: Use exec_size instead of dst.width for computing component size i965/fs_generator: Use inst->exec_size for determining hardware reg widths i965/fs: Remove the width field from fs_reg src/mesa/drivers/dri/i965/brw_blorp_blit_eu.cpp | 9 +- src/mesa/drivers/dri/i965/brw_fs.cpp | 266 ++++++++------------- src/mesa/drivers/dri/i965/brw_fs.h | 21 ++ src/mesa/drivers/dri/i965/brw_fs_builder.h | 37 ++- .../drivers/dri/i965/brw_fs_copy_propagation.cpp | 4 - src/mesa/drivers/dri/i965/brw_fs_cse.cpp | 10 +- src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 23 +- src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 64 ++--- src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp | 4 +- .../drivers/dri/i965/brw_fs_register_coalesce.cpp | 3 +- src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 183 +++++++------- src/mesa/drivers/dri/i965/brw_ir_fs.h | 45 +--- .../drivers/dri/i965/brw_schedule_instructions.cpp | 4 +- 13 files changed, 287 insertions(+), 386 deletions(-) -- 2.4.3 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev