Skip to content

Build opus with NEON intrinsics on ARM (Android ABIs included) #3833

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
mcfnord wants to merge 2 commits into
base: main
Choose a base branch
from
+15 −6
Open

Build opus with NEON intrinsics on ARM (Android ABIs included) #3833

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
8f774fc
Build opus with NEON intrinsics on non-Android 64-bit ARM (#2806)
Jul 24, 2026
cbe7d44
Jamulus.pro: one ARM case for the opus NEON build, NE10 files dropped
Sep 5, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 15 additions & 6 deletions Jamulus.pro
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -668,9 +668,7 @@ SOURCES_OPUS_ARM = libs/opus/celt/arm/armcpu.c \
libs/opus/silk/arm/NSQ_del_dec_neon_intr.c \
libs/opus/silk/arm/NSQ_neon.c \
libs/opus/celt/arm/celt_neon_intr.c \
libs/opus/celt/arm/pitch_neon_intr.c \
libs/opus/celt/arm/celt_fft_ne10.c \
libs/opus/celt/arm/celt_mdct_ne10.c
libs/opus/celt/arm/pitch_neon_intr.c

SOURCES_OPUS_X86_SSE = libs/opus/celt/x86/x86cpu.c \
libs/opus/celt/x86/x86_celt_map.c \
Expand All @@ -685,11 +683,16 @@ SOURCES_OPUS_X86_SSE4 = libs/opus/celt/x86/celt_lpc_sse4_1.c \
libs/opus/silk/x86/VAD_sse4_1.c \
libs/opus/silk/x86/VQ_WMat_EC_sse4_1.c

contains(QT_ARCH, armeabi-v7a) | contains(QT_ARCH, arm64-v8a) {
contains(QT_ARCH, armeabi-v7a) | contains(QT_ARCH, arm64-v8a) | contains(QT_ARCH, arm64) {
# armeabi-v7a and arm64-v8a are the Android ABI names; plain arm64 is what Qt
# reports for 64-bit ARM everywhere else (Apple Silicon macOS, iOS, Linux aarch64).
# OPUS_ARM_MAY_HAVE_NEON_INTR is required in addition to the PRESUME defines:
# opus gates the inclusion of its arm/*.h headers on it
# (see libs/opus/celt/pitch.h and libs/opus/celt/cpu_support.h).
HEADERS_OPUS += $$HEADERS_OPUS_ARM
SOURCES_OPUS_ARCH += $$SOURCES_OPUS_ARM
DEFINES_OPUS += OPUS_ARM_PRESUME_NEON=1 OPUS_ARM_PRESUME_NEON_INTR=1
contains(QT_ARCH, arm64-v8a):DEFINES_OPUS += OPUS_ARM_PRESUME_AARCH64_NEON_INTR
DEFINES_OPUS += OPUS_ARM_MAY_HAVE_NEON_INTR=1 OPUS_ARM_PRESUME_NEON=1 OPUS_ARM_PRESUME_NEON_INTR=1
contains(QT_ARCH, arm64-v8a) | contains(QT_ARCH, arm64):DEFINES_OPUS += OPUS_ARM_PRESUME_AARCH64_NEON_INTR
} else:contains(QT_ARCH, x86) | contains(QT_ARCH, x86_64) {
HEADERS_OPUS += $$HEADERS_OPUS_X86
SOURCES_OPUS_ARCH += $$SOURCES_OPUS_X86_SSE $$SOURCES_OPUS_X86_SSE2 $$SOURCES_OPUS_X86_SSE4
Expand Down Expand Up @@ -1172,6 +1175,12 @@ contains(CONFIG, "opus_shared_lib") {
sse4_cc.variable_out = OBJECTS
QMAKE_EXTRA_COMPILERS += sse_cc sse2_cc sse4_cc
}
} else:contains(QT_ARCH, armeabi-v7a) | contains(QT_ARCH, arm64-v8a) | contains(QT_ARCH, arm64) {
# Unlike the x86 SSE files above, the NEON intrinsics need no special
# compiler flags: NEON is part of the base AArch64 ISA, and the NDK's
# clang enables it by default for armv7a-linux-androideabi. So they
# can be compiled like any other source file.
SOURCES += $$SOURCES_OPUS_ARCH
}
}

Expand Down
Loading