Skip to content

Conversation

@gustavotemple
Copy link

This adds architecture-specific support for the ppc64 of the following functions below:

  • zend_mm_high_bit
  • zend_mm_low_bit

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The added part should be inside the or options.

#elif defined(GNUC) && (defined(arm) || defined(aarch64) || defined(powerpc64))

Otherwise on powerpc64 there will be no GNUC check.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, done.

This adds architecture-specific support for the ppc64 of the following functions below:
* zend_mm_high_bit
* zend_mm_low_bit
@remicollet
Copy link
Member

This PR is open against master, but don't apply there.
Code have change a lot, can you please confirm ?

@gustavotemple
Copy link
Author

@remicollet, yes, you are right, the commands bsf, bsfl, bsr and bsrl were removed, thanks.

kaplanlior pushed a commit to kaplanlior/php-src that referenced this pull request Apr 13, 2015
This adds architecture-specific support for the ppc64 of the following functions below:
* zend_mm_high_bit
* zend_mm_low_bit

Taken from php#737
kaplanlior pushed a commit to kaplanlior/php-src that referenced this pull request Apr 19, 2015
This adds architecture-specific support for the ppc64 of the following functions below:
* zend_mm_high_bit
* zend_mm_low_bit

Taken from php#737
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants