On 22 January 2014 09:16, Connor Abbott <cwabbo...@gmail.com> wrote:
> +class ir_loop_jumps_visitor : public ir_hierarchical_visitor
> +{
> +public:
> + ir_loop_jumps_visitor();
> + ~ir_loop_jumps_visitor();
> +
> + virtual ir_visitor_status visit_enter(ir_loop *);
> + virtual ir_visitor_status visit(ir_loop_jump *);
> +
> + ir_loop_jumps *get_loop_jumps(ir_loop *ir);
> +
> +private:
> + ir_loop *outer_loop;
>
As with outer_if in the previous patch, I think we could save some
unnecessary hashtable lookups if we simply stored the ir_loop_jumps * here
instead of storing the ir_loop *. But I don't feel terribly strongly about
it.
Either way, the patch is:
Reviewed-by: Paul Berry <stereotype...@gmail.com>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
