Page MenuHomeFreeBSD
Differential D22526

[PowerPC] Fix floating point environment manipulation
ClosedPublic
Actions

Authored by bdragon on Nov 24 2019, 12:56 AM.
Tags
Referenced Files
Unknown Object (File)
Mon, Dec 8, 10:00 PM
Unknown Object (File)
Sun, Dec 7, 6:29 PM
Unknown Object (File)
Sat, Dec 6, 3:53 AM
Unknown Object (File)
Sat, Dec 6, 3:53 AM
Unknown Object (File)
Fri, Dec 5, 12:34 AM
Unknown Object (File)
Sun, Nov 30, 7:14 PM
Unknown Object (File)
Sun, Nov 30, 2:21 AM
Unknown Object (File)
Sat, Nov 29, 3:06 PM
View All Files
Subscribers
alfredo
andrew
emaste
imp
luporl

Details

Reviewers
jhibbits
Commits
rGd905133e26d0: [PowerPC] Fix SPE floating point environment manipulation
rS355656: [PowerPC] Fix SPE floating point environment manipulation
Summary
  • Fix endianness problems in functions accessing and manipulating the floating point environment via fenv.h. (jhibbits wrote the original bits)
  • Fix bugs regarding synchronization requirements after writing SPEFSCR.

Diff Detail

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

Event Timeline

bdragon created this revision.Nov 24 2019, 12:56 AM
Herald added a subscriber: imp. · View Herald TranscriptNov 24 2019, 12:56 AM
jhibbits added a comment.Nov 24 2019, 1:00 AM

You found the bugs with the kyua tests, did you retest post-changes?

lib/msun/powerpc/fenv.h
94–95 ↗(On Diff #64789)

I think these need line wrapped.

bdragon updated this revision to Diff 65517.Dec 12 2019, 12:31 AM

Linewrap fenv.h macros.

Herald added a reviewer: manu. · View Herald TranscriptDec 12 2019, 12:31 AM
Herald added subscribers: alfredo, luporl, emaste, andrew. · View Herald Transcript
bdragon added a comment.Dec 12 2019, 12:33 AM

ugh, think that was the wrong diff. Moment.

bdragon updated this revision to Diff 65518.Dec 12 2019, 12:35 AM

Actually upload the *correct* diff this time

bdragon removed a reviewer: manu.Dec 12 2019, 12:36 AM
bdragon marked an inline comment as done.
jhibbits accepted this revision.Dec 12 2019, 4:47 PM
This revision is now accepted and ready to land.Dec 12 2019, 4:47 PM
Closed by commit rS355656: [PowerPC] Fix SPE floating point environment manipulation (authored by bdragon). · Explain WhyDec 12 2019, 5:12 PM
This revision was automatically updated to reflect the committed changes.
bdragon added a commit: rS355656: [PowerPC] Fix SPE floating point environment manipulation.
bdragon added a commit: rGd905133e26d0: [PowerPC] Fix SPE floating point environment manipulation.Mar 8 2021, 7:08 PM

Revision Contents
Changeset List

PathSize
head/
lib/
libc/
powerpcspe/
gen/
2 lines
2 lines
msun/
powerpc/
57 lines
sys/
powerpc/
booke/
2 lines

Diff 65553

View Options

head/lib/libc/powerpcspe/gen/fpsetmask.c

Loading...
View Options

head/lib/libc/powerpcspe/gen/fpsetround.c

Loading...
View Options

head/lib/msun/powerpc/fenv.h

Loading...
View Options

head/sys/powerpc/booke/spe.c

Loading...