<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On Nov 18, 2010, at 2:19 PM, Chris Lattner wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div>On Nov 18, 2010, at 1:50 PM, Bill Wendling wrote:<br><br><blockquote type="cite">@@ -350,11 +343,10 @@<br></blockquote><blockquote type="cite"> Kind = SPRRegisterList;<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite"> ARMOperand *Op = new ARMOperand(Kind);<br></blockquote><blockquote type="cite">- Op->RegList.Registers = new SmallVector<unsigned, 32>();<br></blockquote><blockquote type="cite"> for (SmallVectorImpl<std::pair<unsigned, SMLoc> >::const_iterator<br></blockquote><blockquote type="cite"> I = Regs.begin(), E = Regs.end(); I != E; ++I)<br></blockquote><blockquote type="cite">- Op->RegList.Registers->push_back(I->first);<br></blockquote><blockquote type="cite">- std::sort(Op->RegList.Registers->begin(), Op->RegList.Registers->end());<br></blockquote><blockquote type="cite">+ Op->Registers.push_back(I->first);<br></blockquote><blockquote type="cite">+ std::sort(Op->Registers.begin(), Op->Registers.end());<br></blockquote><br>Hey bill,<br><br>I see that it's not new in this patch, but please use array_pod_sort in STLExtras.h if the reglist is known-contiguous.<br><br></div></blockquote>Donéd.</div><div><br></div><div>-bw</div><div><br></div><br></body></html>