Re: [PATCH v2] arm: Don't require a runtime check function for targets with unconditional NEON

On Tue, 11 Oct 2022, Cosmin Truta wrote:

> On Tue, Oct 11, 2022 at 7:07 PM John Bowler
> <[email protected]> wrote:
>
>> PNG_ARM_NEON_OPT is a *setting*, PNG_ARM_NEON_CHECK_SUPPORTED is an *option*.
>
> Reading this statement reminds me how insanely over-engineered the
> libpng build configuration system is. What other library or codec
> comes even close? (This is not a rhetorical question, and I'm not
> trying to be sarcastic here; just asking about best practices employed
> elsewhere.)

The configuration of the entire Linux Kernel comes close to how 
powerful the configuration of libpng is.

Since Linux is so complicated, it comes with source code to a 
'menuconfig' program which is built in advance, and allows configuring 
everything else.

A problem with simplifying libpng configuration is that some 
applications excessively use libpng configuration options in their 
implementation.  GraphicsMagick's png/mng support may match that 
description.

When libpng was invented, it seems that optimization was everything. 
It did not just optimize compared with GIF, but it also optimized 
every detail of its implementation so it is possible to achieve an 
almost perfect configuration where no dead code is produced.

Bob
-- 
Bob Friesenhahn
[email protected], http://www.simplesystems.org/users/bfriesen/
GraphicsMagick Maintainer,    http://www.GraphicsMagick.org/
Public Key,     http://www.simplesystems.org/users/bfriesen/public-key.txt