[PATCH] D58385: [tools] Rewrite tests for symbol remapping to FileCheck

Thu Feb 28 08:41:29 PST 2019

Hahnfeld added a comment.

In D58385#1410560 <https://reviews.llvm.org/D58385#1410560>, @Hahnfeld wrote:

> Another approach would be to sort the output and make it independent from `reverse-iteration`. See D58631 <https://reviews.llvm.org/D58631> for how this was done for ThinLTO. I'm fine with whatever unbreaks the tests, but I'm not familiar with the code so "fixing" the tests and getting review on this was easier for me.

I've uploaded D58787 <https://reviews.llvm.org/D58787> which replaces a `SmallDenseMap` with `std::map` and fixes the test.

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58385/new/

https://reviews.llvm.org/D58385