Skip to content

aarch64: use -mtls-size=12 for faster tls reads#22284

Open
henderkes wants to merge 1 commit into
php:masterfrom
henderkes:perf/3-tls-read
Open

aarch64: use -mtls-size=12 for faster tls reads#22284
henderkes wants to merge 1 commit into
php:masterfrom
henderkes:perf/3-tls-read

Conversation

@henderkes

Copy link
Copy Markdown
Contributor

picked out from #22231
will help most with #22278

benchmarked on aarch64-linux-gnu oracle cloud:

buildPHPStan instrPHPStan wallphpbench (no JIT)phpbench (JIT)
NTS (base)840.5827 B68.6s7091681179326
ZTS before (gcc)881.9715 B68.0s6693791130824
ZTS after — variant (gcc)872.8856 B (−1.030%)67.8s (−0.294%)678451 (+1.36%)1130269 (−0.05%)
NTS (base)846.6887 B65.7s7056341141506
ZTS before (clang)898.4707 B68.4s6700861089468
ZTS after — variant (clang)890.2960 B (−0.910%)67.4s (−1.462%)672020 (+0.29%)1091158 (+0.16%)

Initially went with a compile check, but tried cross-compiling for aarch64 and while it passed the compiler flag, the linker failed with unknown relocation (550).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant