[PATCH] D93992: [STLExtras] Use return type from operator* of the wrapped iter.

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 7 11:12:06 PST 2021


dblaikie added inline comments.


================
Comment at: llvm/unittests/ADT/STLExtrasTest.cpp:458
+struct CustomPointerIterator
+    : public std::iterator<std::forward_iterator_tag, int *> {
+  std::list<int>::iterator Iter;
----------------
std::iterator is deprecated - you could try using llvm::iterator_adaptor_base, it'll also simplify the implementation by providing default implementations of several operations required for the iterator concepts.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93992



More information about the llvm-commits mailing list