[llvm-commits] [llvm] r173563 - fix use of std::std. it's ordered set.

[reed kotler]

On 01/25/2013 11:18 PM, David Blaikie wrote:
> On Fri, Jan 25, 2013 at 10:58 PM, Reed Kotler <rkotler at mips.com> wrote:
>> Author: rkotler
>> Date: Sat Jan 26 00:58:35 2013
>> New Revision: 173563
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=173563&view=rev
>> Log:
>> fix use of std::std. it's ordered set.
> Any chance of a test case?
I have a test case in mind for this optimization and hopefully if I 
revert this change locally, the code will fail this test.
>>
>> Modified:
>>      llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp
>>
>> Modified: llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp
>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp?rev=173563&r1=173562&r2=173563&view=diff
>> ==============================================================================
>> --- llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp (original)
>> +++ llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp Sat Jan 26 00:58:35 2013
>> @@ -206,14 +206,14 @@ const char *MipsTargetLowering::getTarge
>>   }
>>
>>   namespace {
>> -  struct eqstr {
>> +  struct ltstr {
>>       bool operator()(const char *s1, const char *s2) const
>>       {
>> -      return strcmp(s1, s2) == 0;
>> +      return strcmp(s1, s2) < 0;
>>       }
>>     };
>>
>> -  std::set<const char*, eqstr> noHelperNeeded;
>> +  std::set<const char*, ltstr> noHelperNeeded;
>>
>>     const char* addToNoHelperNeeded(const char* s) {
>>       noHelperNeeded.insert(s);
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits