This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] OpenBSD/amd64 support and OpenBSD/i386 cleanup
- From: Richard Henderson <rth at redhat dot com>
- To: Gerald Pfeifer <gerald at pfeifer dot com>
- Cc: Uros Bizjak <ubizjak at gmail dot com>, Jan Hubicka <jh at suse dot cz>, gcc-patches at gcc dot gnu dot org, Mark Kettenis <mark dot kettenis at xs4all dot nl>
- Date: Thu, 30 Aug 2012 07:52:00 -0700
- Subject: Re: [PATCH] OpenBSD/amd64 support and OpenBSD/i386 cleanup
- References: <201201011222.q01CMqiC015145@glazunov.sibelius.xs4all.nl> <alpine.LNX.2.00.1208261702300.5727@ghan.fvgr>
On 08/26/2012 08:07 AM, Gerald Pfeifer wrote:
> If anyone wants to approve this, I volunteer to commit the patch.
>
> Gerald
>
>
> libgcc/:
>
> 2011-12-27 Mark Kettenis <kettenis@openbsd.org>
>
> * config.host (x86_64-*-openbsd*): New target.
>
> gcc/:
>
> 2011-12-27 Mark Kettenis <kettenis@openbsd.org>
>
> * config.gcc (x86_64-*-openbsd*): New target.
> * config/openbsd.h (TARGET_C99_FUNCTIONS): Define.
> * config/i386/openbsdelf.h: Remove some superfluous defines and
> group things together in a more logical fashion.
> (DBX_REGISTER_NUMBER): Provide a
> definition that works on both 32-bit and 64-bit targets.
> (WCHAR_TYPE_SIZE): Hardcode as 32.
> (NO_DOLLAR_IN_LABEL): Remove undef.
> (TARGET_DEFAULT): Remove.
> (SET_ASM_OP): Remove.
> (DEFAULT_PCC_STRUCT_RETURN): Undef first to prevent warning.
> (ASM_OUTPUT_MAX_SKIP_ALIGN): Synch with x86-64.h
> (DWARF2_UNWIND_INFO): Remove define.
> (HAVE_ENABLE_EXECUTE_STACK): Define.
> * config/host-openbsd.c: New file.
> * config/t-openbsd (USER_H): Add EXTRA_HEADERS.
> * config/x-openbsd: New file.
Looks ok.
Some of the defines in i386/openbsdelf.h look redundant with either
i386/gas.h or i386/x86_64.h or both. But I won't quibble about that
since there are other files in i386/ that ought to get cleaned up
for the same reasons.
r~