[PATCH] D64462: [Support] [WIP] Add UpdatableRange class

[Alex Brachet via Phabricator via llvm-commits]

abrachet added inline comments.


================
Comment at: llvm/include/llvm/ADT/UpdatableRange.h:122
+  void erase(SizeType Index) {
+    assert(Index < Mappings.size() && "Index larger than number of elements");
+    Mappings.erase(Mappings.begin() + Index);
----------------
Should this return Error?


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

https://reviews.llvm.org/D64462