This bug report was migrated from our old Bugzilla tracker.

These attachments are available in the static archive:

• patch (patch_bug_4409_NEON.diff, text/plain, 2018-12-03 08:54:08 +0000, 835 bytes)

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

Comments on the original bug report:

On 2018-12-03 08:51:32 +0000, Sylvain wrote:

(at least) on Android ndk-r18:

if you compile with arm64-v8a, ARM_NEON isn't defined (so SDL2 doesn't include "<arm_neon.h>"), but __ARM_NEON is.

on arm-v7a, if you enable neon with Android.mk, both get defined.

https://patchwork.ozlabs.org/patch/434847/
Seems that ARM_NEON is legacy and we should use __ARM_NEON

On 2018-12-03 08:54:08 +0000, Sylvain wrote:

Created attachment 3518
patch

patch

On 2018-12-04 11:40:29 +0000, Sylvain wrote:

Fixed in http://hg.libsdl.org/SDL/rev/17cb4ff70507

On 2018-12-04 15:55:19 +0000, Sylvain wrote:

Marked as fixed.