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  <kette...@openbsd.org>
> 
>       * config.host (x86_64-*-openbsd*): New target.
> 
> gcc/:
> 
> 2011-12-27  Mark Kettenis  <kette...@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~

Reply via email to