Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

setMathControlState is not implemented on Mac, Linux, or X64 Windows #1179

Open
jamesu opened this issue Feb 10, 2015 · 5 comments
Open

setMathControlState is not implemented on Mac, Linux, or X64 Windows #1179

jamesu opened this issue Feb 10, 2015 · 5 comments
Labels

Comments

@jamesu
Copy link
Contributor

jamesu commented Feb 10, 2015

Platform::setMathControlState does not appear to be implemented for Mac, Linux, or X64 Windows. This function turns out to be very important as otherwise you will get weird rounding errors on floating point operations during execution.

@jamesu
Copy link
Contributor Author

jamesu commented Feb 10, 2015

Still not implemented it for x64 windows, though you should be able to use the same asm. Unless there is something blocking this?

@crabmusket
Copy link
Contributor

Nothing blocking :).

@crabmusket crabmusket added this to the 3.7 milestone Feb 10, 2015
@LuisAntonRebollo
Copy link
Contributor

@jamesu, we really need to handle this FPU changes?

We use a flag for avoid DX9 change FPU precission, and new DX or OpenGL dont change precission.

@jamesu
Copy link
Contributor Author

jamesu commented Feb 15, 2015

@LuisAntonRebollo yes, since the state of the FPU is not guaranteed across platforms (AFAIK). For instance on linux we had big problems with the precision of a float in an AI calculation causing a crash. Best to be explicit IMO ;)

@LuisAntonRebollo
Copy link
Contributor

Ok,thx for info :)

Re-opened for future research.

@crabmusket crabmusket removed this from the 3.7 milestone Feb 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants