Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(361)

Issue 5990056: runtime: SIGILL based floating point emulation *PREVIEW*

Can't Edit
Can't Publish+Mail
Start Review
Created:
14 years, 6 months ago by minux1
Modified:
14 years, 6 months ago
Reviewers:
dave
Visibility:
Public.

Description

runtime: SIGILL based floating point emulation *PREVIEW* WIP code. The implementation details are quick-n-dirty. Requires CL 5987063 and CL 5969064 on ARMv5 (don't set GOARM=5). ChangeLog: PS3: fixed math.sqrtC reentry problem, now math.Sqrt works. PS4: fixed wrong stack frame for stepflt(), I hope everything is ok now. PS5: minor code cleanup, added some comments, and removed _isflt(). still can't pass pkg math and math/rand tests (endless loop). TODO: 1. fix special cases in sqrt() in softfloat64.go, also need to clean up the code a little, should we consider move the math.sqrt to here? 2. rename stepflt (in runtime.h, softfloat_arm.c).

Patch Set 1 #

Patch Set 2 : diff -r 912f98f898d4 https://code.google.com/p/go #

Patch Set 3 : diff -r 912f98f898d4 https://code.google.com/p/go #

Patch Set 4 : diff -r 5e1544310d03 https://code.google.com/p/go/ #

Patch Set 5 : diff -r 5e1544310d03 https://code.google.com/p/go/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+295 lines, -9 lines) Patch
M src/pkg/runtime/runtime.h View 1 2 3 4 2 chunks +4 lines, -0 lines 0 comments Download
M src/pkg/runtime/runtime.c View 1 2 3 5 chunks +6 lines, -6 lines 0 comments Download
M src/pkg/runtime/signal_linux_arm.c View 1 2 3 4 2 chunks +23 lines, -0 lines 0 comments Download
M src/pkg/runtime/softfloat64.go View 1 2 2 chunks +140 lines, -0 lines 0 comments Download
M src/pkg/runtime/softfloat_arm.c View 1 2 3 4 3 chunks +122 lines, -3 lines 0 comments Download

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b