I think we now have adl_begin/adl_end helpers in the llvm namespace that should do the right thigh (which is that the call should be made like "using std::begin; ...begin(X)..." The same way swap is meant to be called (this allows a default implementation in the std namespace and allows for types to implement their own custom implementation in their own namespace to be found via ADL))<br><br><div class="gmail_quote"><div dir="ltr">On Tue., 26 Jun. 2018, 10:19 am Michael Kruse via Phabricator, <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Meinersbur created this revision.<br>
Meinersbur added reviewers: dblaikie, grosbach, aaron.ballman, ruiu.<br>
<br>
The instantiation of the drop_begin function template usually fails because the functions begin() and end() do not exist. Only when using on a container from the std namespace (or `llvm::iterator_range`s of something derived from `std::iterator`), they are matched to std::begin() and std::end() due to Koenig-lookup.<br>
<br>
Explicitly use std::begin and std::end to make drop_begin applicable to anything iterable (including C-style arrays).<br>
<br>
A solution for general `llvm::iterator_range`s was already tried in r244620, but got reverted in r244621 due to MSVC not liking it.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D48598" rel="noreferrer" target="_blank">https://reviews.llvm.org/D48598</a><br>
<br>
Files:<br>
  include/llvm/ADT/iterator_range.h<br>
  unittests/ADT/IteratorTest.cpp<br>
<br>
<br>
Index: unittests/ADT/IteratorTest.cpp<br>
===================================================================<br>
--- unittests/ADT/IteratorTest.cpp<br>
+++ unittests/ADT/IteratorTest.cpp<br>
@@ -373,4 +373,17 @@<br>
   EXPECT_EQ(std::distance(v2.begin(), v2.end()), size(v2));<br>
 }<br>
<br>
+TEST(IteratorRangeTest, DropBegin) {<br>
+  SmallVector<int, 5> vec{0, 1, 2, 3, 4};<br>
+<br>
+  for (int n = 0; n < 5; ++n) {<br>
+    int i = n;<br>
+    for (auto &v : drop_begin(vec, n)) {<br>
+      EXPECT_EQ(v, i);<br>
+      i += 1;<br>
+    }<br>
+    EXPECT_EQ(i, 5);<br>
+  }<br>
+}<br>
+<br>
 } // anonymous namespace<br>
Index: include/llvm/ADT/iterator_range.h<br>
===================================================================<br>
--- include/llvm/ADT/iterator_range.h<br>
+++ include/llvm/ADT/iterator_range.h<br>
@@ -59,9 +59,10 @@<br>
   return iterator_range<T>(std::move(p.first), std::move(p.second));<br>
 }<br>
<br>
-template<typename T><br>
-iterator_range<decltype(begin(std::declval<T>()))> drop_begin(T &&t, int n) {<br>
-  return make_range(std::next(begin(t), n), end(t));<br>
+template <typename T><br>
+iterator_range<decltype(std::begin(std::declval<T>()))> drop_begin(T &&t,<br>
+                                                                   int n) {<br>
+  return make_range(std::next(std::begin(t), n), std::end(t));<br>
 }<br>
 }<br>
<br>
<br>
<br>
</blockquote></div>