[PATCH] D64997: [ADT] Add public iterator and const_iterator types to iterator_range

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 19 09:29:06 PDT 2019


MaskRay added inline comments.


================
Comment at: llvm/include/llvm/ADT/iterator_range.h:39
 
 public:
   //TODO: Add SFINAE to test that the Container's iterators match the range's
----------------
You can insert

  using iterator = IteratorT;
  using const_iterator = IteratorT;

here to avoid adding two access modifiers.


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

https://reviews.llvm.org/D64997





More information about the llvm-commits mailing list