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

Add the correct build definitions for cross compiling with the Native Client toolchain. #204

Closed
wants to merge 6 commits into from

Conversation

sjlangley
Copy link
Contributor

This adds the build rules so that it is possible to cross compile PHP for native client. It also fixes one place where I did not correctly wrap the asm code with a defined(native_client).

The native client compiler defines the C macro __x86_64__, but the size
of an unsigned int is only 4 bytes. This causes the compile to fail
because the inline asm code compiled is for an 8 byte unsigned int. Add
the correct defines to fix this.

As detailed in:
http://code.google.com/p/nativeclient/issues/detail?id=2255
* PHP-5.4:
  Support building PHP with the native client toolchain.
* 5.4-nacl:
  Support building PHP with the NaCl toolchain.
Use the correct basic_machine for nacl.
More wrapping of inline __asm__ with defined(__native_client__)
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

Successfully merging this pull request may close these issues.

None yet

1 participant