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

2015-04-10 Thread Matt Turner
On Fri, Apr 10, 2015 at 12:52 PM, 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 actua

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

2015-04-10 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