svn commit: r522069 - head/databases/redis-devel

[Gerald Pfeifer]

On Sat, 4 Jan 2020, Sergey A. Osokin wrote:
> +.if ${ARCH} == i386 && ! ${CFLAGS:M-march=*}
> +# Needed for __atomic_fetch_add_8
> +USE_GCC=	yes
> +CFLAGS+=	-march=i586
>  .endif

This has been a while ago, yet I noticed it is still in place.

Are you sure clang does not provide that? GCC 4.2 should not be
an issue on i386 any more since FreeBSD 11 is already on clang.

And the default for current GCC on FreeBSD on i386 already is
i586 since

  commit 01a0467e237b69e02f5467babfe2d235c8c68b68
  Author: Gerald Pfeifer <gerald at pfeifer.com>
  Date:   Sun Nov 15 20:03:24 2015 +0000

    freebsd.h (SUBTARGET32_DEFAULT_CPU): Change to i586.
    
            * config/i386/freebsd.h (SUBTARGET32_DEFAULT_CPU): Change to i586.
            Remove support for FreeBSD 5 and earlier.

so the CFLAGS line definitely should not be required.

Gerald