public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH, i386]: Some trivial cleanups
@ 2017-05-06  7:12 Uros Bizjak
  0 siblings, 0 replies; only message in thread
From: Uros Bizjak @ 2017-05-06  7:12 UTC (permalink / raw)
  To: gcc-patches

[-- Attachment #1: Type: text/plain, Size: 340 bytes --]

2017-05-06  Uros Bizjak  <ubizjak@gmail.com>

    * config/i386/i386.c (ext_80387_constant_init): Do not explicitly
    initialize to zero.
    (init_regs): Remove declaration.
    (function_arg_advance_32): Initialize error_p as boolean variable.

Bootstrapped and regression tested on x86_64-linux-gnu.

Committed to mainline SVN.

Uros.

[-- Attachment #2: p.diff.txt --]
[-- Type: text/plain, Size: 1703 bytes --]

diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
index 9c429de..0d27113 100644
--- a/gcc/config/i386/i386.c
+++ b/gcc/config/i386/i386.c
@@ -2569,7 +2569,7 @@ enum x86_64_reg_class
 
 /* Table of constants used by fldpi, fldln2, etc....  */
 static REAL_VALUE_TYPE ext_80387_constants_table [5];
-static bool ext_80387_constants_init = 0;
+static bool ext_80387_constants_init;
 
 \f
 static struct machine_function * ix86_init_machine_status (void);
@@ -8544,9 +8544,6 @@ ix86_asm_output_function_label (FILE *asm_out_file, const char *fname,
     }
 }
 
-/* regclass.c  */
-extern void init_regs (void);
-
 /* Implementation of call abi switching target hook. Specific to FNDECL
    the specific call register sets are set.  See also
    ix86_conditional_register_usage for more details.  */
@@ -9665,7 +9662,7 @@ function_arg_advance_32 (CUMULATIVE_ARGS *cum, machine_mode mode,
 			 HOST_WIDE_INT words)
 {
   int res = 0;
-  bool error_p = NULL;
+  bool error_p = false;
 
   if (TARGET_IAMCU)
     {
@@ -9710,13 +9707,13 @@ pass_in_reg:
 
     case DFmode:
       if (cum->float_in_sse == -1)
-	error_p = 1;
+	error_p = true;
       if (cum->float_in_sse < 2)
 	break;
       /* FALLTHRU */
     case SFmode:
       if (cum->float_in_sse == -1)
-	error_p = 1;
+	error_p = true;
       if (cum->float_in_sse < 1)
 	break;
       /* FALLTHRU */
@@ -9978,13 +9975,13 @@ pass_in_reg:
 
     case DFmode:
       if (cum->float_in_sse == -1)
-	error_p = 1;
+	error_p = true;
       if (cum->float_in_sse < 2)
 	break;
       /* FALLTHRU */
     case SFmode:
       if (cum->float_in_sse == -1)
-	error_p = 1;
+	error_p = true;
       if (cum->float_in_sse < 1)
 	break;
       /* FALLTHRU */

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2017-05-06  7:03 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-06  7:12 [PATCH, i386]: Some trivial cleanups Uros Bizjak

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).