From 80553e4f14b956007e67c9259822a239ef03aeb4 Mon Sep 17 00:00:00 2001
From: Harish Mahendrakar <hmahendrakar@google.com>
Date: Fri, 22 Mar 2024 23:49:00 +0000
Subject: [PATCH] Remove useless `neon:` clause.

There hasn't been a non-neon platform build in years.

Also, remove unused DISABLE_NEONINTR, ARMGCC macro definitions

Bug: 330929681
Test: Builds
Change-Id: Ibf1c8dab90ba77b8725eb623bbee297a43a69bcd
---
 Android.bp        | 40 +++++++++++-----------------------------
 cmake/utils.cmake |  3 +--
 2 files changed, 12 insertions(+), 31 deletions(-)

diff --git a/Android.bp b/Android.bp
index df34bbc..4b82f9f 100644
--- a/Android.bp
+++ b/Android.bp
@@ -86,36 +86,21 @@ cc_library_static {
             srcs: [
                 "common/arm/ideint_function_selector.c",
                 "decoder/arm/impeg2d_function_selector.c",
+                "common/arm/icv_sad_a9.s",
+                "common/arm/icv_variance_a9.s",
+                "common/arm/ideint_cac_a9.s",
+                "common/arm/ideint_function_selector_a9.c",
+                "common/arm/ideint_spatial_filter_a9.s",
+                "common/arm/impeg2_format_conv.s",
+                "common/arm/impeg2_idct.s",
+                "common/arm/impeg2_inter_pred.s",
+                "common/arm/impeg2_mem_func.s",
+                "decoder/arm/impeg2d_function_selector_a9q.c",
             ],
 
-            neon: {
-                srcs: [
-                    "common/arm/icv_sad_a9.s",
-                    "common/arm/icv_variance_a9.s",
-                    "common/arm/ideint_cac_a9.s",
-                    "common/arm/ideint_function_selector_a9.c",
-                    "common/arm/ideint_spatial_filter_a9.s",
-                    "common/arm/impeg2_format_conv.s",
-                    "common/arm/impeg2_idct.s",
-                    "common/arm/impeg2_inter_pred.s",
-                    "common/arm/impeg2_mem_func.s",
-                    "decoder/arm/impeg2d_function_selector_a9q.c",
-                ],
-                cflags: [
-                    "-UDISABLE_NEON",
-                    "-UDEFAULT_ARCH",
-                    "-DDEFAULT_ARCH=D_ARCH_ARM_A9Q",
-                ],
-            },
-
             cflags: [
-                "-DDISABLE_NEONINTR",
                 "-DARM",
-                "-DARMGCC",
-
-                // These are overriden by armv7_a_neon
-                "-DDISABLE_NEON",
-                "-DDEFAULT_ARCH=D_ARCH_ARM_NONEON",
+                "-DDEFAULT_ARCH=D_ARCH_ARM_A9Q",
             ],
             instruction_set: "arm",
         },
@@ -123,10 +108,7 @@ cc_library_static {
         arm64: {
             cflags: [
                 "-DARMV8",
-                "-DDISABLE_NEONINTR",
                 "-DARM",
-                "-DARMGCC",
-
                 "-DDEFAULT_ARCH=D_ARCH_ARMV8_GENERIC",
             ],
             local_include_dirs: [
diff --git a/cmake/utils.cmake b/cmake/utils.cmake
index 559ebe6..cdd091f 100644
--- a/cmake/utils.cmake
+++ b/cmake/utils.cmake
@@ -35,8 +35,7 @@ function(libmpeg2_add_definitions)
   if(${CMAKE_SYSTEM_PROCESSOR} STREQUAL "aarch64")
     add_definitions(-DARMV8 -DDEFAULT_ARCH=D_ARCH_ARMV8_GENERIC -DENABLE_NEON)
   elseif(${CMAKE_SYSTEM_PROCESSOR} STREQUAL "aarch32")
-    add_definitions(-DARMV7 -DDEFAULT_ARCH=D_ARCH_ARM_A9Q -DENABLE_NEON
-                    -DDISABLE_NEONINTR)
+    add_definitions(-DARMV7 -DDEFAULT_ARCH=D_ARCH_ARM_A9Q -DENABLE_NEON)
   else()
     add_definitions(-DX86 -DX86_LINUX=1 -DDISABLE_AVX2
                     -DDEFAULT_ARCH=D_ARCH_X86_SSE42)