public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
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

      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).