This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Steam out non-explicit -fno-tree-loop-distribute-patterns for LTO options
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: Kito Cheng <kito dot cheng at gmail dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Richard Biener <rguenther at suse dot de>
- Date: Thu, 28 Aug 2014 10:45:29 +0200
- Subject: Re: [PATCH] Steam out non-explicit -fno-tree-loop-distribute-patterns for LTO options
- Authentication-results: sourceware.org; auth=none
- References: <CA+yXCZAKvQYY4us+RawpSxmfZ4LFdRSw_qiJDgmOJxAdVLAPQA at mail dot gmail dot com>
On Wed, Aug 27, 2014 at 4:42 PM, Kito Cheng <kito.cheng@gmail.com> wrote:
> Hi all:
>
> This patch basically is extension for r210100[1], stream out
> non-explicit -fno-tree-loop-distribute-patterns since compile with
> `-flto -O3 -fno-builtin` still may gen builtin function call during
> LTO phase.
>
> LTO bootstrapped and tested on x86_64-unknown-linux-gnu.
Hmm, but that will for two units, one compiled with -O2 and one
compiled with -O3, produce -O3 -fno-tree-loop-distribute-patterns.
The previous patch restricted this to -ffreestanding / -fno-builtin.
I think it makes more sense to preserve -fno-builtin, that is,
add 'LTO' to the list of FEs supporting 'fbuiltin' (the non-joined version)
and stream '-fno-builtin' if set, doing similar option post-processing
in the LTO frontend as I added to the C family frontends.
Richard.
> 2014-09-27 Kito Cheng <kito@0xlab.org>
>
> * lto-opts.c (lto_write_options): Output non-explicit
> -fno-tree-loop-distribute-patterns.
> * lto-wrapper.c (merge_and_complain): Merge
> -fno-tree-loop-distribute-patterns conservatively.
> (run_gcc): Pass through -fno-tree-loop-distribute-patterns.
>
>
> [1] https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=210100