On Mon, Dec 9, 2019 at 5:01 AM Clément Bœsch <u...@pkh.me> wrote: > > On Sun, Dec 08, 2019 at 11:08:31PM +0200, Martin Storsjö wrote: > > On Sun, 8 Dec 2019, Clément Bœsch wrote: > > > > > On Wed, Dec 04, 2019 at 05:24:46PM -0600, Sebastian Pop wrote: > > > > Hi Clément, > > > > > > > > please find attached the updated patch addressing all your comments. > > > > Let me know if there is anything else that I missed and that I need to > > > > address. > > > > > > > > > > I can't test but patch LGTM. Aside from the commit message: it's missing a > > > commit prefix (such as "swscale/aarch64: ...") and I'm not sure what this > > > "FMA" is referring to. > > > > FMA is a common acronym for "fused multiplication and addition", i.e. the > > MLA instruction in AArch64. > > Ah right, my bad, I thought the original code was already doing that. >
I replaced "FMA" with "multiply accumulate" in the subject line and added the prefix as recommended. Ok to commit? Thanks, Sebastian
0001-swscale-aarch64-use-multiply-accumulate-and-increase.patch
Description: Binary data
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
