[PATCH] D145194: [BOLT] Remove dependency on StringMap iteration order

[Maksim Panchenko via Phabricator via llvm-commits]

maksfb added a comment.

In D145194#4167432 <https://reviews.llvm.org/D145194#4167432>, @Amir wrote:

> What's the reason behind `Counts`/`JumpTableCounts` being a `MapVector<StringRef, uint64_t` in `ReorderData::assignMemData` but `NamesToBranchesMapTy` being an `std::map<StringRef, FuncBranchData>`?

Different order of iteration over the elements. For `MapVector<>` it's the insertion order.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145194