<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><div style="color: rgb(0, 0, 0); line-height: 1.7; font-family: arial; font-size: 14px;"><div>patch is right, just a little modify in future<br><br>At 2014-11-19 11:07:12,"Divya Manivannan" <<a href="mailto:divya@multicorewareinc.com">divya@multicorewareinc.com</a>> wrote:<br>># HG changeset patch<br>># User Divya Manivannan <<a href="mailto:divya@multicorewareinc.com">divya@multicorewareinc.com</a>><br>># Date 1416366360 -19800<br>>#      Wed Nov 19 08:36:00 2014 +0530<br>># Node ID 2f513ea131131ccfe73826d19c5e275bb4a5013e<br>># Parent  d059cfa88f1ac79b319bd8a05bc70704d454f0ba<br>>asm: luma_vpp[8x8] in avx2: improve 701c->387c<br>><br>>diff -r d059cfa88f1a -r 2f513ea13113 source/common/x86/ipfilter8.asm<br>>--- a/source/common/x86/ipfilter8.asm Tue Nov 18 14:11:12 2014 -0600<br>>+++ b/source/common/x86/ipfilter8.asm Wed Nov 19 08:36:00 2014 +0530<br>>@@ -122,6 +122,26 @@<br>>                   times 8 db 58, -10<br>>                   times 8 db 4, -1<br>> </div>
<div> </div>
<div>we'd better put ALIGN here to avoid crash, because you direct use [r5] below</div>
<div>you can modify it in next patch</div>
<div><br>>+tab_LumaCoeffVer_32: times 16 db 0, 0<br>>+                     times 16 db 0, 64<br>>+                     times 16 db 0, 0<br>>+                     times 16 db 0, 0<br>>+<br>>+                     times 16 db -1, 4<br>>+                     times 16 db -10, 58<br>>+                     times 16 db 17, -5<br>>+                     times 16 db 1, 0<br>>+<br>>+                     times 16 db -1, 4<br>>+                     times 16 db -11, 40<br>>+                     times 16 db 40, -11<br>>+                     times 16 db 4, -1<br>>+<br>>+                     times 16 db 0, 1<br>>+                     times 16 db -5, 17<br>>+                     times 16 db 58, -10<br>>+                     times 16 db 4, -1<br>>+<br>> tab_c_64_n64:   times 8 db 64, -64<br></div></div></div>