This is the mail archive of the
libc-ports@sources.redhat.com
mailing list for the libc-ports project.
Re: [PATCH v2] MIPS: MIPS16 support
- From: "Joseph S. Myers" <joseph at codesourcery dot com>
- To: "Maciej W. Rozycki" <macro at codesourcery dot com>
- Cc: <libc-ports at sourceware dot org>, Chung-Lin Tang <cltang at codesourcery dot com>, Steve Ellcey <Steve dot Ellcey at imgtec dot com>, Richard Sandiford <rdsandiford at googlemail dot com>
- Date: Wed, 20 Feb 2013 16:29:46 +0000
- Subject: Re: [PATCH v2] MIPS: MIPS16 support
- References: <alpine.DEB.1.10.1301230313570.4834@tp.orcam.me.uk> <alpine.DEB.1.10.1301230317220.4834@tp.orcam.me.uk> <Pine.LNX.4.64.1301231706590.7102@digraph.polyomino.org.uk> <alpine.DEB.1.10.1302201557510.6762@tp.orcam.me.uk>
On Wed, 20 Feb 2013, Maciej W. Rozycki wrote:
> * sysdeps/mips/fpu_control.h (__fpu_getcw) [__mips16]: New
> prototype.
> (__fpu_setcw) [__mips16]: Likewise.
Since you've renamed the functions to __mips_fpu_getcw and
__mips_fpu_setcw, you need to update the ChangeLog entry accordingly.
The ChangeLog entries may also need updates for other changes in this
patch version.
> * sysdeps/mips/mips32/mips16/fpu/Versions: New file.
Actually you moved the Versions file to the correct place for the symbol
versions not to depend on whether glibc itself is built as MIPS16. But
the contents of the file - and of the libc.abilist changes - need updating
for the new symbol names.
(Given these inconsistencies, I haven't fully reviewed the rest of the
patch contents.)
--
Joseph S. Myers
joseph@codesourcery.com