Re: [Mesa-dev] [PATCH] i965/fs: Disable CSE optimization for untyped & typed surface reads

2015-10-22 Thread Jordan Justen
On 2015-10-22 00:06:37, Iago Toral wrote: > On Wed, 2015-10-21 at 23:24 -0700, Jordan Justen wrote: > > On 2015-10-20 00:43:13, Iago Toral wrote: > > > On Tue, 2015-10-20 at 00:12 -0700, Jordan Justen wrote: > > > > An untyped surface read is volatile because it might be affected by a > > > > write

Re: [Mesa-dev] [PATCH] i965/fs: Disable CSE optimization for untyped & typed surface reads

2015-10-22 Thread Iago Toral
On Wed, 2015-10-21 at 23:24 -0700, Jordan Justen wrote: > On 2015-10-20 00:43:13, Iago Toral wrote: > > On Tue, 2015-10-20 at 00:12 -0700, Jordan Justen wrote: > > > An untyped surface read is volatile because it might be affected by a > > > write. > > > > > > In the ES31-CTS.compute_shader.resour

Re: [Mesa-dev] [PATCH] i965/fs: Disable CSE optimization for untyped & typed surface reads

2015-10-21 Thread Jordan Justen
On 2015-10-20 00:43:13, Iago Toral wrote: > On Tue, 2015-10-20 at 00:12 -0700, Jordan Justen wrote: > > An untyped surface read is volatile because it might be affected by a > > write. > > > > In the ES31-CTS.compute_shader.resources-max test, two back to back > > read/modify/writes of an SSBO var

Re: [Mesa-dev] [PATCH] i965/fs: Disable CSE optimization for untyped & typed surface reads

2015-10-20 Thread Iago Toral
On Tue, 2015-10-20 at 00:12 -0700, Jordan Justen wrote: > An untyped surface read is volatile because it might be affected by a > write. > > In the ES31-CTS.compute_shader.resources-max test, two back to back > read/modify/writes of an SSBO variable looked something like this: > > r1 = untyped_

[Mesa-dev] [PATCH] i965/fs: Disable CSE optimization for untyped & typed surface reads

2015-10-20 Thread Jordan Justen
An untyped surface read is volatile because it might be affected by a write. In the ES31-CTS.compute_shader.resources-max test, two back to back read/modify/writes of an SSBO variable looked something like this: r1 = untyped_surface_read(ssbo_float) r2 = r1 + 1 untyped_surface_write(ssbo_fl