Skip to content

Conversation

@ptondereau
Copy link
Member

@ptondereau ptondereau commented Dec 18, 2025

Description

bindgen doesn't support calling the preserve_none convention that has been introduced in the last PHP 8.5.1.
We can't exclude it with Clang Builder nor passing CFLAGS because this convention is used everywhere in the PHP src code.

I've forked bindgen + clang-sys to be able to build an extension on arm64 with macOS, but we clearly need an update from these repositories.

PHP issue: php/php-src#20546

@coveralls
Copy link

coveralls commented Dec 18, 2025

Pull Request Test Coverage Report for Build 20397299813

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 36.015%

Totals Coverage Status
Change from base Build 20344194219: 0.0%
Covered Lines: 1652
Relevant Lines: 4587

💛 - Coveralls

@ptondereau ptondereau force-pushed the fix/clang-arm64-preserve-none branch 8 times, most recently from 5057152 to fa17044 Compare December 20, 2025 15:55
@ptondereau ptondereau force-pushed the fix/clang-arm64-preserve-none branch from fa17044 to 25c6a0f Compare December 20, 2025 16:27
@ptondereau ptondereau marked this pull request as ready for review December 20, 2025 17:22
@ptondereau
Copy link
Member Author

@Xenira if you have a better move...

@ptondereau ptondereau merged commit 38c763f into master Dec 21, 2025
65 checks passed
@ptondereau ptondereau deleted the fix/clang-arm64-preserve-none branch December 21, 2025 13:11
@Xenira Xenira mentioned this pull request Dec 18, 2025
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.

3 participants