[llvm] Fix non-determinism in debuginfo (PR #68332)

Orlando Cazalet-Hyams via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 6 05:41:02 PDT 2023


================
@@ -263,7 +265,8 @@ struct VarRecord {
 /// TODO: Backing storage shouldn't be limited to allocas only. Some local
 /// variables have their storage allocated by the calling function (addresses
 /// passed in with sret & byval parameters).
-using StorageToVarsMap = DenseMap<const AllocaInst *, SmallSet<VarRecord, 2>>;
+using StorageToVarsMap =
+    DenseMap<const AllocaInst *, SmallSetVector<VarRecord, 2>>;
----------------
OCHyams wrote:

I don't think a comment would hurt, but I also agree that in this case the type does communicate its purpose here.

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


More information about the llvm-commits mailing list