Re: [Mesa-dev] [PATCH] nv50/ir: only stick one preret per function

2016-10-09 Thread Ilia Mirkin
On Sun, Oct 9, 2016 at 7:53 AM, Samuel Pitoiset wrote: > > > On 10/09/2016 06:12 AM, Ilia Mirkin wrote: >> >> A function with multiple returns would have had multiple preret settings >> at the top of the function. While this is unlikely to have caused issues >> since we don't use funcitons in earn

Re: [Mesa-dev] [PATCH] nv50/ir: only stick one preret per function

2016-10-09 Thread Samuel Pitoiset
On 10/09/2016 06:12 AM, Ilia Mirkin wrote: A function with multiple returns would have had multiple preret settings at the top of the function. While this is unlikely to have caused issues since we don't use funcitons in earnest, it could have in some cases s/funcitons/functions/ :) This loo

[Mesa-dev] [PATCH] nv50/ir: only stick one preret per function

2016-10-08 Thread Ilia Mirkin
A function with multiple returns would have had multiple preret settings at the top of the function. While this is unlikely to have caused issues since we don't use funcitons in earnest, it could have in some cases overflowed the call stack, in case a function had a lot of early returns. Signed-of