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_:
>>      /* Ignore these.  */
>>      continue;
>> +      case OPT_flto_:
>> +      {
>> +        const char *lto_canonical = "-flto";
>> +        switches.safe_push (lto_canonical);
>> +        len += strlen (lto_canonical) + 1;
>> +        break;
>> +      }
> 
> 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: Martin Liska <mli...@suse.cz>
Date: Wed, 10 Jul 2019 13:05:19 +0200
Subject: [PATCH] Use -flto instead of -flto=N in DWARF producer string.

gcc/ChangeLog:

2019-07-10  Martin Liska  <mli...@suse.cz>

	* dwarf2out.c (gen_producer_string): Canonize -flto=N
	to -flto in dwarf producer string.
---
 gcc/dwarf2out.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/gcc/dwarf2out.c b/gcc/dwarf2out.c
index 0449c2b2912..aa7fd7eb465 100644
--- 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 char *lto_canonical = "-flto";
+	  switches.safe_push (lto_canonical);
+	  len += strlen (lto_canonical) + 1;
+	  break;
+	}
       default:
         if (cl_options[save_decoded_options[j].opt_index].flags
 	    & CL_NO_DWARF_RECORD)
-- 
2.22.0

Reply via email to