Patchwork [toplevel] remove empty cases

login
register
mail settings
Submitter Paolo Bonzini
Date March 24, 2011, 5:13 p.m.
Message ID <4D8B7BBB.1050206@redhat.com>
Download mbox | patch
Permalink /patch/88251/
State New
Headers show

Comments

Paolo Bonzini - March 24, 2011, 5:13 p.m.
This enables omit-frame-pointer for i386-solaris.  I honestly haven't 
bootstrapped it, but I don't expect any problems (and if they appear, 
it's much better to fix them ;).

Committed to gcc and (shortly) to src.

Paolo
2011-03-24  Paolo Bonzini  <bonzini@gnu.org>

	* configure.ac: Remove empty cases.
	* configure: Regenerate.
Joseph S. Myers - March 24, 2011, 5:18 p.m.
On Thu, 24 Mar 2011, Paolo Bonzini wrote:

> This enables omit-frame-pointer for i386-solaris.  I honestly haven't
> bootstrapped it, but I don't expect any problems (and if they appear, it's
> much better to fix them ;).

How about killing mh-x86omitfp?  The x86 compiler defaults to 
-fomit-frame-pointer on most platforms now anyway.
Rainer Orth - March 24, 2011, 5:23 p.m.
"Joseph S. Myers" <joseph@codesourcery.com> writes:

> On Thu, 24 Mar 2011, Paolo Bonzini wrote:
>
>> This enables omit-frame-pointer for i386-solaris.  I honestly haven't
>> bootstrapped it, but I don't expect any problems (and if they appear, it's
>> much better to fix them ;).
>
> How about killing mh-x86omitfp?  The x86 compiler defaults to 
> -fomit-frame-pointer on most platforms now anyway.

I'd prefer that: Solaris 10+/x86 defaults to -fomit-frame-pointer for
improved debuggability (cf. gcc/config/sol2-10.h:
USE_IX86_FRAME_POINTER, USE_X86_64_FRAME_POINTER), so it would be better
leave this to the individual platforms instead.

	Rainer

Patch

Index: configure.ac
===================================================================
--- configure.ac	(revision 171416)
+++ configure.ac	(working copy)
@@ -1050,8 +1050,6 @@  case "${host}" in
   *-interix*)
     host_makefile_frag="config/mh-interix"
     ;;
-  *-*-solaris2*)
-    ;;
   hppa*-hp-hpux10*)
     host_makefile_frag="config/mh-pa-hpux10"
     ;;
@@ -1061,8 +1059,6 @@  case "${host}" in
   hppa*-*)	
     host_makefile_frag="config/mh-pa"
     ;;
-  *-hp-hpux*)
-    ;;
   *-*-darwin*)
     host_makefile_frag="config/mh-darwin"
     ;;
@@ -1072,8 +1068,6 @@  case "${host}" in
   rs6000-*-aix*)
     host_makefile_frag="config/mh-ppc-aix"
     ;;
-  *-*-lynxos*)
-    ;;
   # This is placed last to prevent interfering with the cases above.
   i[[3456789]]86-*-*)
     # Build the stage2 and stage3 compilers with -fomit-frame-pointer.
Index: configure
===================================================================
--- configure	(revision 171416)
+++ configure	(working copy)
@@ -3623,8 +3623,6 @@  fi
   *-interix*)
     host_makefile_frag="config/mh-interix"
     ;;
-  *-*-solaris2*)
-    ;;
   hppa*-hp-hpux10*)
     host_makefile_frag="config/mh-pa-hpux10"
     ;;
@@ -3634,8 +3632,6 @@  fi
   hppa*-*)
     host_makefile_frag="config/mh-pa"
     ;;
-  *-hp-hpux*)
-    ;;
   *-*-darwin*)
     host_makefile_frag="config/mh-darwin"
     ;;
@@ -3645,8 +3641,6 @@  fi
   rs6000-*-aix*)
     host_makefile_frag="config/mh-ppc-aix"
     ;;
-  *-*-lynxos*)
-    ;;
   # This is placed last to prevent interfering with the cases above.
   i[3456789]86-*-*)
     # Build the stage2 and stage3 compilers with -fomit-frame-pointer.