Re: [Mesa-dev] [PATCH 1/2] i965/fs: Don't use backend_visitor::instructions after creating the CFG.

2015-03-02 Thread Matt Turner
On Fri, Jan 23, 2015 at 1:38 PM, Matt Turner matts...@gmail.com wrote: On Sat, Jan 17, 2015 at 12:07 AM, Kenneth Graunke kenn...@whitecape.org wrote: With an updated commit message and Piglit passing (I'll test and let you know), Reviewed-by: Kenneth Graunke kenn...@whitecape.org Reminder

Re: [Mesa-dev] [PATCH 1/2] i965/fs: Don't use backend_visitor::instructions after creating the CFG.

2015-01-23 Thread Matt Turner
On Sat, Jan 17, 2015 at 12:07 AM, Kenneth Graunke kenn...@whitecape.org wrote: With an updated commit message and Piglit passing (I'll test and let you know), Reviewed-by: Kenneth Graunke kenn...@whitecape.org Reminder to piglit these the next time you power on your Gen4. :)

Re: [Mesa-dev] [PATCH 1/2] i965/fs: Don't use backend_visitor::instructions after creating the CFG.

2015-01-17 Thread Kenneth Graunke
On Friday, January 16, 2015 11:55:33 PM Matt Turner wrote: On Fri, Jan 16, 2015 at 11:45 PM, Kenneth Graunke kenn...@whitecape.org wrote: On Tuesday, January 13, 2015 03:35:57 PM Matt Turner wrote: This is a fix for a regression introduced in commit a9f8296d (i965/fs: Preserve the CFG in

Re: [Mesa-dev] [PATCH 1/2] i965/fs: Don't use backend_visitor::instructions after creating the CFG.

2015-01-16 Thread Kenneth Graunke
On Tuesday, January 13, 2015 03:35:57 PM Matt Turner wrote: This is a fix for a regression introduced in commit a9f8296d (i965/fs: Preserve the CFG in a few more places.). The errata this code works around is described in a comment before the function: [DevBW, DevCL] Errata: A

Re: [Mesa-dev] [PATCH 1/2] i965/fs: Don't use backend_visitor::instructions after creating the CFG.

2015-01-16 Thread Matt Turner
On Fri, Jan 16, 2015 at 11:45 PM, Kenneth Graunke kenn...@whitecape.org wrote: On Tuesday, January 13, 2015 03:35:57 PM Matt Turner wrote: This is a fix for a regression introduced in commit a9f8296d (i965/fs: Preserve the CFG in a few more places.). The errata this code works around is

[Mesa-dev] [PATCH 1/2] i965/fs: Don't use backend_visitor::instructions after creating the CFG.

2015-01-13 Thread Matt Turner
This is a fix for a regression introduced in commit a9f8296d (i965/fs: Preserve the CFG in a few more places.). The errata this code works around is described in a comment before the function: [DevBW, DevCL] Errata: A destination register from a send can not be used as a destination