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

Felipe de Azevedo Piovezan via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 16 06:04:14 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;
----------------
felipepiovezan wrote:

IIUC the DWARF emitter also wants the fragments ordered by offset? I'm not sure what would happen if we were to emit DWARF fragment expressions out of order 🤔 

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


More information about the llvm-commits mailing list