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 avx instruction set option for compiling php and workaround gmp download freeze in russia #173

Closed
wants to merge 1 commit into from

Conversation

fwflunky
Copy link

@fwflunky fwflunky commented Jul 3, 2023

With -mavx option php works faster than with -f option. Tested with simple bench script and multitime tool. (200 times)

Also i wrote workaround for libgmp, because gmplib.org blocked in russia
https://github.com/carbontwelve/php-performance-benchmark/blob/master/bench.php

@dktapps
Copy link
Member

dktapps commented Jul 3, 2023

GMP is needed by the PM core, you can't just remove it. We're looking at setting up a PMMP mirror. gmplib.org got DoSed by GitHub Actions so it's also broken there.

AVX is not used because it's not supported by all target CPUs. If you want AVX support, your best bet is to use compile.sh directly on the target machine and omit the -t option, which will permit the script to use -march=native. However, beware that the resulting binary may not work on anything but the CPU it was built on.

In the future, please do not put multiple changes in one PR. Make a separate PR for each change you want to make.

@dktapps dktapps closed this Jul 3, 2023
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.

2 participants