diff mbox

PR target/48904 x86_64-knetbsd-gnu missing defs

Message ID 1430294510-18361-1-git-send-email-rep.dot.nop@gmail.com
State New
Headers show

Commit Message

Bernhard Reutner-Fischer April 29, 2015, 8:01 a.m. UTC
2012-09-21  H.J. Lu  <hongjiu.lu@intel.com>

	PR target/48904
	* config.gcc (x86_64-*-knetbsd*-gnu): Add i386/knetbsd-gnu64.h.
	* config/i386/knetbsd-gnu64.h: New file

Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
---

This fixes config-list.mk all-gcc for x86_64-knetbsd-gnu.
Ok for trunk?

 gcc/config.gcc                  |    2 +-
 gcc/config/i386/knetbsd-gnu64.h |   27 +++++++++++++++++++++++++++
 2 files changed, 28 insertions(+), 1 deletion(-)
 create mode 100644 gcc/config/i386/knetbsd-gnu64.h

Comments

Jeff Law April 30, 2015, 5 a.m. UTC | #1
On 04/29/2015 02:01 AM, Bernhard Reutner-Fischer wrote:
> 2012-09-21  H.J. Lu  <hongjiu.lu@intel.com>
>
> 	PR target/48904
> 	* config.gcc (x86_64-*-knetbsd*-gnu): Add i386/knetbsd-gnu64.h.
> 	* config/i386/knetbsd-gnu64.h: New file
OK.  Please install on the trunk.

THanks,
Jeff
Bernhard Reutner-Fischer April 30, 2015, 7:58 a.m. UTC | #2
Hi,

On 30 April 2015 at 07:00, Jeff Law <law@redhat.com> wrote:
> On 04/29/2015 02:01 AM, Bernhard Reutner-Fischer wrote:
>>
>> 2012-09-21  H.J. Lu  <hongjiu.lu@intel.com>
>>
>>         PR target/48904
>>         * config.gcc (x86_64-*-knetbsd*-gnu): Add i386/knetbsd-gnu64.h.
>>         * config/i386/knetbsd-gnu64.h: New file
>
> OK.  Please install on the trunk.

hmz, according to https://www.debian.org/ports/netbsd/ the debian
knetbsd port is abandoned since about 2002.
If this is true (please confirm) then we should probably remove knetbsd from
- upstream config repo
- GCC
- binutils-gdb

instead of the above patchlet.
This would work equally well for me WRT config-list.mk builds..
[I should have checked this earlier, sorry..]
>
> THanks,
> Jeff
>
Guillem Jover April 30, 2015, 2:49 p.m. UTC | #3
Hi!

On Thu, 2015-04-30 at 09:58:28 +0200, Bernhard Reutner-Fischer wrote:
> On 30 April 2015 at 07:00, Jeff Law <law@redhat.com> wrote:
> > On 04/29/2015 02:01 AM, Bernhard Reutner-Fischer wrote:
> >>
> >> 2012-09-21  H.J. Lu  <hongjiu.lu@intel.com>
> >>
> >>         PR target/48904
> >>         * config.gcc (x86_64-*-knetbsd*-gnu): Add i386/knetbsd-gnu64.h.
> >>         * config/i386/knetbsd-gnu64.h: New file
> >
> > OK.  Please install on the trunk.
> 
> hmz, according to https://www.debian.org/ports/netbsd/ the debian
> knetbsd port is abandoned since about 2002.

Actually that page refers to the GNU/NetBSD port, which was based on
NetBSD's libc, the GNU/kNetBSD port (based on glibc), was started at
the same time as GNU/kFreeBSD but was short-lived and put into
hibernation to try to focus the effort on GNU/kFreeBSD. But that
has been a very long winter…

> If this is true (please confirm) then we should probably remove knetbsd from
> - upstream config repo
> - GCC
> - binutils-gdb

The difference between removing support in the toolchain and in the
upstream config repo is that for the former it will just need forward
porting the patches, but for the latter it takes a very long time and
lots of prodding to get upstream projects to update to current config.*
scripts. (In Debian we have been switching to always update the config.*
scripts at build time so it would probably not be too bad going forward
for us, but it would when using projects directly from upstream.)

