Thanks Matt, Eric and Kenneth for the comments, I did rework my set to look a bit different. I noticed Matt has partially overlapping set of patches about dead code elimination and as for my set I did not have access this week to Ironlake machine thus calling this 'RFC'y set. Kenneth, I think pedantic is good :) I hope you don't mind I took your versions of commit messages almost directly as I probably could not come up with anything better.
The biggest differences are that I moved the accumulator flag to backend_instruction class which is accessible from both vec4 and fs. This also allow to see the flag in scheduler easily via backend_instruction::has_side_effects() as well as remove the switch/cases for ADDC/SUBB/MACH from dead code elimination. Also the fs side should be working now, this was anyhow almost cut'n'paste type thing. As for correctly scheduling instructions has_side_effects returning true when writes_accumulator is true seems to make everything ok without need to touch inside calculate_deps() I did run these new LRP pieces on my Ivybridge machine and with Piglit I don't see any regressions on glsl tests. Next week I probably can try with Ironlake. /Juha-Pekka Juha-Pekka Heikkila (5): i965: Add writes_accumulator flag i965/vec4: Add support for the MAC instruction. i965/vec4: Change vec4_visitor::emit_lrp to use MAC for gen<6 i965/fs: Add support for the MAC instruction. i965/fs: Change fs_visitor::emit_lrp to use MAC for gen<6 src/mesa/drivers/dri/i965/brw_eu.h | 1 + src/mesa/drivers/dri/i965/brw_fs.cpp | 35 +++++++++---------- src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 10 +++--- src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 11 +++--- src/mesa/drivers/dri/i965/brw_shader.cpp | 3 ++ src/mesa/drivers/dri/i965/brw_shader.h | 1 + src/mesa/drivers/dri/i965/brw_vec4.cpp | 15 +------- src/mesa/drivers/dri/i965/brw_vec4_generator.cpp | 10 +++--- src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 44 +++++++++++++----------- 9 files changed, 58 insertions(+), 72 deletions(-) -- 1.8.1.2 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
