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 06EB24C3F4 for ; Sat, 2 Aug 2025 18:27:41 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id C65A368BE56; Sat, 2 Aug 2025 21:27:37 +0300 (EEST) Received: from mout.gmx.net (mout.gmx.net [212.227.17.21]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTPS id 943DD687DE4 for ; Sat, 2 Aug 2025 21:27:30 +0300 (EEST) X-UI-Sender-Class: 724b4f7f-cbec-4199-ad4e-598c01a50d3a Received: from metallschleim.local ([91.62.6.9]) by mail.gmx.net (mrgmx104 [212.227.17.168]) with ESMTPSA (Nemesis) id 1MIMbU-1uuORR2jhc-00AKEl for ; Sat, 02 Aug 2025 20:27:29 +0200 Date: Sat, 2 Aug 2025 20:27:28 +0200 To: FFmpeg development discussions and patches Message-ID: References: <20250613083704.6858-1-jiawei@iscas.ac.cn> <20250614224942.GT5177@pb2> <7634ba8e-7e76-a11b-98f-d7d5cb61eb1@martin.st> <20250711214128.GU29660@pb2> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Provags-ID: V03:K1:ahIxCiKEZDJuhkRN0EmR6MMSW502kqjyrB2HORnvcHa/2/JJWjF 8YohF7F7l3sX6jvHP7drwuRCvkIYw+At9Vd+p6eA44sB9ak/qR1aBz8u5RB/f8U5UImXrpE yw8p1a8J0lfsE/kzJw/GwO6h5yoiWl57tDP0It1tzEmXsP0wPgr1v8K45tG8rMWHYljFEdn msYAyitZBIKXn9Cgeoivw== X-Spam-Flag: NO UI-OutboundReport: notjunk:1;M01:P0:1Vz4zcNnUZI=;ixy98uRsWqRywTJEqE9CQKBCaru 8OcbrO3bXzF/XDRwDn5cQCmHFUnlqIoLeTKSocsLJex1UVkBuBpuUBSn2u4jtA9VqXUDv1LWu XS9du+3esspGqhRtPhI2YWIPWcsjX0zeLlleOo2V1YkoyXbcqKLiHa+1MU5/w8RiF48RTHzY+ YKKnvw1zwgU3p5/vtBjmFCnnxQW2c8f5+olt3Hp+nqemhT5Rra7JbyUocZUigdcnQaGdJPg8N cbGiyZibs4bwt1oQ/Q1B6UDt6x8apFIQfbBVFkjDW81iyMMQmyXpBaDZI8Hoy/mQMPT2i4Xcl N2MRdV3HTwGCaZxYwc+eSe5mc0K0bCFQo7sqUcewd/pxBrkuWhD+3NZJ65rDLHNyDswFx0vvt jlH3Ed3anCw/OuWFGVik6H3DA0Q/xm4XU7xunA4TaUwNXiQb6adjnGrr8gzQh/Akn2uPOQ7WM iJX7SPrvA8nNJUyQCOwUGgTU0j07hM6H2SuCpchEWx09cRVdUjkr/kT79eEOzXFr81++tsNP9 QdjPYl6+5RWPte9BMyjr7YveXGOmnbWcQiFCGhdD3yxVjqzBOQtDz8j/dC3CwueextxXw1tTf 0DyKXD/LEjhks9CEZDQJFKHOftiCm5OKg08BXyes0g6imXEZBBlfaxgvxYJfnw3/aS1bQL1er 00GqXM77rgnKxw8Ws8K76nPmXjpJsXrxY3Mp458Su269grczUGTxsNvgkHzRFdtVmBo1jZJRb qGo1z8Ptwax1fcjy4GXm5yYuRuWxO+Ct6xdQn/96GMc1D5CB/g3tL0A5sFjZ9lkhZ9KncdwfL 5NE9bt/XcbN1es52tCUWlz2f5HPcv4/8hNjmT94GCOl9ZOsR1m0hCcj9/w00TLFdlF1Qn5pjW 998NArbrGlIznGolSmKUC+A8pZYnEH3iMyPiZxKClSTcIRc4064k29I+VJjqOMIFXCUuMqRP/ Fe0Wh04FNjxjNVKeGQ2J53J+/hSYNJFlYR+/JMzd1tx88Bh4lHa8Xifge44ppM4qFBdGcCjT6 jDm7CX215Pa6cjVElwaasBowLeRlxAzlg+jjr3zj3z1aqhg7JyoWli5vsL0WHdKSsrtGykz4l AZFNH5rTPStSi3sBb+i8ppujjup/STpqbiVzRhrPCyPht8NNh90AuX2x829+CLBGkwu2vNr+E LmBLlrRuC+/Xwh0JL3Xc2ESqYTKdXaERvRw0+XWUCALWf0q8nJMcuFWfIQFM/V4F9rt0UTiUp 1MI3xW9kxobbhvbDe1ttOAKgsYDuP1tY+is3FRpmzB3oky8QRhQa6f57y1OwFQYAtgY6+AukV C50WuluE1dskdSL0IVz0gJ7IQwoyFG9w/hzUV0GzLw0eZuRMamlNJUK4VLNat95pUaHoRrDOF dvayc4kAQ7jHDe9+7Iy7WWI3Cs9Fb6dl6bYavLPY68ORYFcqRozVJadEQOu7rhcy+thwe1aMh 8ccxv3HqTu5xg311trZDMq+Pll+SR1qhzoTi8+c3qxp0ORe0QQsvidn/xkufX1ih6+vQni9x2 7fep6VIT66pvGFE8CGk1dnGFFPrUDnyPPt5b07Fi1sAxGPKpmVazxpskYEk470ELBHZ/YMuva akLvpdmssVOBrQIRKwKnnbn58cQ6KtApWtHza3XBRz49u/3UVNGnWcQ9blrSqRZayHwOLE9HK j5KFf8mReeKG/sIChk92BBZ/t3qTIzM8DCHYUSWkRFTrAsz6fv5jkK3HJLmJpLyKfP1u4LTit 1M40/aQF9mywKJkM2cGTky2Nma3Icn6DBoTtajIgAkFNMicT9PhLNq4R7BJmx53A3Uj2z6A8B Fd991vCEFO23TWMd6CcFiiVoySK3lIFMWN+AuDDbZiC0208jAqx3l1rM3Iosu336Y132UnTk3 6ebbskVoKmfc3mJzLrxHQ4DVXS/5A8cBtM9LPaU0TPMtyMg1kKQwxps/oS+Fd4czJVnEEt4ac FhR4b/iPir+MM2R+HUMrvnhtf09CBkbpk6CN5luOH1B1WG4WhiC5Q1VdmUcTF+q7OGHT+i4XT ReQeoCnCK5z8EEe5KBJQFr/KxnHdLJF+nAPgEOKi1T0WdyGV4p/q24/E/IjLIzMLnBI6XUMVx a2V5p89/qbZuyiEjkThVqvNCrx6e8RN0GGvsDkckHjkXAK+9tduj018kMlPgBfIEygAQ3LV3o h4iPDfMdwWa1dmTcMFJc6+cQ4B5I/fL3Rsd2SNLrmzOIYOlulxVFxU8okcffcu6jSPqRP5Zx1 CSjHzO4OauSn157tc9UWXXFADhGg+Y8Hw5rbUdCjiUdKhk3Seb0qliPTpNE8onb8byWbCTZqh gR0TMW3FTW5vhatjCa2isGrHCJakc5SXdEjw7AGO/m1YkI6cUVrwyYgrTmg3rlhAXs5y7xha/ 9HuuJFfXuYYRvqumBfWOsk5yX7UQ1osf+J8bnpP3FOKLaWjhhZIdviV2/dwiuEF4I7IXHPGw9 cORSbUHMP6I6b7i/I/qezJ1EjTKtvuvH5MIPu8VlujV5t1K2wRT9viRBD38kRoIk9Asl1kLr4 Otx7N2wOtpnDhYozwFscdvrZhiqZFc0fB7WPtWs56GpBcsxrVMrTOviRH9uMYwCAs980+oeP8 Ve+uB7eohHU6kjkRTeZY7L5AETF08dp48aikEi2Lrfa/TvobNdylQBSNh1UIb3rGZHR5LzUPP mPpcR4gnXtau2fOspNyAsLMqm8e/HNZ7gpKGStOO2grTmE7D04fnM5VES/e1KHYaNdiyjXUzn uO7iRZdsz0O3q0P5FateGkMLJqmKQBrmTrIlcsDv4sIjsdQ5t8KoOp9GlCZU8Xojq2ARfvP9a ydCSRUCZZFGJaTCYNhCieQGgJMog1st38yP7lr/kDCVXuvD9pS5QhacTLlWgaR7Iw61yrwmhI a0300Ck8yV754V5A5AZwCXqtRARrh3qcvOfzqveguHfWLm/7GEZ7PsUZhPO05ZQJlRXT7kUt0 HH6DMXAEi444UDwRpcuitEnvAm4GYWMbddOfCFpd0u1T2eNCxgxGkh7u/X/4p/40t6JfaNjx7 z2q/DaC26jOZ/7UShdw+gA7sGsNkwIBQNwWCHHYv9VhFvJMXas1gcQTE3dv1QYhp+8Vbd/GeY ZHr9IVgpwylCUwhTiYFXliCe6x0HgWPev0Eo+U1CnkvazwuccnWE2/bsxvrKihXPGwhVcCdyB 9F3EQQBGmWd0mT3gs3/3n31cL6aH+bbV64xVdVkcgGEXN5SI7bIqt6gSVnvPQvKLYjZAqRg/3 NTIehAJF3sgJr0SEGbwpISdZJNVG897S7ppTX/Cri/728PISA58T12E/5PbfPLggW0czug0AR n5VMWISC6NQs4AqYHm84yQGkj/uHEa1G74b2wxx7P7pgm0C6JmveGWVqqwaKdIyxO0aQZHd+s xmKUBWJvSbdVr/bhAZt25VYIN74AlJyz6h2EiZHxrLthBwS4K62FD7FatYvATlBiETYAMvnQ6 Q6pvjQ97Zv95lRrvl1GyP8K0wbwbkrxPbknkVQxzfPdHeApNgci5wHTo004tr/ibv6X9XM11k 0g== Subject: Re: [FFmpeg-devel] [PATCH v5] gcc: Don't disable '-ftree-vectorize' if gcc version higher than 13. 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: , From: Alexander Strasser via ffmpeg-devel Reply-To: FFmpeg development discussions and patches Cc: Alexander Strasser Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On 2025-07-18 17:42 +0200, Kacper Michajlow wrote: > On Fri, 11 Jul 2025 at 23:41, Michael Niedermayer > wrote: > > On Thu, Jul 10, 2025 at 02:35:10PM +0300, Martin Storsj=F6 wrote: [...] > > > > > I'm unsure if we should allow it everywhere, and just single out loon= garch > > > as a case where it is known broken, or if we should only let GCC enab= le it > > > on the major architectures that we've tested. > > > > I think for architectures where there is an active maintainer, > > that maintainer should decide. > > > > For architectures where there is noone looking after them, its > > safer to leave it disabled. Otherwsie users could be having > > complex to debug issues to deal with and noone in ffmpeg able to > > replicate > = > I agree. In a perfect world we would enable it everywhere, but if > there is a worry that GCC is not a mature enough compiler to enable > it, it's better to gradually roll it out on an opt-in basis. We can > start with major architectures and for other architectures allow > maintainers to decide. > = > Alternatively enable it everywhere for start and selectively disable > it, once issues are identified. This may cause breakage, but is easy > to workaround (by adding a compiler flag), and will likely identify > issues with GCC faster. > = > Bottom line is that adding `-fno-tree-vectorize` in current year is > quite a heavy hammer to avoid possible issues and especially on major > architectures like x86 it should be safe to enable vectorization. It > also currently gives clang and advantage over gcc if we compare pure c > code in some cases. Do we have concrete numbers how much better it is? I mean pure C versions of FFmpeg were never really recommended. Would also need the numbers per architecture. Otherwise I would argue people that care about it because they don't have Assembly implementations for their target or use more C code because of other reasons should just enable vectorization as well if they think it works for them. Anyway I see no problem to enable compiler vectorization for compilers and architectures where it works correctly and active maintainers agree on it. And act, should it ever break again. Alexander _______________________________________________ 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".