From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.ffmpeg.org (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id BC78244B1F for ; Mon, 14 Jul 2025 08:49:10 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id 0579568E89B; Mon, 14 Jul 2025 11:49:08 +0300 (EEST) Received: from haasn.dev (haasn.dev [78.46.187.166]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id ACF7968E845 for ; Mon, 14 Jul 2025 11:49:01 +0300 (EEST) Received: from haasn.dev (unknown [10.30.1.1]) by haasn.dev (Postfix) with UTF8SMTP id 637A141A4B for ; Mon, 14 Jul 2025 10:49:01 +0200 (CEST) Date: Mon, 14 Jul 2025 10:49:01 +0200 Message-ID: <20250714104901.GB56899@haasn.xyz> From: Niklas Haas To: ffmpeg-devel@ffmpeg.org In-Reply-To: References: <20250712104459.52153-1-ffmpeg@haasn.xyz> <20250712104459.52153-13-ffmpeg@haasn.xyz> MIME-Version: 1.0 Content-Disposition: inline Subject: Re: [FFmpeg-devel] [PATCH v8 13/18] swscale/ops_backend: add reference backend basend on C templates 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 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On Sun, 13 Jul 2025 22:14:44 +0200 Andreas Rheinhardt wrote: > Niklas Haas: > > From: Niklas Haas > > > > This will serve as a reference for the SIMD backends to come. That said, > > with auto-vectorization enabled, the performance of this is not atrocious. > > It easily beats the old C code and sometimes even the old SIMD. > > > > In theory, we can dramatically speed it up by using GCC vectors instead of > > arrays, but the performance gains from this are too dependent on exact GCC > > versions and flags, so it practice it's not a substitute for a SIMD > > implementation. > > --- > > diff --git a/libswscale/ops.c b/libswscale/ops.c > > index c4f03531eb..2d8f9e6084 100644 > > --- a/libswscale/ops.c > > +++ b/libswscale/ops.c > > @@ -27,7 +27,10 @@ > > #include "ops.h" > > #include "ops_internal.h" > > > > +extern SwsOpBackend backend_c; > > Why are this and the other backends not const? Fixed. > > > + > > const SwsOpBackend * const ff_sws_op_backends[] = { > > + &backend_c, > > NULL > > }; > > > _______________________________________________ > 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". _______________________________________________ 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".