On Fri, 10 Feb 2017, Janne Grunau wrote:

On 2017-02-09 13:27:04 +0200, Martin Storsjö wrote:
The idct32x32 function actually backed up and restored d8-d15 even

... pushed onto the stack ... is imo clearer even though there are no explicit push/pop instructions

though it didn't clobber them; there are plenty of registers that
can be used to allow keeping all the idct coefficients in registers
without having to reload different subsets of them at different
stages in the transform.

After this, we still can skip backing up and restoring d12-d15.

same

Thanks, amended the commit message locally, similarly for the arm version as well.

// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to