Re: [PATCH] newlib: update nds32 configure checks

On Jan 22 20:39, Mike Frysinger wrote:
> This subdir doesn't link any code, so there's no need to require
> AC_NO_EXECUTABLES, and no other dir uses this either.  Punt it.
> 
> The two tests in here are using compile tests to look for some
> preprocessor defines, but we don't need to compile the code,
> just preprocess it.  So switch to AC_PREPROC_IFELSE.
> ---
>  newlib/libm/machine/nds32/Makefile.in  |   1 +
>  newlib/libm/machine/nds32/configure    | 266 +++++++++++++++++--------
>  newlib/libm/machine/nds32/configure.ac |   5 +-
>  3 files changed, 186 insertions(+), 86 deletions(-)
>  mode change 100644 => 100755 newlib/libm/machine/nds32/configure

LGTM

Thanks,
Corinna