Fix some narrowing conversion issues

Message ID	20170627054824.GN2123@tucnak
State	New
Headers	show Return-Path: <gcc-patches-return-456877-incoming=patchwork.ozlabs.org@gcc.gnu.org> DomainKey-Signature: a=rsa-sha1; c=nofws; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender:date :from:to:subject:message-id:reply-to:mime-version:content-type; q=dns; s=default; b=C4eaLQs8aWuYTaPOerzfg2N7BjYx7SSNylYNE3VcZyX plY8IWFm85Q1y5LP2HjKCDXagNv94ifrikP21VUdSgUQNEsxqkJ4q4wF2qV4AYZS 18piDNHmWTtgnDj3WgDUUMuqSFFqBf8ZKtfD7UacIPgIeb1bQvXtEfeh/jOj/jrw = Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk Sender: gcc-patches-owner@gcc.gnu.org DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com DCCEE80C0D Date: Tue, 27 Jun 2017 07:48:24 +0200 From: Jakub Jelinek <jakub@redhat.com> To: gcc-patches@gcc.gnu.org Subject: [PATCH] Fix some narrowing conversion issues Message-ID: <20170627054824.GN2123@tucnak> Reply-To: Jakub Jelinek <jakub@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.7.1 (2016-10-04)

Message ID

20170627054824.GN2123@tucnak

State

New

Headers

DomainKey-Signature: a=rsa-sha1; c=nofws; d=gcc.gnu.org; h=list-id
	:list-unsubscribe:list-archive:list-post:list-help:sender:date
	:from:to:subject:message-id:reply-to:mime-version:content-type;
	q=dns; s=default; b=C4eaLQs8aWuYTaPOerzfg2N7BjYx7SSNylYNE3VcZyX
	plY8IWFm85Q1y5LP2HjKCDXagNv94ifrikP21VUdSgUQNEsxqkJ4q4wF2qV4AYZS
	18piDNHmWTtgnDj3WgDUUMuqSFFqBf8ZKtfD7UacIPgIeb1bQvXtEfeh/jOj/jrw
	=
Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
Sender: gcc-patches-owner@gcc.gnu.org
DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com DCCEE80C0D
Date: Tue, 27 Jun 2017 07:48:24 +0200
From: Jakub Jelinek <jakub@redhat.com>
To: gcc-patches@gcc.gnu.org
Subject: [PATCH] Fix some narrowing conversion issues
Message-ID: <20170627054824.GN2123@tucnak>
Reply-To: Jakub Jelinek <jakub@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
User-Agent: Mutt/1.7.1 (2016-10-04)

Commit Message

Jakub Jelinek June 27, 2017, 5:48 a.m. UTC

Hi!

We build gcc with -Wno-narrowing, for some reason I ended up with old
Makefile without that and discovered a couple of -Wnarrowing errors.

This patch fixes them.  Bootstrapped/regtested on x86_64-linux (i686-linux
fails to bootstrap with/without this patch), ok for trunk?

2017-06-27  Jakub Jelinek  <jakub@redhat.com>

	* predict.c (test_prediction_value_range): Use -1U instead of -1
	to avoid narrowing conversion warning.
	* dumpfile.c (dump_options): Wrap all value into dump_flags_t cast
	to avoid narrowing conversion warning.
	* opt-functions.awk (var_ref): Return (unsigned short) -1 instead of
	-1.
	* optc-gen.awk (END): Expect (unsigned short) -1 instead of -1.


	Jakub

Comments

Richard Biener June 27, 2017, 8:50 a.m. UTC | #1

On Tue, Jun 27, 2017 at 7:48 AM, Jakub Jelinek <jakub@redhat.com> wrote:
> Hi!
>
> We build gcc with -Wno-narrowing, for some reason I ended up with old
> Makefile without that and discovered a couple of -Wnarrowing errors.
>
> This patch fixes them.  Bootstrapped/regtested on x86_64-linux (i686-linux
> fails to bootstrap with/without this patch), ok for trunk?

Ok.

