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

Mandeep Singh Grang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 24 23:47:17 PDT 2017


mgrang added a comment.

In https://reviews.llvm.org/D39214#906176, @bkramer wrote:

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


The following 3 tests fail:

  LLVM :: CodeGen/ARM/available_externally.ll
  LLVM :: CodeGen/ARM/darwin-tls.ll
  LLVM :: CodeGen/ARM/indirect-hidden.ll


Repository:
  rL LLVM

https://reviews.llvm.org/D39214





More information about the llvm-commits mailing list