[llvm] [NFC][AsmPrinter] Refactor FrameIndexExprs as a std::set (PR #66433)

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 15 22:08:42 PDT 2023


================
@@ -158,7 +153,7 @@ class Multi {
 };
 /// Single location defined by (potentially multiple) MMI entries.
 struct MMI {
-  mutable SmallVector<FrameIndexExpr, 1> FrameIndexExprs;
+  std::set<FrameIndexExpr> FrameIndexExprs;
----------------
dwblaikie wrote:

Another possibilty would be a `SetVector`? (though then it's insertion ordered, rather than `<` ordered - not sure if that's sufficiently stable for the needs? perhaps not)

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


More information about the llvm-commits mailing list