Re: [Mesa-dev] [PATCH] i965/vec4: Don't overwrite op[1] when doing a UBO load.

2013-10-31 Thread Paul Berry
On 30 October 2013 17:45, Eric Anholt e...@anholt.net wrote: Prior to the GLSL CSE pass, all of our testing happened to have a freshly computed temporary in op[1], from the multiply by 16 to get a byte offset. As of CSE you'll get var_refs of a reused value when you've got multiple loads from

[Mesa-dev] [PATCH] i965/vec4: Don't overwrite op[1] when doing a UBO load.

2013-10-30 Thread Eric Anholt
Prior to the GLSL CSE pass, all of our testing happened to have a freshly computed temporary in op[1], from the multiply by 16 to get a byte offset. As of CSE you'll get var_refs of a reused value when you've got multiple loads from the same offset. Make a proper temporary for computing our