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

Serialization of 64-bit floats broken on iOS ARM 32-bit #101

Closed
darknoon opened this issue Aug 7, 2014 · 3 comments
Closed

Serialization of 64-bit floats broken on iOS ARM 32-bit #101

darknoon opened this issue Aug 7, 2014 · 3 comments

Comments

@darknoon
Copy link

darknoon commented Aug 7, 2014

This has to do with the __ARM_EABI__ conditional in unpack_template.h / pack_template.h.

If I set a build setting __ARM_EABI__ = 1, then it works, but this kinda sucks. Is there another way to conditionalize this hack for that platform without messing up the build for iOS?

@redboltz
Copy link
Contributor

Thank you for reporting the problem. The ARM_EABI conditional in unpack_template.h and pack_template.h are required for dispatch the proccess ARM EABI / OABI.

I don't have iOS developing environment. I asked some iOS developers.The macro ARM_EABI is not defined on iOS developing environment. I don't know why, and I think that ARM_EABI should be defined is the platform is EABI. If we could get some different predefined macro for iOS, I would implement using it.

Any ideas?

@rggjan
Copy link

rggjan commented Mar 27, 2015

Maybe a different endianness check like this one would be more robust:
http://stackoverflow.com/a/12792301/369009

Alternatively, a patch along these lines:
https://code.google.com/p/v8/issues/attachmentText?id=1312&aid=13120010000&name=v8-ios.patch&token=704fa2b9305af97f5c33d2aaad55fb30

Might also work:
-#if defined(arm) && !defined(ARM_EABI)
+#if defined(arm) && !defined(APPLE) && !defined(ARM_EABI)

@redboltz
Copy link
Contributor

redboltz commented Aug 6, 2015

This issue has been fixed by #270

@redboltz redboltz closed this as completed Aug 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants