Differential D29386

RISC-V: Fix feenableexcept return value
ClosedPublic
Actions

Authored by arichardson on Tue, Mar 23, 9:23 AM.

Details

Reviewers

mhorne

Group Reviewers

riscv

Commits

R10:0eafcef4552c: RISC-V: Fix feenableexcept return value
R10:dd5ed53a2f93: RISC-V: Fix feenableexcept return value

Summary

The man page says "The feenableexcept(), fedisableexcept(), and
fegetexcept() functions return a bitmap of the exceptions that were
unmasked prior to the call.", so we should return zero not -1.

Diff Detail

Repository

R10 FreeBSD src repository

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

arichardson created this revision.Tue, Mar 23, 9:23 AM

Herald added a subscriber: imp. · View Herald TranscriptTue, Mar 23, 9:23 AM

Harbormaster completed remote builds in B38002: Diff 86179.Tue, Mar 23, 9:23 AM

arichardson requested review of this revision.Tue, Mar 23, 9:23 AM

I've wondered about this case before, but this solution seems correct.

How well do we do running the fenv tests with this change (and your other recent commits) ?

This revision is now accepted and ready to land.Tue, Mar 23, 1:12 PM

In D29386#658222, @mhorne wrote:

I've wondered about this case before, but this solution seems correct.

How well do we do running the fenv tests with this change (and your other recent commits) ?

This fixes one of the fenv tests. There are more failures that I need to look into.

Closed by commit R10:dd5ed53a2f93: RISC-V: Fix feenableexcept return value (authored by arichardson). · Explain WhyThu, Mar 25, 12:08 PM

This revision was automatically updated to reflect the committed changes.

arichardson added a commit: R10:dd5ed53a2f93: RISC-V: Fix feenableexcept return value.

arichardson added a commit: R10:0eafcef4552c: RISC-V: Fix feenableexcept return value.Sat, Apr 10, 1:30 PM