[llvm] Use MapVector to fix lld thinLTO bep issue. (PR #117551)

Vitaly Buka via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 1 18:32:22 PST 2024


================
@@ -28,11 +29,11 @@ class RecordStreamer : public MCStreamer {
 
 private:
   const Module &M;
-  StringMap<State> Symbols;
+  MapVector<StringRef, State> Symbols;
   // Map of aliases created by .symver directives, saved so we can update
   // their symbol binding after parsing complete. This maps from each
   // aliasee to its list of aliases.
-  DenseMap<const MCSymbol *, std::vector<StringRef>> SymverAliasMap;
+  MapVector<const MCSymbol *, std::vector<StringRef>> SymverAliasMap;
----------------
vitalybuka wrote:

This one I see that it was non deterministic `DenseMap<const MCSymbol *`

https://github.com/llvm/llvm-project/pull/117551


More information about the llvm-commits mailing list