<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Jul 18, 2014 at 2:30 AM, Tim Northover <span dir="ltr"><<a href="mailto:t.p.northover@gmail.com" target="_blank">t.p.northover@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">> An alternative implementation to deleting nodes would be to just add a<br>
> "llvm.loop.unroll.enable false" metadata node after unrolling.  It's a<br>
> little funny because you could then have, say, both "llvm.loop.unroll.enable<br>
> false" and "llvm.loop.unroll.enable true" attached to a loop which is a bit<br>
> funny.  Yet another alternative is to make up some new metadata node type<br>
> like "llvm.loop.unroll.already_unrolled" and add it to the loop.<br>
<br>
</div>What about dividing the unroll hint by the unroll count and dropping<br>
it if that's <= 1?<br></blockquote><div><br></div><div>This is a destructive change very similar to Mark's original in the patch. My issue with destructive changes is that they may make things more difficult to debug by "deleting the audit trail", so to say. But this isn't a strong objection, so if others feel it's appropriate in this case, I'm fine with that.</div>

<div><br></div><div>Eli</div><div><br></div><div> </div></div></div></div>