From: John David Anglin <dave.anglin@bell.net>
To: coypu@sdf.org, gcc-patches@gcc.gnu.org, law@redhat.com
Subject: Re: [patch] Add NetBSD/hppa target
Date: Wed, 31 Jul 2019 14:26:00 -0000 [thread overview]
Message-ID: <bc8c231e-dfa9-e7f6-fec2-53933e9b1209@bell.net> (raw)
In-Reply-To: <20190614154446.GB20055@SDF.ORG>
Committed as revision 273933.
Dave
On 2019-06-14 11:44 a.m., coypu@sdf.org wrote:
> This adds netbsd/hppa support. I tested it on the shiny new QEMU-git
> which can now boot NetBSD too :-)
>
> Files are very similar to the linux code.
>
> Please let me know if any changes need to be made.
>
> Matt Thomas <matt@3am-software.com>
> Nick Hudson <nick@nthcliff.demon.co.uk>
> Matthew Green <mrg@eterna.com.au>
> Maya Rashish <coypu@sdf.org>
>
> gcc/ChangeLog:
> config.gcc (hppa*-*-netbsd*): New target.
> config/pa/pa-netbsd.h: New file.
> config/pa/pa32-netbsd.h: New file.
>
> libgcc/ChangeLog:
> config.host (hppa*-*-netbsd*): New case.
> config/pa/t-netbsd: New file.
>
> ---
> gcc/config.gcc | 8 +++
> gcc/config/pa/pa-netbsd.h | 137 ++++++++++++++++++++++++++++++++++++
> gcc/config/pa/pa32-netbsd.h | 37 ++++++++++
> libgcc/config.host | 3 +
> libgcc/config/pa/t-netbsd | 9 +++
> 5 files changed, 194 insertions(+)
> create mode 100644 gcc/config/pa/pa-netbsd.h
> create mode 100644 gcc/config/pa/pa32-netbsd.h
> create mode 100644 libgcc/config/pa/t-netbsd
>
> diff --git a/gcc/config.gcc b/gcc/config.gcc
> index 76bb316942d..ba93bb41ec8 100644
> --- a/gcc/config.gcc
> +++ b/gcc/config.gcc
> @@ -1481,6 +1481,14 @@ hppa*-*-openbsd*)
> gas=yes
> gnu_ld=yes
> ;;
> +hppa*-*-netbsd*)
> + target_cpu_default="MASK_PA_11|MASK_NO_SPACE_REGS"
> + tm_file="${tm_file} dbxelf.h elfos.h ${nbsd_tm_file} \
> + pa/pa-netbsd.h pa/pa32-regs.h pa/pa32-netbsd.h"
> + tmake_file="${tmake_file}"
> + tm_defines="${tm_defines} CHAR_FAST8=1 SHORT_FAST16=1"
> + extra_options="${extra_options} netbsd.opt netbsd-elf.opt"
> + ;;
> hppa[12]*-*-hpux10*)
> case ${target} in
> hppa1.1-*-* | hppa2*-*-*)
> diff --git a/gcc/config/pa/pa-netbsd.h b/gcc/config/pa/pa-netbsd.h
> new file mode 100644
> index 00000000000..88790987561
> --- /dev/null
> +++ b/gcc/config/pa/pa-netbsd.h
> @@ -0,0 +1,137 @@
> +/* Definitions for PA_RISC with ELF format
> + Copyright (C) 1999-2019 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/>. */
> +
> +
> +#undef TARGET_OS_CPP_BUILTINS
> +#define TARGET_OS_CPP_BUILTINS() \
> + do \
> + { \
> + NETBSD_OS_CPP_BUILTINS_ELF(); \
> + builtin_assert ("machine=bigendian"); \
> + } \
> + while (0)
> +
> +#undef CPP_SPEC
> +#define CPP_SPEC NETBSD_CPP_SPEC
> +
> +#undef ASM_SPEC
> +#define ASM_SPEC \
> + "%{v:-V} %{n} %{T} %{Ym,*} %{Yd,*} %{Wa,*:%*}"
> +
> +#undef EXTRA_SPECS
> +#define EXTRA_SPECS NETBSD_SUBTARGET_EXTRA_SPECS
> +#undef SUBTARGET_EXTRA_SPECS
> +
> +#define NETBSD_ENTRY_POINT "__start"
> +
> +#undef LINK_SPEC
> +#define LINK_SPEC NETBSD_LINK_SPEC_ELF
> +
> +/* NetBSD profiling functions don't need gcc to allocate counters. */
> +#define NO_DEFERRED_PROFILE_COUNTERS 1
> +
> +/* Define the strings used for the special svr4 .type and .size directives.
> + These strings generally do not vary from one system running svr4 to
> + another, but if a given system (e.g. m88k running svr) needs to use
> + different pseudo-op names for these, they may be overridden in the
> + file which includes this one. */
> +
> +#undef STRING_ASM_OP
> +#define STRING_ASM_OP "\t.stringz\t"
> +
> +#define TEXT_SECTION_ASM_OP "\t.text"
> +#define DATA_SECTION_ASM_OP "\t.data"
> +#define BSS_SECTION_ASM_OP "\t.section\t.bss"
> +
> +#define TARGET_ASM_FILE_START pa_linux_file_start
> +
> +/* We want local labels to start with period if made with asm_fprintf. */
> +#undef LOCAL_LABEL_PREFIX
> +#define LOCAL_LABEL_PREFIX "."
> +
> +/* Define these to generate the Linux/ELF/SysV style of internal
> + labels all the time - i.e. to be compatible with
> + ASM_GENERATE_INTERNAL_LABEL in <elfos.h>. Compare these with the
> + ones in pa.h and note the lack of dollar signs in these. FIXME:
> + shouldn't we fix pa.h to use ASM_GENERATE_INTERNAL_LABEL instead? */
> +
> +#undef ASM_OUTPUT_ADDR_VEC_ELT
> +#define ASM_OUTPUT_ADDR_VEC_ELT(FILE, VALUE) \
> + fprintf (FILE, "\t.word .L%d\n", VALUE)
> +
> +#undef ASM_OUTPUT_ADDR_DIFF_ELT
> +#define ASM_OUTPUT_ADDR_DIFF_ELT(FILE, BODY, VALUE, REL) \
> + fprintf (FILE, "\t.word .L%d-.L%d\n", VALUE, REL)
> +
> +/* Use the default. */
> +#undef ASM_OUTPUT_LABEL
> +
> +/* NOTE: (*targetm.asm_out.internal_label)() is defined for us by elfos.h, and
> + does what we want (i.e. uses colons). It must be compatible with
> + ASM_GENERATE_INTERNAL_LABEL(), so do not define it here. */
> +
> +/* Use the default. */
> +#undef ASM_OUTPUT_INTERNAL_LABEL
> +
> +/* Use the default. */
> +#undef TARGET_ASM_GLOBALIZE_LABEL
> +/* Globalizing directive for a label. */
> +#define GLOBAL_ASM_OP ".globl "
> +
> +/* FIXME: Hacked from the <elfos.h> one so that we avoid multiple
> + labels in a function declaration (since pa.c seems determined to do
> + it differently) */
> +
> +#undef ASM_DECLARE_FUNCTION_NAME
> +#define ASM_DECLARE_FUNCTION_NAME(FILE, NAME, DECL) \
> + do \
> + { \
> + ASM_OUTPUT_TYPE_DIRECTIVE (FILE, NAME, "function"); \
> + ASM_DECLARE_RESULT (FILE, DECL_RESULT (DECL)); \
> + } \
> + while (0)
> +
> +/* As well as globalizing the label, we need to encode the label
> + to ensure a plabel is generated in an indirect call. */
> +
> +#undef ASM_OUTPUT_EXTERNAL_LIBCALL
> +#define ASM_OUTPUT_EXTERNAL_LIBCALL(FILE, FUN) \
> + do \
> + { \
> + if (!FUNCTION_NAME_P (XSTR (FUN, 0))) \
> + pa_encode_label (FUN); \
> + (*targetm.asm_out.globalize_label) (FILE, XSTR (FUN, 0)); \
> + } \
> + while (0)
> +
> +/* NetBSD always uses gas. */
> +#undef TARGET_GAS
> +#define TARGET_GAS 1
> +
> +/* Use long int for these type to make hppa64 compatibility easier. */
> +#undef SIZE_TYPE
> +#define SIZE_TYPE "long unsigned int"
> +
> +#undef PTRDIFF_TYPE
> +#define PTRDIFF_TYPE "long int"
> +
> +#if 0
> +#undef TARGET_SYNC_LIBCALL
> +#define TARGET_SYNC_LIBCALL 1
> +#endif
> diff --git a/gcc/config/pa/pa32-netbsd.h b/gcc/config/pa/pa32-netbsd.h
> new file mode 100644
> index 00000000000..29837ce6f12
> --- /dev/null
> +++ b/gcc/config/pa/pa32-netbsd.h
> @@ -0,0 +1,37 @@
> +/* Definitions for PA_RISC with ELF-32 format
> + Copyright (C) 2000-2019 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 2, 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 COPYING. If not, write to
> +the Free Software Foundation, 59 Temple Place - Suite 330,
> +Boston, MA 02111-1307, USA. */
> +
> +/* Turn off various SOM crap we don't want. */
> +#undef TARGET_ELF32
> +#define TARGET_ELF32 1
> +
> +/* The libcall __canonicalize_funcptr_for_compare is referenced in
> + crtend.o and the reference isn't resolved in objects that don't
> + compare function pointers. Thus, we need to play games to provide
> + a reference in crtbegin.o. The rest of the define is the same
> + as that in crtstuff.c */
> +#define CTOR_LIST_BEGIN \
> + asm (".type __canonicalize_funcptr_for_compare,@function\n" \
> +" .text\n" \
> +" .word __canonicalize_funcptr_for_compare-$PIC_pcrel$0"); \
> + STATIC func_ptr __CTOR_LIST__[1] \
> + __attribute__ ((__used__, section(".ctors"), \
> + aligned(sizeof(func_ptr)))) \
> + = { (func_ptr) (-1) }
> diff --git a/libgcc/config.host b/libgcc/config.host
> index 91fed5fa473..602fc97a321 100644
> --- a/libgcc/config.host
> +++ b/libgcc/config.host
> @@ -616,6 +616,9 @@ hppa[12]*-*-hpux11*)
> hppa*-*-openbsd*)
> tmake_file="$tmake_file pa/t-openbsd"
> ;;
> +hppa*-*-netbsd*)
> + tmake_file="$tmake_file pa/t-netbsd"
> + ;;
> i[34567]86-*-darwin*)
> tmake_file="$tmake_file i386/t-crtpc t-crtfm i386/t-msabi"
> tm_file="$tm_file i386/darwin-lib.h"
> diff --git a/libgcc/config/pa/t-netbsd b/libgcc/config/pa/t-netbsd
> new file mode 100644
> index 00000000000..8b99068ce42
> --- /dev/null
> +++ b/libgcc/config/pa/t-netbsd
> @@ -0,0 +1,9 @@
> +#Plug millicode routines into libgcc.a We want these on both native and
> +#cross compiles. We use the "64-bit" routines because the "32-bit" code
> +#is broken for certain corner cases.
> +LIB1ASMSRC = pa/milli64.S
> +LIB1ASMFUNCS = _divI _divU _remI _remU _div_const _mulI _dyncall
> +
> +HOST_LIBGCC2_CFLAGS += -DELF=1 -DLINUX=1
> +
> +LIB2ADD = $(srcdir)/config/pa/fptr.c
--
John David Anglin dave.anglin@bell.net
prev parent reply other threads:[~2019-07-31 14:15 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-06-14 15:44 coypu
2019-06-14 16:02 ` Jeff Law
2019-06-14 17:32 ` John David Anglin
2019-06-25 7:57 ` coypu
2019-06-25 14:45 ` Jeff Law
2019-07-31 14:26 ` John David Anglin [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=bc8c231e-dfa9-e7f6-fec2-53933e9b1209@bell.net \
--to=dave.anglin@bell.net \
--cc=coypu@sdf.org \
--cc=gcc-patches@gcc.gnu.org \
--cc=law@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).