<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 7, 2016 at 4:57 PM, Matt Turner <span dir="ltr"><<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, Apr 7, 2016 at 1:56 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> Now that we can use the much simpler rgba8_copy function, we don't need to<br>
> hand different functions out based on direction.<br>
> ---<br>
<br>
</span>Typo in the subject, and PATCH 2/3 needs a bugzilla link.<br></blockquote><div><br></div><div>Fixed<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The series is<br>
<br>
Reviewed-by: Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>><br>
<br>
I noticed that gcc does *not* unroll the loop in the 64-byte case.<br>
I've got some small optimizations to this code that I'll send after<br>
this series, so I'll just fix that up at the same time.<br>
</blockquote></div><br></div><div class="gmail_extra">I just pushed.  I added a Cc to stable for the first two patches.  I recommend you do as well for whatever you send that fixes the unrolling problem.<br></div><div class="gmail_extra">--Jason<br></div></div>