Re: CVS commit: src/lib/libc_fp

2016-07-15 Thread Matt Thomas

> On Jul 14, 2016, at 11:39 PM, David Holland 
>  wrote:
> 
> Can you call this MTC1_HAZARD like the MFC0_HAZARD that got added
> recently elsewhere? It's a pipeline hazard and not a synchronization
> (which to me at least indicates some form of handshake) and it's also
> specific to mtc1/dmtc1.

That's reasonable.

> (also while this code appears to currently be 64-bit only, in the long
> run the condition is wrong...)

Well, O32 handles FP emulation.

So this is really for N32/N64 since they default to softfloat.  They use
64-bit registers so I don't see the point.

Re: CVS commit: src/lib/libc_fp

2016-07-15 Thread David Holland
On Thu, Jul 14, 2016 at 01:59:19AM +, Matt Thomas wrote:
 > Added Files:
 >  src/lib/libc_fp: Makefile gcc-softfloat.c
 >  src/lib/libc_fp/arch/mips: Makefile.inc fpdf.S fpsf.S shlib_version
 > 
 > Log Message:
 > Library which implements the softfloat primitives using FP instructions
 > Currently contains only untested MIPS routines.
 > XXX move arm libc_vfp to here.
 > 
 > [...]
 > #ifdef MIPS3
 > #defineCOP1_SYNC   nop
 > #else
 > #defineCOP1_SYNC
 > #endif

Can you call this MTC1_HAZARD like the MFC0_HAZARD that got added
recently elsewhere? It's a pipeline hazard and not a synchronization
(which to me at least indicates some form of handshake) and it's also
specific to mtc1/dmtc1.

(also while this code appears to currently be 64-bit only, in the long
run the condition is wrong...)

-- 
David A. Holland
dholl...@netbsd.org