[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