Re: [Mesa-dev] [PATCH] nir: Walk blocks in source code order in lower_vars_to_ssa.

2016-08-25 Thread Jason Ekstrand
On Wed, Aug 24, 2016 at 9:04 PM, Matt Turner wrote: > Prior to this commit rename_variables_block() is recursively called, > performing a depth-first traversal of the control flow graph. The > function uses a non-trivial amount of stack space for local variables, > which puts us in danger of smas

Re: [Mesa-dev] [PATCH] nir: Walk blocks in source code order in lower_vars_to_ssa.

2016-08-25 Thread Connor Abbott
On Thu, Aug 25, 2016 at 2:26 PM, Matt Turner wrote: > On Wed, Aug 24, 2016 at 11:25 PM, Connor Abbott wrote: >> On Thu, Aug 25, 2016 at 12:04 AM, Matt Turner wrote: >>> Prior to this commit rename_variables_block() is recursively called, >>> performing a depth-first traversal of the control flow

Re: [Mesa-dev] [PATCH] nir: Walk blocks in source code order in lower_vars_to_ssa.

2016-08-25 Thread Matt Turner
On Wed, Aug 24, 2016 at 11:25 PM, Connor Abbott wrote: > On Thu, Aug 25, 2016 at 12:04 AM, Matt Turner wrote: >> Prior to this commit rename_variables_block() is recursively called, >> performing a depth-first traversal of the control flow graph. The >> function uses a non-trivial amount of stack

Re: [Mesa-dev] [PATCH] nir: Walk blocks in source code order in lower_vars_to_ssa.

2016-08-24 Thread Connor Abbott
On Thu, Aug 25, 2016 at 12:04 AM, Matt Turner wrote: > Prior to this commit rename_variables_block() is recursively called, > performing a depth-first traversal of the control flow graph. The > function uses a non-trivial amount of stack space for local variables, > which puts us in danger of smas