Re: [PATCH] Use -flto instead of -flto=N in DWARF producer string.

2019-07-22 Thread Jeff Law
sorry for that. > > Martin > > > 0001-Use-flto-instead-of-flto-N-in-DWARF-producer-string.patch > > From eda41b25bf8b91412683ad542074724c872b18a4 Mon Sep 17 00:00:00 2001 > From: Martin Liska > Date: Wed, 10 Jul 2019 13:05:19 +0200 > Subject: [PATCH] Use -flto inst

Re: [PATCH] Use -flto instead of -flto=N in DWARF producer string.

2019-07-15 Thread Martin Liška
On 7/10/19 1:16 PM, Martin Liška wrote: > On 7/10/19 1:15 PM, Jakub Jelinek wrote: >> On Wed, Jul 10, 2019 at 01:08:52PM +0200, Martin Liška wrote: >>> --- a/gcc/dwarf2out.c >>> +++ b/gcc/dwarf2out.c >>> @@ -24460,6 +24460,13 @@ gen_producer_string (void) >>>case OPT_fchecking_: >>> /*

Re: [PATCH] Use -flto instead of -flto=N in DWARF producer string.

2019-07-10 Thread Martin Liška
; > The indentation looks off, when case is indented by 6 columns, > { should be by 8 (i.e. a tab) and const by 10 (i.e. a tab + 2 spaces). > > Jakub > You are right, sorry for that. Martin >From eda41b25bf8b91412683ad542074724c872b18a4 Mon Sep 17 00:00:00 2001 From: Mar

Re: [PATCH] Use -flto instead of -flto=N in DWARF producer string.

2019-07-10 Thread Jakub Jelinek
On Wed, Jul 10, 2019 at 01:08:52PM +0200, Martin Liška wrote: > --- a/gcc/dwarf2out.c > +++ b/gcc/dwarf2out.c > @@ -24460,6 +24460,13 @@ gen_producer_string (void) >case OPT_fchecking_: > /* Ignore these. */ > continue; > + case OPT_flto_: > + { > + const cha

[PATCH] Use -flto instead of -flto=N in DWARF producer string.

2019-07-10 Thread Martin Liška
Hi. We're using LTO in openSUSE package builds and we use -flto=N based on number of CPU that a builder machine has. That leads to debug info divergence and so that I would like to canonize the option in producer string. Ready to be installed after tests & bootstrap? Thanks, Martin gcc/ChangeLog