Although I always find it a bit sad to remove ports support, I have the
impression the GNU/kNetBSD port is not coming to live any time soon. So
you might want to wait a bit for comments from others, perhaps someone
has been working on such port that we are not aware of, but otherwise I
think removal would be fine.

Thanks,
Guillem
Jeff Law April 30, 2015, 3:53 p.m. UTC | #4
On 04/30/2015 01:58 AM, Bernhard Reutner-Fischer wrote:
> Hi,
>
> On 30 April 2015 at 07:00, Jeff Law <law@redhat.com> wrote:
>> On 04/29/2015 02:01 AM, Bernhard Reutner-Fischer wrote:
>>>
>>> 2012-09-21  H.J. Lu  <hongjiu.lu@intel.com>
>>>
>>>          PR target/48904
>>>          * config.gcc (x86_64-*-knetbsd*-gnu): Add i386/knetbsd-gnu64.h.
>>>          * config/i386/knetbsd-gnu64.h: New file
>>
>> OK.  Please install on the trunk.
>
> hmz, according to https://www.debian.org/ports/netbsd/ the debian
> knetbsd port is abandoned since about 2002.
> If this is true (please confirm) then we should probably remove knetbsd from
> - upstream config repo
> - GCC
> - binutils-gdb
>
> instead of the above patchlet.
> This would work equally well for me WRT config-list.mk builds..
> [I should have checked this earlier, sorry..]
Given what Guillem indicated, I'd support removal.

It's often the case that we mark it as deprecated and issue an explicit 
error if someone tries to build the port.  That seems wise here.

jeff
Bernhard Reutner-Fischer April 30, 2015, 9:58 p.m. UTC | #5
On April 30, 2015 5:53:02 PM GMT+02:00, Jeff Law <law@redhat.com> wrote:
>On 04/30/2015 01:58 AM, Bernhard Reutner-Fischer wrote:
>> Hi,
>>
>> On 30 April 2015 at 07:00, Jeff Law <law@redhat.com> wrote:
>>> On 04/29/2015 02:01 AM, Bernhard Reutner-Fischer wrote:
>>>>
>>>> 2012-09-21  H.J. Lu  <hongjiu.lu@intel.com>
>>>>
>>>>          PR target/48904
>>>>          * config.gcc (x86_64-*-knetbsd*-gnu): Add
>i386/knetbsd-gnu64.h.
>>>>          * config/i386/knetbsd-gnu64.h: New file
>>>
>>> OK.  Please install on the trunk.
>>
>> hmz, according to https://www.debian.org/ports/netbsd/ the debian
>> knetbsd port is abandoned since about 2002.
>> If this is true (please confirm) then we should probably remove
>knetbsd from
>> - upstream config repo
>> - GCC
>> - binutils-gdb
>>
>> instead of the above patchlet.
>> This would work equally well for me WRT config-list.mk builds..
>> [I should have checked this earlier, sorry..]
>Given what Guillem indicated, I'd support removal.
>
>It's often the case that we mark it as deprecated and issue an explicit
>
>error if someone tries to build the port.  That seems wise here.

I will apply the abovementioned patch ASAP and let somebody else with janitorial spare cleanup cycles propose removal of
*-knetbsd-* then.
Given previous discussion in https://gcc.gnu.org/ml/gcc-patches/2015-04/msg00582.html someone may also be in a position to remove *-openbsd3-* support, FWIW.
I personally cannot take care of these due to limited (essentially non-existing ;) spare time.
Same, BTW for Ultrix support which was officially removed from GCC last cycle, IIRC -- unless config entries are meant to be sticky from a GNU tool chain support POV per design/decision?

Either way, cannot clean these up properly ATM and not familiar with those policies, so punting for now..

