[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 0/4] limit number of cat and sed forks
From: |
Paolo Bonzini |
Subject: |
Re: [PATCH v3 0/4] limit number of cat and sed forks |
Date: |
Sat, 5 Nov 2016 10:16:38 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 |
On 04/11/2016 03:37, Eric Blake wrote:
> Work in progress, because I still have to review patches 2 and 3
> from Paolo's v2. (Patch 3 can probably be used as-is; patch 2
> will probably necessitate the same changes to CHECK_DECLS_ONCE for
> honoring AC_LANG and reworking the shell list variable to hold
> tuples as I do here for CHECK_FUNCS_ONCE).
>
> The end result is the same number of processes shaved as in Paolo's
> v2 patch 1/3, but now the code is no longer subject to subtle
> semantic differences (the ONCE functions are still checked in the
> same sequence in relation to the rest of configure as before), and
> the configure script does not grow quite as large (on coreutils,
> Paolo's cost about 17k, my approach cost about 2k).
>
> Eric Blake (3):
> AC_CHECK_FUNCS_ONCE: honor current AC_LANG
> AC_CHECK_FUNCS_ONCE: hoist CPP name computation to m4 time
> AC_CHECK_FUNCS_ONCE: hoist cache name computation to m4 time
>
> Paolo Bonzini (1):
> autoconf: prefer an unrolled loop for trivial AC_CHECK_FUNCS
>
> NEWS | 4 ++++
> lib/autoconf/functions.m4 | 60
> +++++++++++++++++++++++++++++++++--------------
> 2 files changed, 47 insertions(+), 17 deletions(-)
>
Looks good, thanks!
Paolo