Page MenuHomeFreeBSD

libc/libstand for RISC-V
ClosedPublic

Authored by br on Jan 14 2016, 5:06 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 1, 8:18 AM
Unknown Object (File)
Sat, Nov 1, 3:38 AM
Unknown Object (File)
Sat, Nov 1, 3:38 AM
Unknown Object (File)
Sat, Nov 1, 3:38 AM
Unknown Object (File)
Sat, Nov 1, 3:38 AM
Unknown Object (File)
Fri, Oct 31, 10:39 PM
Unknown Object (File)
Fri, Oct 24, 9:24 PM
Unknown Object (File)
Wed, Oct 22, 3:06 AM
Subscribers

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

br retitled this revision from to libc/libstand for RISC-V.
br updated this object.
br edited the test plan for this revision. (Show Details)
br added reviewers: andrew, emaste.

Replace "This software..." with "Portions of this software..."

andrew added inline comments.
lib/libc/riscv/_fpmath.h
38 ↗(On Diff #12322)

I should check the packing :-)

lib/libc/xdr/xdr_float.c
64–68 ↗(On Diff #12322)

I've been wondering why this exists, all architectures we support (and some we don't) are listed here.

lib/libc/xdr/xdr_float.c
64–68 ↗(On Diff #12322)

I suspect that ppc64 isn't. The code was written by Sun and then not sanitized.

andrew added inline comments.
lib/libc/xdr/xdr_float.c
64–68 ↗(On Diff #12322)

I thought (and @jhibbits confirmed) that __ppc__ was defined when building for ppc64.

lib/libc/xdr/xdr_float.c
64–68 ↗(On Diff #12322)

Agree, the #if should just be removed.

lib/libstand/Makefile
78–81 ↗(On Diff #12322)

perhaps combine aarch64 and riscv case into

.PATH ${LIBC_SRC}/${MACHINE_CPUARCH}/gen
  • arith.h and gd_qnan.h regenerated;
  • architectures check in lib/libc/xdr/xdr_float.c removed;
  • aarch64 and riscv cases in libstand/Makefile combined.
andrew edited edge metadata.
andrew added inline comments.
lib/libc/xdr/xdr_float.c
63–64 ↗(On Diff #12362)

I've built a similar change where I also removed all the Vax code. Without it this file is much simpler. I'm happy with just removing the #if for now.

This revision is now accepted and ready to land.Jan 15 2016, 6:11 PM
This revision was automatically updated to reflect the committed changes.