This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [build,darwin] Fix toplevel configure test for LTO on darwin

From: FX <fxcoudert at gmail dot com>
To: gcc-patches <gcc-patches at gcc dot gnu dot org>
Cc: mikestump at comcast dot net
Date: Wed, 22 Mar 2017 12:53:16 +0100
Subject: Re: [build,darwin] Fix toplevel configure test for LTO on darwin
Authentication-results: sourceware.org; auth=none
References: <DD3DCA96-E79A-4832-BC47-6A510D6B34B1@gmail.com>

*ping*


When LTO was introduced, on macOS it needed darwin9 to work. Over time, most tests for “*-apple-darwin9” in the toplevel configure were changed to also include later darwin versions: *-darwin[[912]]*  However, the check for LTO was not updated. As far as I know, this is merely an oversight: LTO works fine on Darwin, and most vendors (homebrew, macports, etc.) have been shipping with --enable-lto for years.

The attached patch makes it build LTO by default on darwin >= 9. I realize it’s late but I hope the very restricted nature (and ultra-low risk) of the patch make it a candidate for inclusion in trunk nonetheless.

Bootstrapped and regtested on x86_64-apple-darwin16. OK to commit?

FX

Attachment: lto_darwin.ChangeLog
Description: Binary data

Attachment: lto_darwin.diff
Description: Binary data

References:
- [build,darwin] Fix toplevel configure test for LTO on darwin
  - From: FX

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]