diff --git a/.github/workflows/android.yml b/.github/workflows/android.yml index 469b94256c1..0a62eff83fa 100644 --- a/.github/workflows/android.yml +++ b/.github/workflows/android.yml @@ -27,7 +27,7 @@ jobs: runs-on: ubuntu-22.04 strategy: matrix: - arch: ['armv7-linux-androideabi', 'i686-linux-android'] + arch: ['armv7-linux-androideabi', 'i686-linux-android', 'x86_64-linux-android'] steps: - uses: actions/checkout@v4 if: github.event_name != 'issue_comment' && github.event_name != 'pull_request_target' diff --git a/python/servo/command_base.py b/python/servo/command_base.py index 63b61b07bf0..74de8850164 100644 --- a/python/servo/command_base.py +++ b/python/servo/command_base.py @@ -588,6 +588,19 @@ class CommandBase(object): def to_ndk_bin(prog): return path.join(llvm_toolchain, "bin", prog) + # This workaround is due to an issue in the x86_64 Android NDK that introduces + # an undefined reference to the symbol '__extendsftf2'. + # See https://github.com/termux/termux-packages/issues/8029#issuecomment-1369150244 + if "x86_64" in self.cross_compile_target: + libclangrt_filename = subprocess.run( + [to_ndk_bin(f"x86_64-linux-android{android_api}-clang"), "--print-libgcc-file-name"], + check=True, + capture_output=True, + encoding="utf8" + ).stdout + env['RUSTFLAGS'] = env.get('RUSTFLAGS', "") + env["RUSTFLAGS"] += f"-C link-arg={libclangrt_filename}" + env["RUST_TARGET"] = self.cross_compile_target env['HOST_CC'] = host_cc env['HOST_CXX'] = host_cxx @@ -912,6 +925,14 @@ class CommandBase(object): self.config["android"]["lib"] = "x86" self.config["android"]["toolchain_name"] = "i686-linux-android30" return True + elif target == "x86_64-linux-android": + self.config["android"]["platform"] = "android-30" + self.config["android"]["target"] = target + self.config["android"]["toolchain_prefix"] = target + self.config["android"]["arch"] = "x86_64" + self.config["android"]["lib"] = "x86_64" + self.config["android"]["toolchain_name"] = "x86_64-linux-android30" + return True return False def ensure_bootstrapped(self): diff --git a/python/servo/package_commands.py b/python/servo/package_commands.py index 54afdd6a9e4..dca00cc8ae7 100644 --- a/python/servo/package_commands.py +++ b/python/servo/package_commands.py @@ -146,6 +146,8 @@ class PackageCommands(CommandBase): arch_string = "Armv7" elif "i686" in android_target: arch_string = "x86" + elif "x86_64" in android_target: + arch_string = "x64" else: arch_string = "Arm" diff --git a/support/android/apk/build.gradle b/support/android/apk/build.gradle index c2619efb978..12f0318fb1c 100644 --- a/support/android/apk/build.gradle +++ b/support/android/apk/build.gradle @@ -27,7 +27,8 @@ ext.getRustTarget = { String arch -> switch (arch.toLowerCase()) { case 'armv7' : return 'armv7-linux-androideabi' case 'arm64' : return 'aarch64-linux-android' - case 'x86' : return 'i686-linux-android' + case 'x86': return 'i686-linux-android' + case 'x64': return 'x86_64-linux-android' default: throw new GradleException("Invalid target architecture " + arch) } } @@ -36,7 +37,8 @@ ext.getNDKAbi = { String arch -> switch (arch.toLowerCase()) { case 'armv7' : return 'armeabi-v7a' case 'arm64' : return 'arm64-v8a' - case 'x86' : return 'x86' + case 'x86': return 'x86' + case 'x64': return 'x86_64' default: throw new GradleException("Invalid target architecture " + arch) } } diff --git a/support/android/apk/jni/Application.mk b/support/android/apk/jni/Application.mk index 5b349f06d44..2a6cbd50ea7 100644 --- a/support/android/apk/jni/Application.mk +++ b/support/android/apk/jni/Application.mk @@ -2,4 +2,7 @@ NDK_TOOLCHAIN_VERSION := clang APP_MODULES := c++_shared servojni APP_PLATFORM := android-30 APP_STL := c++_shared -APP_ABI := armeabi-v7a x86 +APP_ABI := armeabi-v7a x86 x86_64 +ifeq ($(NDK_DEBUG),1) + APP_STRIP_MODE := none +endif diff --git a/support/android/apk/servoapp/build.gradle b/support/android/apk/servoapp/build.gradle index 6e1cb514d26..88f989bfdd8 100644 --- a/support/android/apk/servoapp/build.gradle +++ b/support/android/apk/servoapp/build.gradle @@ -97,6 +97,18 @@ android { abiFilters getNDKAbi('x86') } } + x64Debug { + initWith(debug) + ndk { + abiFilters getNDKAbi('x64') + } + } + x64Release { + initWith(release) + ndk { + abiFilters getNDKAbi('x64') + } + } } // Ignore default 'debug' and 'release' build types diff --git a/support/android/apk/servoview/build.gradle b/support/android/apk/servoview/build.gradle index 5898bb7bca2..f0511735fc5 100644 --- a/support/android/apk/servoview/build.gradle +++ b/support/android/apk/servoview/build.gradle @@ -76,6 +76,12 @@ android { x86Release { initWith(release) } + x64Debug { + initWith(debug) + } + x64Release { + initWith(release) + } } sourceSets { @@ -99,6 +105,12 @@ android { x86Release { jniLibs.srcDirs = [getJniLibsPath(false, 'x86')] } + x64Debug { + jniLibs.srcDirs = [getJniLibsPath(true, 'x64')] + } + x64Release { + jniLibs.srcDirs = [getJniLibsPath(false, 'x64')] + } } // Ignore default 'debug' and 'release' build types @@ -183,7 +195,7 @@ dependencies { ] // Iterate all build types and dependencies // For each dependency call the proper implementation command and set the correct dependency path - def list = ['armv7', 'arm64', 'x86'] + def list = ['armv7', 'arm64', 'x86', 'x64'] for (arch in list) { for (debug in [true, false]) { String basePath = getTargetDir(debug, arch) + "/build"