Page MenuHomeFreeBSD

compiler-rt: add floatditf and floatunditf
ClosedPublic

Authored by emaste on Mar 30 2015, 1:48 PM.
Tags
None
Referenced Files
F103218411: D2174.diff
Fri, Nov 22, 8:27 AM
Unknown Object (File)
Thu, Nov 21, 7:12 AM
Unknown Object (File)
Tue, Nov 19, 1:52 PM
Unknown Object (File)
Fri, Nov 8, 4:48 AM
Unknown Object (File)
Thu, Nov 7, 11:32 PM
Unknown Object (File)
Wed, Nov 6, 12:30 PM
Unknown Object (File)
Wed, Nov 6, 12:30 PM
Unknown Object (File)
Wed, Nov 6, 12:30 PM
Subscribers
None

Details

Summary

These are long integer (di_int/du_int) to quad precision floating point conversions.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

emaste retitled this revision from to compiler-rt: add floatditf and floatunditf.
emaste updated this object.
emaste edited the test plan for this revision. (Show Details)
emaste added reviewers: ed, dim, andrew.
contrib/compiler-rt/lib/builtins/floatditf.c
12 ↗(On Diff #4509)

Out of curiosity, is this also what this function is supposed to do unconditionally? These functions are not supposed to integrate with fegetround() / FLT_ROUNDS?

47 ↗(On Diff #4509)

These functions do not support subnormal floating point numbers. Should this be implemented, or at least documented?

contrib/compiler-rt/lib/builtins/floatditf.c
12 ↗(On Diff #4509)

Good question; this was inherited from the floatsitf function I started from.

The thread I started on llvmdev is here:
http://lists.cs.uiuc.edu/pipermail/llvmdev/2015-March/084064.html

47 ↗(On Diff #4509)

I'm missing something - how would you implement int to subnormal float conversion?

ed edited edge metadata.
This revision is now accepted and ready to land.Mar 31 2015, 7:28 AM

Any plan to commit this? It will be needed for the arm64 buildworld.

emaste updated this revision to Diff 4725.

Closed by commit rS281223 (authored by @emaste).