Page MenuHomeFreeBSD

Enable long double tests on RISC-V
ClosedPublic

Authored by mhorne on Jun 23 2020, 8:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 22, 6:09 AM
Unknown Object (File)
Sat, Apr 20, 4:39 AM
Unknown Object (File)
Feb 4 2024, 4:11 PM
Unknown Object (File)
Dec 23 2023, 4:05 AM
Unknown Object (File)
Dec 23 2023, 12:51 AM
Unknown Object (File)
Dec 20 2023, 4:56 AM
Unknown Object (File)
Dec 11 2023, 4:19 AM
Unknown Object (File)
Nov 27 2023, 3:08 PM
Subscribers

Details

Reviewers
ngie
Group Reviewers
riscv
Commits
rS362576: Enable long double tests on RISC-V
Summary

Some of the NetBSD contributed tests are gated behind the
__HAVE_LONG_DOUBLE flag. This flag seems to be defined only for
platforms whose long double is larger than their double. I could not
find this explicitly documented anywhere, but it is implied by the
definitions in NetBSD's sys/arch/${arch}/include/math.h headers, and the
following assertion from the ubsan code:

#ifdef __HAVE_LONG_DOUBLE
    ASSERT(sizeof(LD) > sizeof(uint64_t));
#endif

RISC-V has 128-bit long doubles, so enable the tests on this platform,
and update the comments to better explain the purpose of this flag.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 31907
Build 29459: arc lint + arc unit

Event Timeline

mhorne created this revision.
jrtc27 added inline comments.
lib/libc/tests/stdlib/Makefile
23

This comment wasn't updated

Update comment in tests/stdlib/Makefile.

This revision is now accepted and ready to land.Jun 24 2020, 3:32 AM
This revision was automatically updated to reflect the committed changes.