<div dir="ltr">Looks like exactly the same SFINAE as the existing ArrayRef->ArrayRef case, so it sounds good to me.<br><br>Thanks!<br><br>- Dave</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 13, 2015 at 3:31 PM, Pete Cooper <span dir="ltr"><<a href="mailto:peter_cooper@apple.com" target="_blank">peter_cooper@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi David<br>
<br>
This was on the discussion of r237226 (<a href="http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20150511/276285.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20150511/276285.html</a>).<br>
<br>
This patch allows construction of ArrayRef<const T*> from SmallVector<T*> and std::vector<T*>.  This avoids having to use an intermediate call to makeArrayRef which is the current method to achieve this.<br>
<br>
I’ve used this to update the calls in LICM and SROA which had been changed to use makeArrayRef in r237226.<br>
<br>
Comments welcome.<br>
<br>
Cheers,<br>
Pete<br>
<br>
</blockquote></div><br></div>