[PATCH] D125611: DWARFVerifier: Change vector of IntervalMap to vector of unique_ptrs

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 24 14:48:41 PDT 2022


dblaikie added a comment.

Looking at D125899 <https://reviews.llvm.org/D125899> in more detail I'm OK with deciding that adding good quality copy/move semantics to IntervalMap is a bigger project than I'm up for right now & so let's go with this direction instead (sorry for the back and forth - thanks for understanding).

If you want to submit a follow-up patch to disable copy/move (construction and assignment) for IntervalMap, that'd be great, otherwise I can do that once this lands.



================
Comment at: llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp:426
+        Sections[Col] = std::make_unique<MapType>(Alloc);
       auto &M = Sections[Col];
+      auto I = M->find(SC.Offset);
----------------
Perhaps this could be:
```
auto &M = *Sections[Col];
```
To keep the rest of the code simpler/without having to add a few `->`?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125611/new/

https://reviews.llvm.org/D125611



More information about the llvm-commits mailing list