<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 16 September 2016 at 16:07, George Rimar via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




<div dir="ltr" style="font-size:12pt;color:#000000;background-color:#ffffff;font-family:Calibri,Arial,Helvetica,sans-serif">
<p><br>
</p>
<div style="color:rgb(33,33,33)">
<div>
<div dir="ltr">
<div class="gmail_extra"><span class="">
<div class="gmail_quote"><span style="font-size:12pt">>I didn't read the entire patch yet, but why did you use <list>? It is a doubly-linked list and slower than the vector, so you don't >want to use it unless you really want doubly-linked lists.</span>
</div>
<div class="gmail_quote"><span style="font-size:12pt"><br>
</span></div>
</span><div class="gmail_quote"><span style="font-size:12pt">Please look at my comment about this at review page, it is a workaround for compiler bug. No difference with list here in plan of speed because 99% of time this container has single element.</span></div></div></div></div></div></div></blockquote><div><br></div><div><br></div><div>Can you add a comment saying that list is used to avoid a compiler (libc++?) bug? std::vector is far more common, so it is nice to justify why a list is used.</div><div><br></div><div><br></div><div>Thanks,</div><div>Rafael</div><div><br></div><div> </div></div></div></div>