[PATCH] D124350: [llvm-gsymutil][NFC] refactor AddressRange&AddresRanges structures.

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: llvm/lib/Support/AddressRanges.cpp:24
+  if (It != It2) {
+    Range = {Range.start(), std::max(Range.end(), It2[-1].end())};
+    It = Ranges.erase(It, It2);
----------------
@avl I'm seeing static analyzer warnings about using signed (negative) values into It2[] - maybe add a static cast?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124350