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

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


================
@@ -28,11 +29,11 @@ class RecordStreamer : public MCStreamer {
 
 private:
   const Module &M;
-  StringMap<State> Symbols;
+  MapVector<StringRef, State> Symbols;
----------------
vitalybuka wrote:

unordered does not mean nondeterminism.

So I believe change in this line is unnecessary.

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


More information about the llvm-commits mailing list