Looks good to me.
Bryan
On 10/18/2011 05:20 PM, Kenneth Graunke wrote:
> Setting MaxIfDepth to UINT_MAX effectively means "don't lower anything."
>
> Explicitly checking for this common case allows us to avoid walking the
> IR, computing nesting levels, and so on.
>
> Signed-off-by: Kenneth Graunke <kenn...@whitecape.org>
> Cc: Bryan Cain <bryanca...@gmail.com>
> Cc: Ian Romanick <i...@freedesktop.org>
> ---
> src/glsl/lower_if_to_cond_assign.cpp | 3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/src/glsl/lower_if_to_cond_assign.cpp
> b/src/glsl/lower_if_to_cond_assign.cpp
> index 7b89a15..2c5d561 100644
> --- a/src/glsl/lower_if_to_cond_assign.cpp
> +++ b/src/glsl/lower_if_to_cond_assign.cpp
> @@ -79,6 +79,9 @@ public:
> bool
> lower_if_to_cond_assign(exec_list *instructions, unsigned max_depth)
> {
> + if (max_depth == UINT_MAX)
> + return false;
> +
> ir_if_to_cond_assign_visitor v(max_depth);
>
> visit_list_elements(&v, instructions);
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
