Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(20)

Unified Diff: third_party/opus/BUILD.gn

Issue 2427333004: [Opus] Only include NEON sources and macros when arm_use_neon==true. (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/opus/BUILD.gn
diff --git a/third_party/opus/BUILD.gn b/third_party/opus/BUILD.gn
index 5e5c9ccea8b2c3d262a2c66d5b5e92e9c3d9738c..528bcb417647549e91dfc9c1c502dfddc30e5200 100644
--- a/third_party/opus/BUILD.gn
+++ b/third_party/opus/BUILD.gn
@@ -372,12 +372,9 @@ static_library("opus") {
"src/celt/arm/arm_celt_map.c",
"src/celt/arm/armcpu.c",
"src/celt/arm/armcpu.h",
- "src/celt/arm/celt_neon_intr.c",
"src/celt/arm/fft_arm.h",
"src/celt/arm/mdct_arm.h",
"src/celt/arm/pitch_arm.h",
- "src/silk/arm/NSQ_neon.c",
- "src/silk/arm/NSQ_neon.h",
"src/silk/arm/arm_silk_map.c",
]
@@ -386,14 +383,25 @@ static_library("opus") {
defines += [
"OPUS_ARM_MAY_HAVE_EDSP",
"OPUS_ARM_MAY_HAVE_MEDIA",
- "OPUS_ARM_MAY_HAVE_NEON",
- "OPUS_ARM_MAY_HAVE_NEON_INTR",
"OPUS_HAVE_RTCD",
]
deps = [
":convert_rtcd_assembler",
]
+
+ if (arm_use_neon) {
+ sources += [
+ "src/celt/arm/celt_neon_intr.c",
+ "src/silk/arm/NSQ_neon.c",
+ "src/silk/arm/NSQ_neon.h",
+ ]
+
+ defines += [
+ "OPUS_ARM_MAY_HAVE_NEON",
+ "OPUS_ARM_MAY_HAVE_NEON_INTR",
+ ]
+ }
}
}
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698