[PATCH] D93106: [dsymutil][DWARFLinker][NFC] make AddressManager not depending on the order of checks for relocations.

[Alexey Lapshin via Phabricator via llvm-commits]

avl added inline comments.


================
Comment at: llvm/tools/dsymutil/DwarfLinkerForBinary.h:89
+    bool
+    enumerateRelocations(uint64_t StartPos, uint64_t EndPos,
+                         function_ref<bool(const ValidReloc &rel)> RelHandler);
----------------
JDevlieghere wrote:
> Can we return a list of relocations as an alternative to using a callback? 
I like that callback approach since it allows not to move/copy/resize/fill containers. But if you think it would be better to return list of found relocations then I would change it accordingly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93106