php 7.1.3 gettype(-1) returns "double" instead of "integer" when built with clang

[vdb]

Android package php 7.1.3. Command php -r "echo gettype(-1);" returns ''double", should be "integer". This breaks programs that rely on correct type value. I've checked php7.1.3 amd64 package on ubuntu and return value is "integer" as expected.

[vishalbiswas]

For me, it returns integer as expected

[Grimler91]

It works as expected on my arm device. Which arch are you using?

[vdb]

uname -a:
Linux localhost 3.18.20-ge45c102 #1 SMP PREEMT aarch64 Android

[vdb]

CPU: Qualcomm Snapdragon 430 MSM8937

[mawatech]

Same for me:

$ uname -a
Linux localhost 3.18.19+ #1 SMP PREEMPT Tue Dec 6 15:46:33 CST 2016 aarch64 Android
$ php -r "echo gettype(-1);"
double$
$

[fornwall]

Hm, so it seems to be a problem at least for 64-bit arm, and working on 32-bit arm.

[Neo-Oli]

That's really annoying. It means that functions like substr($str,-1); don't work, which breaks a hell of a lot of programs.

[Neo-Oli]

I tried running Ubuntu on aarch64 (using proot on termux) with version 7.0.15 and it works as expected returning integer. I compiled the same version of php for termux and the issue remains. So either it's a termux specific issue or debian/ubuntu have a fix in their arm64 package.

[its-pointless]

compiling with gcc fixes this on arm64
as in build.sh add line TERMUX_PKG_CLANG=no

[Neo-Oli]

I tried building it with gcc, and it does indeed fix the problem.

[its-pointless]

The tricky bit now is getting a test case or something so the ndk guys can fix it.
https://github.com/android-ndk/ndk etc ...

[fornwall]

Wow, building with clang causes this? Interesting!

[fornwall]

The 7.1.5 update of the php package is built with gcc and should have the problem fixed.

We still need to find the reason why the problem occurs when built with clang as gcc is planned to be removed from the NDK, but we can build with gcc for now to have a working package, and await further updated PHP and NDK versions to see if they fix the problem.

[fornwall]

Just tried again with PHP v7.1.6 and clang from NDK r15 - still same problem.

[its-pointless]

Time to post issue on ndk ?

[fornwall]

This turned out to be a bug in php (inline aarch64 assembly) fixed in php/php-src@8c86792.

This fix has been backported to the newly released 7.1.7 version of the php package, which is now built with clang.