Removing -frename-registers from -O3

Bernd Schmidt bernds@redhat.com
Fri Dec 22 07:51:00 GMT 2000


Richard,

how do you feel about this patch?  IMO -frename-registers is too harmful
to debugging to include it by default on any target (except ia64 for
which it's uncommonly useful).

Maybe we ought to add some entries to TORTURE_OPTIONS, though.


Bernd

Index: toplev.c
===================================================================
RCS file: /cvs/gcc/egcs/gcc/toplev.c,v
retrieving revision 1.406
diff -u -p -r1.406 toplev.c
--- toplev.c	2000/12/17 14:35:05	1.406
+++ toplev.c	2000/12/22 15:49:11
@@ -4653,7 +4653,6 @@ main (argc, argv)
   if (optimize >= 3)
     {
       flag_inline_functions = 1;
-      flag_rename_registers = 1;
     }

   if (optimize < 2 || optimize_size)
Index: config/ia64/ia64.c
===================================================================
RCS file: /cvs/gcc/egcs/gcc/config/ia64/ia64.c,v
retrieving revision 1.63
diff -u -p -r1.63 ia64.c
--- ia64.c	2000/11/17 22:18:25	1.63
+++ ia64.c	2000/12/22 15:49:12
@@ -3666,6 +3674,9 @@ ia64_override_options ()
   if (ia64_fixed_range_string)
     fix_range (ia64_fixed_range_string);

+  if (optimize >= 3)
+    flag_rename_registers = 1;
+
   ia64_section_threshold = g_switch_set ? g_switch_value : IA64_DEFAULT_GVALUE;

   init_machine_status = ia64_init_machine_status;



More information about the Gcc-patches mailing list