> 2017-06-27  Jakub Jelinek  <jakub@redhat.com>
>
>         * predict.c (test_prediction_value_range): Use -1U instead of -1
>         to avoid narrowing conversion warning.
>         * dumpfile.c (dump_options): Wrap all value into dump_flags_t cast
>         to avoid narrowing conversion warning.
>         * opt-functions.awk (var_ref): Return (unsigned short) -1 instead of
>         -1.
>         * optc-gen.awk (END): Expect (unsigned short) -1 instead of -1.
>
> --- gcc/predict.c.jj    2017-06-21 16:53:37.000000000 +0200
> +++ gcc/predict.c       2017-06-26 18:39:33.640190953 +0200
> @@ -4031,7 +4031,7 @@ test_prediction_value_range ()
>  {
>    branch_predictor predictors[] = {
>  #include "predict.def"
> -    {NULL, -1}
> +    {NULL, -1U}
>    };
>
>    for (unsigned i = 0; predictors[i].name != NULL; i++)
> --- gcc/dumpfile.c.jj   2017-06-19 08:27:22.000000000 +0200
> +++ gcc/dumpfile.c      2017-06-26 18:38:44.383762664 +0200
> @@ -110,9 +110,9 @@ static const struct dump_option_value_in
>    {"missed", MSG_MISSED_OPTIMIZATION},
>    {"note", MSG_NOTE},
>    {"optall", MSG_ALL},
> -  {"all", ~(TDF_RAW | TDF_SLIM | TDF_LINENO | TDF_GRAPH | TDF_STMTADDR
> -           | TDF_RHS_ONLY | TDF_NOUID | TDF_ENUMERATE_LOCALS | TDF_SCEV
> -           | TDF_GIMPLE)},
> +  {"all", dump_flags_t (~(TDF_RAW | TDF_SLIM | TDF_LINENO | TDF_GRAPH
> +                       | TDF_STMTADDR | TDF_RHS_ONLY | TDF_NOUID
> +                       | TDF_ENUMERATE_LOCALS | TDF_SCEV | TDF_GIMPLE))},
>    {NULL, 0}
>  };
>
> --- gcc/opt-functions.awk.jj    2017-01-08 17:41:19.000000000 +0100
> +++ gcc/opt-functions.awk       2017-06-26 18:54:15.174940306 +0200
> @@ -275,7 +275,7 @@ function var_ref(name, flags)
>                 return "offsetof (struct gcc_options, x_target_flags)"
>         if (opt_args("InverseMask", flags) != "")
>                 return "offsetof (struct gcc_options, x_target_flags)"
> -       return "-1"
> +       return "(unsigned short) -1"
>  }
>
>  # Given the option called NAME return a sanitized version of its name.
> --- gcc/optc-gen.awk.jj 2017-02-25 00:15:02.000000000 +0100
> +++ gcc/optc-gen.awk    2017-06-26 18:55:41.613928361 +0200
> @@ -336,7 +336,7 @@ for (i = 0; i < n_opts; i++) {
>                 alias_posarg = nth_arg(1, alias_arg)
>                 alias_negarg = nth_arg(2, alias_arg)
>
> -               if (var_ref(opts[i], flags[i]) != "-1")
> +               if (var_ref(opts[i], flags[i]) != "(unsigned short) -1")
>                         print "#error Alias setting variable"
>
>                 if (alias_posarg != "" && alias_negarg == "") {
>
>         Jakub

--- gcc/predict.c.jj	2017-06-21 16:53:37.000000000 +0200
+++ gcc/predict.c	2017-06-26 18:39:33.640190953 +0200
@@ -4031,7 +4031,7 @@  test_prediction_value_range ()
 {
   branch_predictor predictors[] = {
 #include "predict.def"
-    {NULL, -1}
+    {NULL, -1U}
   };
 
   for (unsigned i = 0; predictors[i].name != NULL; i++)
--- gcc/dumpfile.c.jj	2017-06-19 08:27:22.000000000 +0200
+++ gcc/dumpfile.c	2017-06-26 18:38:44.383762664 +0200
@@ -110,9 +110,9 @@  static const struct dump_option_value_in
   {"missed", MSG_MISSED_OPTIMIZATION},
   {"note", MSG_NOTE},
   {"optall", MSG_ALL},
-  {"all", ~(TDF_RAW | TDF_SLIM | TDF_LINENO | TDF_GRAPH | TDF_STMTADDR
-	    | TDF_RHS_ONLY | TDF_NOUID | TDF_ENUMERATE_LOCALS | TDF_SCEV
-	    | TDF_GIMPLE)},
+  {"all", dump_flags_t (~(TDF_RAW | TDF_SLIM | TDF_LINENO | TDF_GRAPH
+			| TDF_STMTADDR | TDF_RHS_ONLY | TDF_NOUID
+			| TDF_ENUMERATE_LOCALS | TDF_SCEV | TDF_GIMPLE))},
   {NULL, 0}
 };
 
--- gcc/opt-functions.awk.jj	2017-01-08 17:41:19.000000000 +0100
+++ gcc/opt-functions.awk	2017-06-26 18:54:15.174940306 +0200
@@ -275,7 +275,7 @@  function var_ref(name, flags)
 		return "offsetof (struct gcc_options, x_target_flags)"
 	if (opt_args("InverseMask", flags) != "")
 		return "offsetof (struct gcc_options, x_target_flags)"
-	return "-1"
+	return "(unsigned short) -1"
 }
 
 # Given the option called NAME return a sanitized version of its name.
--- gcc/optc-gen.awk.jj	2017-02-25 00:15:02.000000000 +0100
+++ gcc/optc-gen.awk	2017-06-26 18:55:41.613928361 +0200
@@ -336,7 +336,7 @@  for (i = 0; i < n_opts; i++) {
 		alias_posarg = nth_arg(1, alias_arg)
 		alias_negarg = nth_arg(2, alias_arg)
 
-		if (var_ref(opts[i], flags[i]) != "-1")
+		if (var_ref(opts[i], flags[i]) != "(unsigned short) -1")
 			print "#error Alias setting variable"
 
 		if (alias_posarg != "" && alias_negarg == "") {

Fix some narrowing conversion issues

Commit Message

Comments

Patch