Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 22 Oct 2016 15:43:01 +0300
From:      Konstantin Belousov <kostikbel@gmail.com>
To:        Justin Hibbits <jhibbits@FreeBSD.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r307761 - in head: . contrib/binutils/bfd contrib/gcc/config/rs6000 gnu/lib/libgcc gnu/lib/libgomp gnu/usr.bin/binutils gnu/usr.bin/cc gnu/usr.bin/cc/cc_tools gnu/usr.bin/cc/include gnu...
Message-ID:  <20161022124301.GZ54029@kib.kiev.ua>
In-Reply-To: <20161022122017.GY54029@kib.kiev.ua>
References:  <201610220157.u9M1vFXx080331@repo.freebsd.org> <20161022122017.GY54029@kib.kiev.ua>

index | next in thread | previous in thread | raw e-mail

On Sat, Oct 22, 2016 at 03:20:17PM +0300, Konstantin Belousov wrote:
> On Sat, Oct 22, 2016 at 01:57:15AM +0000, Justin Hibbits wrote:
> > Author: jhibbits
> > Date: Sat Oct 22 01:57:15 2016
> > New Revision: 307761
> > URL: https://svnweb.freebsd.org/changeset/base/307761
> > 
> > Log:
> >   Create a new MACHINE_ARCH for Freescale PowerPC e500v2
> 
> > Added:
> >   head/lib/libc/powerpcspe/sys/ptrace.S   (contents, props changed)
> 
> This is not needed, libc provides MI implementation in libc/sys/ptrace.c.

Why the decision to copy whole lib/libc/powerpc to lib/libc/powerpcspe
was used to create the architecture ? I did a primitive-science
overview, and got total ~2300 lines in lib/libc/powerpcspe, while
$ diff -Nrua lib/libc/powerpc lib/libc/powerpcspe | diffstat -p0 | tail -1
 16 files changed, 240 insertions(+), 174 deletions(-)

It seems to be possible to only create several new files, like
*setjmp.S and fp*.c, and reuse existing stock files from libc/powerpc
for anything else.

I am the one who does most of the sweeping changes over libc, and
I am somewhat saddened by addition of almost identical copy of the
existing architecture files which need to be maintained together with
the original.


home | help

Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20161022124301.GZ54029>