joakim.tjernlund at transmode.se
Wed Nov 15 23:35:30 UTC 2006
> -----Original Message-----
> From: uclibc-bounces at uclibc.org
> [mailto:uclibc-bounces at uclibc.org] On Behalf Of Joakim Tjernlund
> Sent: den 15 november 2006 09:00
> To: kraj at mvista.com; uclibc at uclibc.org
> Subject: RE: Thumb patches
> > -----Original Message-----
> > From: uclibc-bounces at uclibc.org
> > [mailto:uclibc-bounces at uclibc.org] On Behalf Of Khem Raj
> > Sent: 10 November 2006 19:34
> > To: uclibc at uclibc.org
> > Subject: Thumb patches
> > Hi
> > These two patches are required to let uclibc compile and boot
> > in thumb
> > mode. The patches were mainly written by John Bowler. I have
> > kept them
> > alive
> > on the current trunk. I have been testing them for quite a
> > while without
> > regression.
> > Can someone review them so that they can be committed.
> > Thanks
> I belive this is already in SVN, but I don't like the changes to
> dl_hash(). It is a hack and I think it should be removed.
> Discussion about this is in
> and on the ML.
Sorry for that somewhat confusing reply. However I have now looked a the
code in _dl_find_hash:
1) You removed the type_class in
if (type_class & (sym->st_shndx == SHN_UNDEF))
for all archs and that will probably break something.
2) You also removed DL_RELOC_ADDR macro which will break some archs(FRV?)
3) The other arm #ifdef code is to support very old binutils/gcc, right?
That doesn't belong in a new uClibc code.
This is crap code and should be removed.
More information about the uClibc