i965/fs: Remove special casing of framebuffer writes in scheduler code.
authorFrancisco Jerez <currojerez@riseup.net>
Thu, 21 Jul 2016 23:56:05 +0000 (16:56 -0700)
committerFrancisco Jerez <currojerez@riseup.net>
Fri, 26 Aug 2016 01:36:09 +0000 (18:36 -0700)
commitbe12a1f36efcdd4628f199d4e11b01cc06787e8a
treef391d55260f2642bc7f3b0240f3e01df6e6c42ca
parent3daa0fae4b39a271f50f473edbe44712b6c8f040
i965/fs: Remove special casing of framebuffer writes in scheduler code.

The reason why it was safe for the scheduler to ignore the side
effects of framebuffer write instructions was that its side effects
couldn't have had any influence on any other instruction in the
program, because we weren't doing framebuffer reads, and framebuffer
writes were always non-overlapping.  We need actual memory dependency
analysis in order to determine whether a side-effectful instruction
can be reordered with respect to other instructions in the program.

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp