From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id 9EFED446F9 for ; Wed, 21 Sep 2022 10:12:01 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 80E7568BB88; Wed, 21 Sep 2022 13:11:58 +0300 (EEST) Received: from mail8.parnet.fi (mail8.parnet.fi [77.234.108.134]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 6253868B89E for ; Wed, 21 Sep 2022 13:11:51 +0300 (EEST) Received: from mail9.parnet.fi (mail9.parnet.fi [77.234.108.21]) by mail8.parnet.fi with ESMTP id 28LABnUg016588-28LABnUh016588; Wed, 21 Sep 2022 13:11:49 +0300 Received: from foo.martin.st (host-97-187.parnet.fi [77.234.97.187]) by mail9.parnet.fi (Postfix) with ESMTPS id 8F5C6A1467; Wed, 21 Sep 2022 13:11:49 +0300 (EEST) Date: Wed, 21 Sep 2022 13:11:49 +0300 (EEST) From: =?ISO-8859-15?Q?Martin_Storsj=F6?= To: Hubert Mazur In-Reply-To: <20220920110158.15384-1-hum@semihalf.com> Message-ID: References: <20220920110158.15384-1-hum@semihalf.com> MIME-Version: 1.0 X-FE-Policy-ID: 3:14:2:SYSTEM Subject: Re: [FFmpeg-devel] [PATCH 0/3] Provide neon implementations X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Cc: gjb@semihalf.com, upstream@semihalf.com, jswinney@amazon.com, ffmpeg-devel@ffmpeg.org, mw@semihalf.com, spop@amazon.com Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On Tue, 20 Sep 2022, Hubert Mazur wrote: > This fixes issues addressed in previous patchset: > - move sub instruction in vsad8_intra, > - remove unnecessary mov instructions, > - remove single lane extraction in loop and place it at the end. > > Removing mov instructions from pix_median_abs functions significantly > increased peformance for both. I'm quite sure that it wasn't the removed mov instructions that improved performance (those instructions should be essentially free, they're just misleading), but the fact that you got rid of the extra single-element handling within the loop. Anyway, the patches seem fine to me now, so I'll push them. Thanks! // Martin _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".