Date: Tue, 26 Oct 2021 17:09:26 +0200
From: Christoph Berg <myon@debian.org>
To: "Pop, Sebastian" <spop@amazon.com>
Cc: PostgreSQL in Debian <pgsql-pkg-debian@postgresql.org>,
	"Saidi, Ali" <alisaidi@amazon.com>
Subject: Re: [PATCH] audo-detect and use -moutline-atomics compilation flag
 for aarch64
Message-ID: <YXgaJiuh0iM5igSS@msg.df7cb.de>
Mail-Followup-To: Christoph Berg <myon@debian.org>,
	"Pop, Sebastian" <spop@amazon.com>,
	PostgreSQL in Debian <pgsql-pkg-debian@postgresql.org>,
	"Saidi, Ali" <alisaidi@amazon.com>
References: <1635221042457.21654@amazon.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1635221042457.21654@amazon.com>
Archived-At: 
 <https://www.postgresql.org/message-id/YXgaJiuh0iM5igSS%40msg.df7cb.de>
Precedence: bulk

Re: Pop, Sebastian
> ?Hi Christoph,
> 
> 
> This is a follow-up email based on recommendations from Heikki Linnakangas:
> 
> https://www.postgresql.org/message-id/5f048a5a-c471-50ae-30ad-385dcafc616b%40iki.fi
> 
> and Tom Lane:
> 
> https://www.postgresql.org/message-id/888692.1601489331%40sss.pgh.pa.us

Well I wouldn't exactly call these messages "recommendations".

> ?The change is to compile PostgreSQL for arm64 with an extra flag -moutline-atomics when the compiler is gcc-8.5 or gcc-9.4.
> 
> -moutline-atomics is enabled by default in all newer compilers gcc-{10, 11, trunk}, and on clang-{12, 13, trunk}.
> 
> The change is needed only for a limited number of Debian and Ubuntu releases compiled by gcc versions with the outline-atomics flag disabled by default.

None of the Debian releases are using 8.5 or 9.4, and on the Ubuntu
side focal is using 9.3. Is that gcc version even affected?

Christoph