<div dir="ltr">How would this change fix that? Am I missing something? (the class and typename keyword in the template arg list are semantically identical)<br><br>What Eli said.</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jun 5, 2017 at 4:21 PM Francis Ricci 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">fjricci created this revision.<br>
<br>
This prevents the iterator override of assign from being selected in<br>
the case where two unsigned integers are used as arguments,<br>
which is a case that should be covered by `assign(size_type, const T &)`<br>
instead.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D33919" rel="noreferrer" target="_blank">https://reviews.llvm.org/D33919</a><br>
<br>
Files:<br>
  include/llvm/ADT/SmallVector.h<br>
<br>
<br>
Index: include/llvm/ADT/SmallVector.h<br>
===================================================================<br>
--- include/llvm/ADT/SmallVector.h<br>
+++ include/llvm/ADT/SmallVector.h<br>
@@ -426,7 +426,7 @@<br>
     std::uninitialized_fill(this->begin(), this->end(), Elt);<br>
   }<br>
<br>
-  template <typename in_iter> void assign(in_iter in_start, in_iter in_end) {<br>
+  template <class in_iter> void assign(in_iter in_start, in_iter in_end) {<br>
     clear();<br>
     append(in_start, in_end);<br>
   }<br>
<br>
<br>
</blockquote></div>