<div dir="ltr">All the patches of this have been pushed to the master branch. <br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><b>__________________________</b></div><div><b>Karam Singh</b></div><div><b>Ph.D. IIT Guwahati</b></div><div><font size="1">Senior Software (Video Coding) Engineer  </font></div><div><font size="1">Mobile: +91 8011279030</font></div><div><font size="1">Block 9A, 6th floor, DLF Cyber City</font></div><div><font size="1">Manapakkam, Chennai 600 089</font></div></div></div></div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Aug 16, 2024 at 11:52 AM chen <<a href="mailto:chenm003@163.com">chenm003@163.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="line-height:1.7;color:rgb(0,0,0);font-size:14px;font-family:Arial"><div id="m_7324152658486262547spnEditorContent"><p style="margin:0px">Hi Hari,</p><p style="margin:0px"><br></p><p style="margin:0px">Thank for the explain, agree with you, we can keep your version.</p><p style="margin:0px"><br></p><p style="margin:0px">Regards,</p><p style="margin:0px">Chen</p></div><pre>At 2024-08-15 18:58:09, "Hari Limaye" <<a href="mailto:hari.limaye@arm.com" target="_blank">hari.limaye@arm.com</a>> wrote:
>Hi Chen,
>
>Thank you for reviewing the patches.
>
>>What's reason rename these parameters? in the AArch64 ABI, only first 8 parameters passthrough SIMD registers, compiler will take care these inout parameters.
>
>The reason for separating and renaming these parameters is because the ACLE type that this function "returns" via the output parameters is either int16x8_t or int32x4_t depending on whether HIGH_BIT_DEPTH is set. Explicitly separating the parameters here allows us to use the correct ACLE types for all variables and operations, enabling the use of the compiler flag -flax-vector-conversions=none (which is the primary purpose of this patch series). It is for code sanitisation, rather than optimisation.
>
>>if you want to improve these interpolate functions, how about also improve algorithm?
>
>Thanks for the suggestion here - these patches are intended purely to clean up the code (with only a couple of minor peephole optimisations here and there), to enable future optimisation patches. We have a large set of patches providing significant uplift for interpolation functions that will be pushed after this series.
>
>Many thanks,
>Hari
</pre></div>_______________________________________________<br>
x265-devel mailing list<br>
<a href="mailto:x265-devel@videolan.org" target="_blank">x265-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/x265-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
</blockquote></div>