This bug report was migrated from our old Bugzilla tracker.

These attachments are available in the static archive:

• patch (patch_bug_4406_HasNeon.diff, text/plain, 2018-12-01 13:47:25 +0000, 3130 bytes)

Reported in version: don't know
Reported for operating system, platform: Android (All), ARM

Comments on the original bug report:

On 2018-12-01 13:41:54 +0000, Sylvain wrote:

When compiling with armeabi-v7a, SDL_HasNEON() doesn't detect NEON.

In this mode, it uses "readProcAuxvForNeon()" which reads "/proc/self/auxv".
But this is sometimes not accessible.

in ndk cpufeatures it says:

// Parsing /proc/self/auxv will fail from regular application
// processes on some Android platform versions, when this happens
// parse proc/cpuinfo instead.

We should use cpufeatures that also reads /proc/cpuinfo

On 2018-12-01 13:47:25 +0000, Sylvain wrote:

Created attachment 3517
patch

Patch that uses cpu-features.

• I am not 100% sure of the Android.mk (Mine is a little bit different).

• it also fix a comment missing in blendmode.h (-Wdocumentation)

On 2018-12-01 17:20:24 +0000, Ryan C. Gordon wrote:

Patch that uses cpu-features.

This is now https://hg.libsdl.org/SDL/rev/3a0793e5e134

( and the blendmode doxygen comment in that patch is https://hg.libsdl.org/SDL/rev/a107bc9988d1 )

Thanks!

--ryan.