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

Unified Diff: webrtc/BUILD.gn

Issue 1181373004: Add a [rtc_]build_with_neon variable to unify conditions. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Rewording. Created 5 years, 6 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 | webrtc/build/arm_neon.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/BUILD.gn
diff --git a/webrtc/BUILD.gn b/webrtc/BUILD.gn
index 6c6c23cfa0550cc00fbe5fac426d1f6ebece599d..483f8089510bbf94f5ada7263dba9f3e258c4a14 100644
--- a/webrtc/BUILD.gn
+++ b/webrtc/BUILD.gn
@@ -164,6 +164,25 @@ config("common_config") {
}
}
+# Add to ARM NEON targets.
+config("arm_neon_config") {
Andrew MacDonald 2015/06/17 17:44:16 kjellander: I'm not super familiar with this stuff
+ if (current_cpu == "arm") {
+ # Enable compilation for the ARM v7 Neon instruction set. This is needed
+ # since //build/config/arm.gni only enables Neon for iOS, not Android.
+ # This provides the same functionality as webrtc/build/arm_neon.gypi.
+ configs -= [ "//build/config/compiler:compiler_arm_fpu" ]
+ cflags = [ "-mfpu=neon" ]
+ }
+
+ # Disable LTO due to compiler bug.
+ if (rtc_use_lto) {
+ cflags -= [
+ "-flto",
+ "-ffat-lto-objects",
+ ]
+ }
+}
+
source_set("webrtc") {
sources = [
"call.h",
« no previous file with comments | « no previous file | webrtc/build/arm_neon.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698