diff mbox

Delete files unused since openbsd 2 & 3 support was dropped.

Message ID 1500949765-3577-1-git-send-email-jim.wilson@linaro.org
State Accepted
Commit df7c5241f8aff0ffeca5e356e8f92960d2177289
Headers show

Commit Message

Jim Wilson July 25, 2017, 2:29 a.m. UTC 
Trevor Saunders deleted the x86 openbsd 2 & 3 support here.
    https://gcc.gnu.org/ml/gcc-patches/2016-06/msg01368.html
With that change, there are now 3 unused files that he missed.  I noticed
the gstabs.h file while looking at debugging info related files, and then
did a consistency check and noticed the other two.

Jeff Law preapproved the gstabs.h change, but I still need approval for the
other two files.

OK?

Jim

	gcc/
	* config/i386/gstabs.h: Delete.
	* config/i386/openbsd.h, config/i386/t-openbsd: Likewise.
---
 gcc/config/i386/gstabs.h  |   7 ----
 gcc/config/i386/openbsd.h | 101 ----------------------------------------------
 gcc/config/i386/t-openbsd |   4 --
 3 files changed, 112 deletions(-)
 delete mode 100644 gcc/config/i386/gstabs.h
 delete mode 100644 gcc/config/i386/openbsd.h
 delete mode 100644 gcc/config/i386/t-openbsd

-- 
2.7.4

Comments

Jeff Law July 25, 2017, 5:04 a.m. UTC | #1 
On 07/24/2017 08:29 PM, Jim Wilson wrote:
> Trevor Saunders deleted the x86 openbsd 2 & 3 support here.

>     https://gcc.gnu.org/ml/gcc-patches/2016-06/msg01368.html

> With that change, there are now 3 unused files that he missed.  I noticed

> the gstabs.h file while looking at debugging info related files, and then

> did a consistency check and noticed the other two.

> 

> Jeff Law preapproved the gstabs.h change, but I still need approval for the

> other two files.

> 

> OK?

> 

> Jim

> 

> 	gcc/

> 	* config/i386/gstabs.h: Delete.

> 	* config/i386/openbsd.h, config/i386/t-openbsd: Likewise.

OK.  FWIW, I think removing an unused config file like this would fall
under the obvious policy.  So no need to wait for approval on this kind
of change.

jeff
diff mbox

Patch

diff --git a/gcc/config/i386/gstabs.h b/gcc/config/i386/gstabs.h
deleted file mode 100644
index e9a6218..0000000
--- a/gcc/config/i386/gstabs.h
+++ /dev/null
@@ -1,7 +0,0 @@ 
-/* We do not want to output SDB debugging information.  */
-
-#undef SDB_DEBUGGING_INFO
-
-/* We want to output DBX debugging information.  */
-
-#define DBX_DEBUGGING_INFO 1
diff --git a/gcc/config/i386/openbsd.h b/gcc/config/i386/openbsd.h
deleted file mode 100644
index 7d37c0d..0000000
--- a/gcc/config/i386/openbsd.h
+++ /dev/null
@@ -1,101 +0,0 @@ 
-/* Configuration for an OpenBSD i386 target.
-   Copyright (C) 1999-2017 Free Software Foundation, Inc.
-
-This file is part of GCC.
-
-GCC is free software; you can redistribute it and/or modify
-it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 3, or (at your option)
-any later version.
-
-GCC is distributed in the hope that it will be useful,
-but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-GNU General Public License for more details.
-
-You should have received a copy of the GNU General Public License
-along with GCC; see the file COPYING3.  If not see
-<http://www.gnu.org/licenses/>.  */
-
-
-/* This goes away when the math-emulator is fixed */
-#undef TARGET_SUBTARGET_DEFAULT
-#define TARGET_SUBTARGET_DEFAULT \
-  (MASK_80387 | MASK_IEEE_FP | MASK_FLOAT_RETURNS | MASK_NO_FANCY_MATH_387)
-
-#define TARGET_OS_CPP_BUILTINS()		\
-  do						\
-    {						\
-	builtin_define ("__unix__");		\
-	builtin_define ("__OpenBSD__");		\
-	builtin_assert ("system=unix");		\
-	builtin_assert ("system=bsd");		\
-	builtin_assert ("system=OpenBSD");	\
-    }						\
-  while (0)
-
-/* Layout of source language data types.  */
-
-/* This must agree with <machine/ansi.h> */
-#undef SIZE_TYPE
-#define SIZE_TYPE "unsigned int"
-
-#undef PTRDIFF_TYPE
-#define PTRDIFF_TYPE "int"
-
-#undef WCHAR_TYPE
-#define WCHAR_TYPE "int"
-
-#undef WCHAR_TYPE_SIZE
-#define WCHAR_TYPE_SIZE 32
-
-/* Assembler format: overall framework.  */
-
-#undef ASM_APP_ON
-#define ASM_APP_ON "#APP\n"
-
-#undef ASM_APP_OFF
-#define ASM_APP_OFF "#NO_APP\n"
-
-/* Stack & calling: aggregate returns.  */
-
-/* Don't default to pcc-struct-return, because gcc is the only compiler, and
-   we want to retain compatibility with older gcc versions.  */
-#define DEFAULT_PCC_STRUCT_RETURN 0
-
-/* Assembler format: alignment output.  */
-
-/* Kludgy test: when gas is upgraded, it will have p2align, and no problems
-   with nops.  */
-#ifndef HAVE_GAS_MAX_SKIP_P2ALIGN
-/* i386 OpenBSD still uses an older gas that doesn't insert nops by default
-   when the .align directive demands to insert extra space in the text
-   segment.  */
-#undef ASM_OUTPUT_ALIGN
-#define ASM_OUTPUT_ALIGN(FILE,LOG) \
-  if ((LOG)!=0) fprintf ((FILE), "\t.align %d,0x90\n", (LOG))
-#endif
-
-/* Stack & calling: profiling.  */
-
-/* OpenBSD's profiler recovers all information from the stack pointer.
-   The icky part is not here, but in machine/profile.h.  */
-#undef FUNCTION_PROFILER
-#define FUNCTION_PROFILER(FILE, LABELNO)  \
-  fputs (flag_pic ? "\tcall mcount@PLT\n": "\tcall mcount\n", FILE);
-
-/* Assembler format: exception region output.  */
-
-/* All configurations that don't use elf must be explicit about not using
-   dwarf unwind information.  */
-#define DWARF2_UNWIND_INFO 0
-
-#undef ASM_PREFERRED_EH_DATA_FORMAT
-
-#undef ASM_COMMENT_START
-#define ASM_COMMENT_START ";#"
-
-/* OpenBSD gas currently does not support quad, so do not use it.  */
-#undef ASM_QUAD
-
-#define TARGET_HAVE_NAMED_SECTIONS false
diff --git a/gcc/config/i386/t-openbsd b/gcc/config/i386/t-openbsd
deleted file mode 100644
index 4f8ff65..0000000
--- a/gcc/config/i386/t-openbsd
+++ /dev/null
@@ -1,4 +0,0 @@ 
-# gdb gets confused if pic code is linked with non pic
-# We cope by building variants of libgcc.
-MULTILIB_OPTIONS = fpic
-MULTILIB_MATCHES=fpic=fPIC