[PATCH] D39214: [MachineModuleInfoImpls] Replace qsort with std::stable_sort

[Grang, Mandeep Singh via llvm-commits]

Oops, I mixed this with another one of my patches. Sorry. The tests 
which fail with this patch are:
     LLVM :: CodeGen/ARM/available_externally.ll
     LLVM :: CodeGen/ARM/darwin-tls.ll
     LLVM :: CodeGen/ARM/indirect-hidden.ll

On 10/24/2017 11:41 PM, Grang, Mandeep Singh via llvm-commits wrote:
>
> Benjamin,
>
> I have listed the failing tests in my patch 
> https://reviews.llvm.org/D39245. Listing them again here:
>
> Clang :: Misc/diag-template-diffing.cpp
> Clang :: Misc/error-limit-multiple-notes.cpp
> Clang :: OpenMP/task_firstprivate_codegen.cpp
> Clang :: OpenMP/task_private_codegen.cpp
> Clang :: OpenMP/taskloop_firstprivate_codegen.cpp
> Clang :: OpenMP/taskloop_lastprivate_codegen.cpp
> Clang :: OpenMP/taskloop_private_codegen.cpp
> Clang :: OpenMP/taskloop_simd_firstprivate_codegen.cpp
> Clang :: OpenMP/taskloop_simd_private_codegen.cpp
> LLVM :: CodeGen/AArch64/global-merge-group-by-use.ll
> LLVM :: Transforms/GVNHoist/hoist.ll
> LLVM :: Transforms/Util/PredicateInfo/testandor.ll
> LLVM :: tools/llvm-cov/hideUnexecutedSubviews.test
> LLVM :: tools/llvm-cov/showTemplateInstantiations.cpp
> LLVM :: tools/llvm-xray/X86/graph-zero-latency-calls.yaml
>
> --Mandeep
>
>
> On 10/24/2017 11:25 PM, Benjamin Kramer via Phabricator wrote:
>> bkramer added a comment.
>>
>> Can you point at the test cases that are failing? This would mean that there are two symbols with the same name, which seems like something that shouldn't happen.
>>
>>
>> Repository:
>>    rL LLVM
>>
>> https://reviews.llvm.org/D39214
>>
>>
>>
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171024/dc9c72b1/attachment.html>