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

[Grang, Mandeep Singh via llvm-commits]

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
>
>
>

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