[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:23:37 PST 2024


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

I would not expect nondeterminism from Symbols
Order in hash table should be determined character in StringRefs and order they inserted.
Should be the same all the time?

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


More information about the llvm-commits mailing list