On Tuesday, May 3, 2016 9:26:13 PM PDT Francisco Jerez wrote: > Instead of using the LOAD_PAYLOAD instruction (emitted through the > emit_transpose() helper that is no longer useful and this commit > removes) which had to be marked force_writemask_all in some cases, > emit a series of moves to apply proper channel enable signals to the > destination. Until now lower_simd_width() had mainly been used to > lower things that invariably had a basic block-local temporary as > destination so it didn't seem like a big deal, but I found it to be > the reason for several Piglit regressions in my SIMD32 branch and > Igalia discovered the same issue independently while working on FP64 > support. > --- > This is taken from the following WIP series: > https://cgit.freedesktop.org/~currojerez/mesa/log/?h=i965-late-simd-lowering > > See also: > https://lists.freedesktop.org/archives/mesa-dev/2016-May/115596.html > > src/mesa/drivers/dri/i965/brw_fs.cpp | 58 ++++++++++ +------------------------- > 1 file changed, 18 insertions(+), 40 deletions(-)
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 https://lists.freedesktop.org/mailman/listinfo/mesa-dev