[PATCH] D57618: [ADT] Add a fallible_iterator wrapper.

[Alexander Shaposhnikov via Phabricator via llvm-commits]

alexshap added inline comments.


================
Comment at: include/llvm/ADT/fallible_iterator.h:19
+namespace llvm {
+
+/// A wrapper class for fallible iterators.
----------------
general comment: @lhames , have you considered using CRTP instead of introducing this wrapper/ forwarding etc ?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57618