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

Unified Diff: build/config/chromecast/BUILD.gn

Issue 1510973005: [Chromecast] GN - Only apply linker flags for arm (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: Style fix. Created 5 years 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: build/config/chromecast/BUILD.gn
diff --git a/build/config/chromecast/BUILD.gn b/build/config/chromecast/BUILD.gn
index 41f1c079638218a271fe6184f0f78fd10483fe3f..a99ca5fb181056c6656195ff8777bf348a7b48b8 100644
--- a/build/config/chromecast/BUILD.gn
+++ b/build/config/chromecast/BUILD.gn
@@ -15,28 +15,32 @@ config("static_config") {
}
config("executable_config") {
- ldflags = [
- # Export stdlibc++ and libgcc symbols to force shlibs to refer to these
- # symbols from the executable.
- "-Wl,--export-dynamic",
-
- "-lm", # stdlibc++ requires math.h
-
- # In case we redefined stdlibc++ symbols (e.g. tc_malloc)
- "-Wl,--allow-multiple-definition",
-
- "-Wl,--whole-archive",
- "-l:libstdc++.a",
- "-l:libgcc.a",
- "-Wl,--no-whole-archive",
- ]
-
- # Despite including libstdc++/libgcc archives, we still need to specify
- # static linking for them in order to prevent the executable from having a
- # dynamic dependency on them.
- configs = [ ":static_config" ]
+ if (current_cpu == "arm") {
+ ldflags = [
+ # Export stdlibc++ and libgcc symbols to force shlibs to refer to these
+ # symbols from the executable.
+ "-Wl,--export-dynamic",
+
+ "-lm", # stdlibc++ requires math.h
+
+ # In case we redefined stdlibc++ symbols (e.g. tc_malloc)
+ "-Wl,--allow-multiple-definition",
+
+ "-Wl,--whole-archive",
+ "-l:libstdc++.a",
+ "-l:libgcc.a",
+ "-Wl,--no-whole-archive",
+ ]
+
+ # Despite including libstdc++/libgcc archives, we still need to specify
+ # static linking for them in order to prevent the executable from having a
+ # dynamic dependency on them.
+ configs = [ ":static_config" ]
+ }
}
config("shared_library_config") {
- configs = [ ":static_config" ]
+ if (current_cpu == "arm") {
+ configs = [ ":static_config" ]
+ }
}
« 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