Page MenuHomeFreeBSD

libc/libstand for RISC-V
ClosedPublic

Authored by br on Jan 14 2016, 5:06 PM.
Tags
None
Referenced Files
F96791720: D4943.diff
Thu, Sep 26, 3:20 PM
Unknown Object (File)
Wed, Sep 25, 8:18 PM
Unknown Object (File)
Tue, Sep 24, 4:56 PM
Unknown Object (File)
Tue, Sep 24, 1:27 PM
Unknown Object (File)
Sun, Sep 22, 12:33 PM
Unknown Object (File)
Sun, Sep 22, 5:07 AM
Unknown Object (File)
Sat, Sep 21, 3:08 PM
Unknown Object (File)
Sat, Sep 21, 2:23 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.