<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><div>just C pointer, no review necessary<br><br>At 2015-04-27 21:48:09,"Divya Manivannan" <divya@multicorewareinc.com> wrote:<br>># HG changeset patch<br>># User Divya Manivannan <divya@multicorewareinc.com><br>># Date 1430142169 -19800<br>>#      Mon Apr 27 19:12:49 2015 +0530<br>># Node ID 063c929c30fd7f1e2af3ac1b0fd1d647d690f781<br>># Parent  ffe98e3a23af5a05d5cedafb876db1b4960c71fc<br>>sao: add comment for the reason of two versions of sao primitives<br>><br>>diff -r ffe98e3a23af -r 063c929c30fd source/common/primitives.h<br>>--- a/source/common/primitives.h     Mon Apr 27 18:07:45 2015 +0530<br>>+++ b/source/common/primitives.h      Mon Apr 27 19:12:49 2015 +0530<br>>@@ -275,8 +275,17 @@<br>> <br>>     sign_t                sign;<br>>     saoCuOrgE0_t          saoCuOrgE0;<br>>+<br>>+    /* To avoid the overhead in avx2 optimization in handling width=16, SAO_E0_1 is split<br>>+     * into two parts: saoCuOrgE1, saoCuOrgE1_2Rows */<br>>     saoCuOrgE1_t          saoCuOrgE1, saoCuOrgE1_2Rows;<br>>+<br>>+    // saoCuOrgE2[0] is used for width<=16 and saoCuOrgE2[1] is used for width > 16.<br>>     saoCuOrgE2_t          saoCuOrgE2[2];<br>>+<br>>+    /* In avx2 optimization, two rows cannot be handled simultaneously since it requires <br>>+     * a pixel from the previous row. So, saoCuOrgE3[0] is used for width<=16 and <br>>+     * saoCuOrgE3[1] is used for width > 16. */<br>>     saoCuOrgE3_t          saoCuOrgE3[2];<br>>     saoCuOrgB0_t          saoCuOrgB0;<br>> <br>>_______________________________________________<br>>x265-devel mailing list<br>>x265-devel@videolan.org<br>>https://mailman.videolan.org/listinfo/x265-devel<br></div></div>