Samuel Iglesias Gonsálvez <sigles...@igalia.com> writes: > Hello, > > As mentioned in the patch series that implemented Ivybridge support > ARB_gpu_shader_fp64 [0], the only missing feature in that series was > register spilling of 64-bit data and, because of that, about ~39 fp64 > piglit tests failed to spill registers. > > This new patch series implement register spilling of 64-bit data for > IVB and splitted DF instructions on vec4 backend in general. > Unfortunately, this doesn't make the previous failed tests to pass :-( >
I think the reason why the tests keep failing to register allocate on you is because you aren't setting up the vec4_instruction::size_written fields correctly throughout this series. It's in *byte* units but you seem to be providing the number in GRF units. For that reason the liveness analysis pass will think most of the unspilled data you read is not fully initialized by the scratch reads, and incorrectly extend their live ranges all the way up to the program entry point, so spilling of DF variables will hugely *increase* register pressure instead of reducing it... > Nevertheless, I think this is still useful to have it in place. The > implementation uses 1-OWord block write/read messages by reusing the > existing implementation. Thanks to that, we can write/read valid dvecN > data to/from scratch memory even under non-uniform control flow. > > If you want to test the branch: > > $ git clone -b fp64-ivb-vec4-spilling \ > https://github.com/Igalia/mesa.git > > Thanks, > > Sam > > [0] https://lists.freedesktop.org/archives/mesa-dev/2017-March/148646.html > > Samuel Iglesias Gonsálvez (6): > i965/eu: add support for 1-OWord Block Read/Write messages > i965/vec4/generator: use 1-Oword Block Read/Write messages for DF > scratch writes/reads > i965/generator: use MRF when sending 1-OWord read messages for DF > scratch reads on IVB > i965/vec4: add support for doing DF register spilling on IVB > i965/vec4: fix resolve reladdr case on DF scratch read/write on IVB > i965/vec4: allow partial DF register spilling > > src/intel/compiler/brw_eu.h | 18 ++-- > src/intel/compiler/brw_eu_defines.h | 2 + > src/intel/compiler/brw_eu_emit.c | 42 +++++++-- > src/intel/compiler/brw_fs_generator.cpp | 5 +- > src/intel/compiler/brw_shader.cpp | 5 + > src/intel/compiler/brw_vec4.cpp | 10 ++ > src/intel/compiler/brw_vec4.h | 17 +++- > src/intel/compiler/brw_vec4_generator.cpp | 136 > +++++++++++++++++++++++++++ > src/intel/compiler/brw_vec4_reg_allocate.cpp | 42 +++++++-- > src/intel/compiler/brw_vec4_visitor.cpp | 125 ++++++++++++++++++++++-- > 10 files changed, 361 insertions(+), 41 deletions(-) > > -- > 2.11.0
signature.asc
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
