[llvm-commits] [llvm] r82040 - /llvm/trunk/lib/CodeGen/MachineModuleInfoImpls.cpp

[Chris Lattner]

On Sep 17, 2009, at 2:16 PM, Mike Stump wrote:

> On Sep 16, 2009, at 10:17 PM, Nick Lewycky wrote:
>> Benjamin Kramer wrote:
>>> Author: d0k
>>> Date: Wed Sep 16 06:43:12 2009
>>> New Revision: 82040
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=82040&view=rev
>>> Log:
>>> Don't sort the vector when it is empty. This should fix some
>>> expensive checking
>>> failures.
>>
>> qsort? Why are we using qsort?
>
> Further hate for qsort, we want a stable sort, qsort isn't, so that we
> can have determinism and cross host comparisons of codegen...

Stability doesn't matter, there is only one entry for each key.

-Chris