Fix ZTS build for Alpine (musl libc)#8180
Conversation
|
@cmb69 please see the 2nd commit it seems the JIT/ASM code use the 2nd commit fixes the ZTS build issues on Alpine, however if such fix is correct (I am not quite sure), then the condition must be exactly the same in all places which is not the case, compare: Line 150 in abd9529 and Line 739 in abd9529 and php-src/ext/opcache/jit/zend_jit_x86.dasc Line 2977 in abd9529 |
902cacf to
abd9529
Compare
|
@dstogov, you are the author of TSRM/JIT code, can you please review this PR? |
|
based on docker-library/php#1076 (comment) comment, this patch is tested to be working on Alpine/musl |
|
Looks good. Please merge. |
fix #8160
credits go to @cmb69, thanks!
discussion 804420b#r68129969