Cheers,
>
>jeff
Trevor Saunders April 30, 2015, 11:23 p.m. UTC | #6
On Thu, Apr 30, 2015 at 11:58:09PM +0200, Bernhard Reutner-Fischer wrote:
> On April 30, 2015 5:53:02 PM GMT+02:00, Jeff Law <law@redhat.com> wrote:
> >On 04/30/2015 01:58 AM, Bernhard Reutner-Fischer wrote:
> >> Hi,
> >>
> >> On 30 April 2015 at 07:00, Jeff Law <law@redhat.com> wrote:
> >>> On 04/29/2015 02:01 AM, Bernhard Reutner-Fischer wrote:
> >>>>
> >>>> 2012-09-21  H.J. Lu  <hongjiu.lu@intel.com>
> >>>>
> >>>>          PR target/48904
> >>>>          * config.gcc (x86_64-*-knetbsd*-gnu): Add
> >i386/knetbsd-gnu64.h.
> >>>>          * config/i386/knetbsd-gnu64.h: New file
> >>>
> >>> OK.  Please install on the trunk.
> >>
> >> hmz, according to https://www.debian.org/ports/netbsd/ the debian
> >> knetbsd port is abandoned since about 2002.
> >> If this is true (please confirm) then we should probably remove
> >knetbsd from
> >> - upstream config repo
> >> - GCC
> >> - binutils-gdb
> >>
> >> instead of the above patchlet.
> >> This would work equally well for me WRT config-list.mk builds..
> >> [I should have checked this earlier, sorry..]
> >Given what Guillem indicated, I'd support removal.
> >
> >It's often the case that we mark it as deprecated and issue an explicit
> >
> >error if someone tries to build the port.  That seems wise here.
> 
> I will apply the abovementioned patch ASAP and let somebody else with janitorial spare cleanup cycles propose removal of
> *-knetbsd-* then.
> Given previous discussion in https://gcc.gnu.org/ml/gcc-patches/2015-04/msg00582.html someone may also be in a position to remove *-openbsd3-* support, FWIW.
> I personally cannot take care of these due to limited (essentially non-existing ;) spare time.

I'll try to get to it before the end of stage 1.  Its pretty easy
really, you just add the triple you want to obsolete to the switch in
config.gcc at about line 240, and then wait for the next release.

btw thanks for cleaning up config-list.mk :)

> Same, BTW for Ultrix support which was officially removed from GCC last cycle, IIRC -- unless config entries are meant to be sticky from a GNU tool chain support POV per design/decision?

It looks like most of the ultrix references are in config.guess /
config.sub / configure so from upstream repositories.  THere's a few
references in gcc/doc/ that could probably be cleaned up, and a little
bit of something in fixincludes I think.

Trev

> 
> Either way, cannot clean these up properly ATM and not familiar with those policies, so punting for now..
> 
> Cheers,
> >
> >jeff
> 
>
Bernhard Reutner-Fischer May 8, 2015, 7:37 a.m. UTC | #7
On 1 May 2015 at 01:23, Trevor Saunders <tbsaunde@tbsaunde.org> wrote:
> On Thu, Apr 30, 2015 at 11:58:09PM +0200, Bernhard Reutner-Fischer wrote:
>> On April 30, 2015 5:53:02 PM GMT+02:00, Jeff Law <law@redhat.com> wrote:
>> >On 04/30/2015 01:58 AM, Bernhard Reutner-Fischer wrote:
>> >> Hi,
>> >>
>> >> On 30 April 2015 at 07:00, Jeff Law <law@redhat.com> wrote:
>> >>> On 04/29/2015 02:01 AM, Bernhard Reutner-Fischer wrote:
>> >>>>
>> >>>> 2012-09-21  H.J. Lu  <hongjiu.lu@intel.com>
>> >>>>
>> >>>>          PR target/48904
>> >>>>          * config.gcc (x86_64-*-knetbsd*-gnu): Add
>> >i386/knetbsd-gnu64.h.
>> >>>>          * config/i386/knetbsd-gnu64.h: New file
>> >>>
>> >>> OK.  Please install on the trunk.

Applied to trunk as r222903
Thanks,
Bernhard Reutner-Fischer Dec. 2, 2015, 8:45 p.m. UTC | #8
Trevor,

On 1 May 2015 at 01:23, Trevor Saunders <tbsaunde@tbsaunde.org> wrote:
> On Thu, Apr 30, 2015 at 11:58:09PM +0200, Bernhard Reutner-Fischer wrote:
>> On April 30, 2015 5:53:02 PM GMT+02:00, Jeff Law <law@redhat.com> wrote:
>> >On 04/30/2015 01:58 AM, Bernhard Reutner-Fischer wrote:
>> >> Hi,
>> >>
>> >> On 30 April 2015 at 07:00, Jeff Law <law@redhat.com> wrote:
>> >>> On 04/29/2015 02:01 AM, Bernhard Reutner-Fischer wrote:
>> >>>>
>> >>>> 2012-09-21  H.J. Lu  <hongjiu.lu@intel.com>
>> >>>>
>> >>>>          PR target/48904
>> >>>>          * config.gcc (x86_64-*-knetbsd*-gnu): Add
>> >i386/knetbsd-gnu64.h.
>> >>>>          * config/i386/knetbsd-gnu64.h: New file
>> >>>
>> >>> OK.  Please install on the trunk.
>> >>
>> >> hmz, according to https://www.debian.org/ports/netbsd/ the debian
>> >> knetbsd port is abandoned since about 2002.
>> >> If this is true (please confirm) then we should probably remove
>> >knetbsd from
>> >> - upstream config repo
>> >> - GCC
>> >> - binutils-gdb
>> >>
>> >> instead of the above patchlet.
>> >> This would work equally well for me WRT config-list.mk builds..
>> >> [I should have checked this earlier, sorry..]
>> >Given what Guillem indicated, I'd support removal.
>> >
>> >It's often the case that we mark it as deprecated and issue an explicit
>> >
>> >error if someone tries to build the port.  That seems wise here.
>>
>> I will apply the abovementioned patch ASAP and let somebody else with janitorial spare cleanup cycles propose removal of
>> *-knetbsd-* then.
>> Given previous discussion in https://gcc.gnu.org/ml/gcc-patches/2015-04/msg00582.html someone may also be in a position to remove *-openbsd3-* support, FWIW.
>> I personally cannot take care of these due to limited (essentially non-existing ;) spare time.
>
> I'll try to get to it before the end of stage 1.  Its pretty easy
> really, you just add the triple you want to obsolete to the switch in
> config.gcc at about line 240, and then wait for the next release.
>
> btw thanks for cleaning up config-list.mk :)
>
>> Same, BTW for Ultrix support which was officially removed from GCC last cycle, IIRC -- unless config entries are meant to be sticky from a GNU tool chain support POV per design/decision?
>
> It looks like most of the ultrix references are in config.guess /
> config.sub / configure so from upstream repositories.  THere's a few
> references in gcc/doc/ that could probably be cleaned up, and a little
> bit of something in fixincludes I think.
>
> Trev
>
>>
>> Either way, cannot clean these up properly ATM and not familiar with those policies, so punting for now..
>>
>> Cheers,
>> >
>> >jeff
>>

GCC-6 stage1 now ended. Can you take care of some of these
deprecations now, please?
TIA,
diff mbox

Patch

diff --git a/gcc/config.gcc b/gcc/config.gcc
index a1df043..77ef473 100644
--- a/gcc/config.gcc
+++ b/gcc/config.gcc
@@ -1517,7 +1517,7 @@  x86_64-*-linux* | x86_64-*-kfreebsd*-gnu | x86_64-*-knetbsd*-gnu)
 		tm_file="${tm_file} kfreebsd-gnu.h i386/kfreebsd-gnu64.h"
 		;;
 	x86_64-*-knetbsd*-gnu)
-		tm_file="${tm_file} knetbsd-gnu.h"
+		tm_file="${tm_file} knetbsd-gnu.h i386/knetbsd-gnu64.h"
 		;;
 	esac
 	tmake_file="${tmake_file} i386/t-linux64"
diff --git a/gcc/config/i386/knetbsd-gnu64.h b/gcc/config/i386/knetbsd-gnu64.h
new file mode 100644
index 0000000..d621bbe
--- /dev/null
+++ b/gcc/config/i386/knetbsd-gnu64.h
@@ -0,0 +1,27 @@ 
+/* Definitions for AMD x86-64 running kNetBSD-based GNU systems with ELF format
+   Copyright (C) 2012
+   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/>.  */
+
+#define GNU_USER_LINK_EMULATION32 "elf_i386"
+#define GNU_USER_LINK_EMULATION64 "elf_x86_64"
+#define GNU_USER_LINK_EMULATIONX32 "elf32_x86_64"
+
+#define GNU_USER_DYNAMIC_LINKER32 "/lib/ld.so.1"
+#define GNU_USER_DYNAMIC_LINKER64 "/lib/ld-knetbsd-x86-64.so.1"
+#define GNU_USER_DYNAMIC_LINKERX32 "/lib/ld-knetbsd-x32.so.1"