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
F169135542: D22526.id64789.diff
Mon, Aug 31, 1:25 PM
F169122947: D22526.id65518.diff
Mon, Aug 31, 11:57 AM
F169110577: D22526.id65517.diff
Mon, Aug 31, 10:55 AM
Unknown Object (File)
Mon, Aug 31, 12:36 AM
Unknown Object (File)
Sun, Aug 30, 10:25 PM
Unknown Object (File)
Sun, Aug 30, 5:43 PM
Unknown Object (File)
Sat, Aug 29, 1:26 PM
Unknown Object (File)
Sat, Aug 29, 12:04 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 Skipped
Unit
Tests Skipped

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–98

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
lib/
libc/
powerpcspe/
gen/
2 lines
2 lines
msun/
powerpc/
57 lines
sys/
powerpc/
booke/
2 lines

Diff 65518

View Options

lib/libc/powerpcspe/gen/fpsetmask.c

Loading...
View Options

lib/libc/powerpcspe/gen/fpsetround.c

Loading...
View Options

lib/msun/powerpc/fenv.h

Loading...
View Options

sys/powerpc/booke/spe.c

Loading...