Page MenuHomeFreeBSD

libc/libstand for RISC-V
ClosedPublic

Authored by br on Jan 14 2016, 5:06 PM.
Tags
None
Referenced Files
F83670791: D4943.id.diff
Mon, May 13, 9:45 AM
F83634755: D4943.id12362.diff
Sun, May 12, 10:35 PM
F83634752: D4943.id12321.diff
Sun, May 12, 10:35 PM
F83615296: D4943.id12322.diff
Sun, May 12, 5:48 PM
F83611156: D4943.diff
Sun, May 12, 4:28 PM
Unknown Object (File)
Fri, Apr 26, 5:28 AM
Unknown Object (File)
Mon, Apr 22, 9:57 AM
Unknown Object (File)
Mon, Apr 22, 9:27 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.