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

via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 1 18:35:46 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;
----------------
llvmssh wrote:

yes, this may cause nondeterminism problem.

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


More information about the llvm-commits mailing list