[Chromecast] GN - Only apply linker flags for arm

build/config/chromecast/BUILD.gn

 # Copyright 2015 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//build/config/chromecast_build.gni")
 
 assert(is_chromecast)
 
 config("static_config") {
   ldflags = [
     # We want to statically link libstdc++/libgcc.
     "-static-libstdc++",
     "-static-libgcc",
   ]
 }
 
 config("executable_config") {
-  ldflags = [
-    # Export stdlibc++ and libgcc symbols to force shlibs to refer to these
-    # symbols from the executable.
-    "-Wl,--export-dynamic",
+  ldflags = []

[slan, 2015/12/10 01:25:54]

No need to delcare these here, just assign them directly inside the conditional.

[bcf, 2015/12/10 01:46:38]

Done.

+  configs = []
 
-    "-lm",  # stdlibc++ requires math.h
+  if (current_cpu == "arm") {

[bcf, 2015/12/10 01:17:21]

Is it more preferred to put the conditionals in the config, or use variables
outside of the config set based on the conditional?

+    ldflags += [
+      # Export stdlibc++ and libgcc symbols to force shlibs to refer to these
+      # symbols from the executable.
+      "-Wl,--export-dynamic",
 
-    # In case we redefined stdlibc++ symbols (e.g. tc_malloc)
-    "-Wl,--allow-multiple-definition",
+      "-lm",  # stdlibc++ requires math.h
 
-    "-Wl,--whole-archive",
-    "-l:libstdc++.a",
-    "-l:libgcc.a",
-    "-Wl,--no-whole-archive",
-  ]
+      # In case we redefined stdlibc++ symbols (e.g. tc_malloc)
+      "-Wl,--allow-multiple-definition",
 
-  # 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" ]
+      "-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" ]
+  }
 }