Re: [Mesa-dev] [PATCH 1/2] i965: Create a has_side_effects for fs_inst

2015-04-01 Thread Matt Turner
On Wed, Apr 1, 2015 at 10:39 AM, Ben Widawsky wrote: > When an instruction has a side effect, it impacts the available options when > reordering an instruction. As the EOT flag is an implied write to the render > target in the FS, it can be considered a side effect. > > This patch shouldn't actual

Re: [Mesa-dev] [PATCH 1/2] i965: Create a has_side_effects for fs_inst

2015-04-01 Thread Kenneth Graunke
On Wednesday, April 01, 2015 10:39:16 AM Ben Widawsky wrote: > When an instruction has a side effect, it impacts the available options when > reordering an instruction. As the EOT flag is an implied write to the render > target in the FS, it can be considered a side effect. > > This patch shouldn'

[Mesa-dev] [PATCH 1/2] i965: Create a has_side_effects for fs_inst

2015-04-01 Thread Ben Widawsky
When an instruction has a side effect, it impacts the available options when reordering an instruction. As the EOT flag is an implied write to the render target in the FS, it can be considered a side effect. This patch shouldn't actually have any impact on the current code since the EOT flag impli