<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 29, 2014 at 10:33 AM, Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":16m" class="a3s" style="overflow:hidden">Given that this is all still experimental, I'm not at the stage of<br>

profiling this.  I'd used `SmallVector` without really thinking about<br>
it, and then when I noticed the error I replaced it with something<br>
strictly better.  If you feel strongly enough that this needs to be<br>
reverted immediately (well, switch to `std::vector`), let me know.<br>
<br>
Otherwise, it's likely to change anyway, since shuffles don't need<br>
nearly this much memory (see my way-too-long-to-read llvmdev post).</div></blockquote></div><br>If this code isn't likely to stay, and can't even be profiled, *please* stop tuning it and introducing needless abstractions. What is it even doing in tree??? =[[[[</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">I really dislike committing "throw-away" code. It wastes reviewers time (see this thread), it wastes contributors time (see chapuni's fixes to your patches to keep the bots green) and it makes it really hard to see the actual code intended.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">Either pull this out of tree, or treat it as real code and respond to the code review? =[</div></div>