<div dir="ltr">On 22 January 2014 09:16, Connor Abbott <span dir="ltr"><<a href="mailto:cwabbott0@gmail.com" target="_blank">cwabbott0@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+class ir_loop_jumps_visitor : public ir_hierarchical_visitor<br>
+{<br>
+public:<br>
+   ir_loop_jumps_visitor();<br>
+   ~ir_loop_jumps_visitor();<br>
+<br>
+   virtual ir_visitor_status visit_enter(ir_loop *);<br>
+   virtual ir_visitor_status visit(ir_loop_jump *);<br>
+<br>
+   ir_loop_jumps *get_loop_jumps(ir_loop *ir);<br>
+<br>
+private:<br>
+   ir_loop *outer_loop;<br></blockquote><div><br></div><div>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.<br>
<br></div><div>Either way, the patch is:<br><br>Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br></div></div></div></div>