[gn] Consolidate calls to gypi_to_gn to speed up GN generation

runtime/vm/BUILD.gn

 # Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS file
 # for details. All rights reserved. Use of this source code is governed by a
 # BSD-style license that can be found in the LICENSE file.
 
+import("gypi_contents.gni")
+
 config("libdart_vm_config") {
   if (defined(is_fuchsia) && is_fuchsia) {
     libs = [ "magenta" ]
   } else if (is_win) {
     libs = [
       "advapi32.lib",
       "shell32.lib",
       "dbghelp.lib",
     ]
   } else {
     libs = [ "dl" ]
     if (!is_android) {
       libs += [ "pthread" ]
     }
     if (is_linux) {
       libs += [ "rt" ]
     }
   }
 }
 
 static_library("libdart_platform") {
   configs += [
     "..:dart_config",
     "..:dart_maybe_product_config",
   ]
   public_configs = [ ":libdart_vm_config" ]
 
-  platform_headers_gypi =
-      exec_script("../../tools/gypi_to_gn.py",
-                  [ rebase_path("../platform/platform_headers.gypi") ],
-                  "scope",
-                  [ "../platform/platform_headers.gypi" ])
-  platform_headers =
-      rebase_path(platform_headers_gypi.sources, ".", "../platform")
+  sources = rebase_path(processed_gypis.platform_sources, ".", "../platform")
 
-  platform_sources_gypi =
-      exec_script("../../tools/gypi_to_gn.py",
-                  [ rebase_path("../platform/platform_sources.gypi") ],
-                  "scope",
-                  [ "../platform/platform_sources.gypi" ])
-  platform_sources =
-      rebase_path(platform_sources_gypi.sources, ".", "../platform")
-
-  sources = platform_headers + platform_sources
   include_dirs = [ ".." ]
 }
 
-vm_sources_list = exec_script("../../tools/gypi_to_gn.py",
-                              [ rebase_path("vm_sources.gypi") ],
-                              "scope",
-                              [ "vm_sources.gypi" ])
+vm_sources_list = processed_gypis.vm_sources
 
 static_library("libdart_vm") {
   configs += [
     "..:dart_config",
     "..:dart_maybe_product_config",
     "..:dart_maybe_precompiled_runtime_config",
   ]
   public_configs = [ ":libdart_vm_config" ]
   set_sources_assignment_filter([
                                   "*_test.cc",
                                   "*_test.h",
                                 ])
-  sources = vm_sources_list.sources
+  sources = vm_sources_list
   include_dirs = [ ".." ]
 }
 
 static_library("libdart_vm_noopt") {
   configs += [
     "..:dart_config",
     "..:dart_maybe_product_config",
     "..:dart_precompiler_config",
     "..:dart_maybe_precompiled_runtime_config",
   ]
   public_configs = [ ":libdart_vm_config" ]
   set_sources_assignment_filter([
                                   "*_test.cc",
                                   "*_test.h",
                                 ])
-  sources = vm_sources_list.sources
+  sources = vm_sources_list
   include_dirs = [ ".." ]
 }
 
 static_library("libdart_vm_precompiled_runtime") {
   configs += [
     "..:dart_config",
     "..:dart_maybe_product_config",
     "..:dart_precompiled_runtime_config",
   ]
   public_configs = [ ":libdart_vm_config" ]
   set_sources_assignment_filter([
                                   "*_test.cc",
                                   "*_test.h",
                                 ])
-  sources = vm_sources_list.sources
+  sources = vm_sources_list
   include_dirs = [ ".." ]
 }
 
 static_library("libdart_vm_nosnapshot") {
   configs += [
     "..:dart_config",
     "..:dart_maybe_product_config",
     "..:dart_maybe_precompiled_runtime_config",
     "..:dart_no_snapshot_config",
   ]
   public_configs = [ ":libdart_vm_config" ]
   set_sources_assignment_filter([
                                   "*_test.cc",
                                   "*_test.h",
                                 ])
-  sources = vm_sources_list.sources
+  sources = vm_sources_list
   include_dirs = [ ".." ]
 }
 
 static_library("libdart_vm_nosnapshot_precompiled_runtime") {
   configs += [
     "..:dart_config",
     "..:dart_maybe_product_config",
     "..:dart_precompiled_runtime_config",
     "..:dart_no_snapshot_config",
   ]
   public_configs = [ ":libdart_vm_config" ]
   set_sources_assignment_filter([
                                   "*_test.cc",
                                   "*_test.h",
                                 ])
-  sources = vm_sources_list.sources
+  sources = vm_sources_list
   include_dirs = [ ".." ]
 }
 
 static_library("libdart_vm_nosnapshot_with_precompiler") {
   configs += [
     "..:dart_config",
     "..:dart_maybe_product_config",
     "..:dart_precompiler_config",
     "..:dart_no_snapshot_config",
   ]
   public_configs = [ ":libdart_vm_config" ]
   set_sources_assignment_filter([
                                   "*_test.cc",
                                   "*_test.h",
                                 ])
-  sources = vm_sources_list.sources
+  sources = vm_sources_list
   include_dirs = [ ".." ]
 }
 
 template("process_library_source") {
   assert(defined(invoker.filename), "Need filename in $target_name")
+  assert(defined(invoker.type), "Need type in $target_name")
   assert(defined(invoker.output), "Need output in $target_name")
+  assert(defined(invoker.type), "Need type in $target_name")
   assert(defined(invoker.path), "Need path in $target_name")
   action(target_name) {
     visibility = [ ":*" ]  # Only targets in this file can see this.
     filename = invoker.filename
+    type = invoker.type
     path = invoker.path
 
-    lib_sources_gypi =
-        exec_script("../../tools/gypi_to_gn.py",
-                    [ rebase_path("${path}/${filename}_sources.gypi") ],
-                    "scope",
-                    [ "${path}/${filename}_sources.gypi" ])
-    lib_sources = rebase_path(lib_sources_gypi.sources, ".", path)
+    lib_sources =
+        rebase_path(processed_gypis["${filename}_${type}_sources"], ".", path)
 
     script = invoker.script
     inputs = invoker.inputs + [ script ]
     inputs += lib_sources
     outputs = [
       invoker.output,
     ]
     args = invoker.args + rebase_path(lib_sources, root_build_dir)
   }
 }
 
 template("generate_library_source") {
   assert(defined(invoker.libname), "Need libname in $target_name")
   assert(defined(invoker.filename), "Need a filename in $target_name")
   assert(defined(invoker.kind), "Need kind in $target_name")
+  assert(defined(invoker.type), "Need type in $target_name")
   assert(defined(invoker.output), "Need output in $target_name")
-  assert(defined(invoker.path), "Need path in $target_name")
 
   process_library_source(target_name) {
     libname = invoker.libname
     filename = invoker.filename
     kind = invoker.kind
+    type = invoker.type
     output = invoker.output
-    path = invoker.path
+
+    if (type == "sdk") {
+      path = "../../sdk/lib/" + filename
+    } else {
+      path = "../lib"
+    }
     script = "../tools/gen_library_src_paths.py"
     inputs = [
       "../lib/libgen_in.cc",
     ]
     args = [
       "--output",
       rebase_path(invoker.output, root_build_dir),
       "--input_cc",
       rebase_path("../lib/libgen_in.cc", root_build_dir),
       "--include",
@@ skipping 16 matching lines @@
 # The template iterates over the list, and generates generate_library_source
 # actions for each. After that, it generates targets to compile the generated
 # sources to make libdart_lib_nosnapshot and libdart_lib.
 template("generate_core_libraries") {
   assert(defined(invoker.sources), "Need sources in $target_name")
   liboutputs = []
   libsources = []
   libdeps = []
   foreach(lib, invoker.sources) {
     libname = lib[0]
-    filename = lib[1]
-    do_patch = lib[2]
-    source_path = lib[3]
+    type = lib[1]
+    filename = lib[2]
+    do_patch = lib[3]
     generate_library_source("generate_${filename}_cc_file") {
       libname = libname
       filename = filename
       kind = "source"
-      path = source_path
+      type = type
       output = "$target_gen_dir/${filename}_gen.cc"
     }
     if (do_patch) {
-      patch_path = lib[4]
       generate_library_source("generate_${filename}_patch_cc_file") {
         libname = libname
         filename = filename
         kind = "patch"
-        path = patch_path
+        type = "runtime"
         output = "$target_gen_dir/${filename}_patch_gen.cc"
       }
     }
-    lib_sources_gypi = {
-    }
-    lib_sources_gypi =
-        exec_script("../../tools/gypi_to_gn.py",
-                    [ rebase_path("../lib/${filename}_sources.gypi") ],
-                    "scope",
-                    [ "../lib/${filename}_sources.gypi" ])
-    libsources += rebase_path(lib_sources_gypi.sources, ".", "../lib")
+    libsources += rebase_path(processed_gypis["${filename}_runtime_sources"],
+                              ".",
+                              "../lib")
     liboutputs += [ "$target_gen_dir/${filename}_gen.cc" ]
     libdeps += [ ":generate_${filename}_cc_file" ]
     if (do_patch) {
       liboutputs += [ "$target_gen_dir/${filename}_patch_gen.cc" ]
       libdeps += [ ":generate_${filename}_patch_cc_file" ]
     }
   }
 
   static_library("libdart_lib_nosnapshot") {
     configs += [
@@ skipping 42 matching lines @@
     ]
     sources = libsources + [ "bootstrap_nocore.cc" ]
     include_dirs = [ ".." ]
   }
 }
 
 generate_core_libraries("core_libraries") {
   sources = [
     [
       "async",
+      "sdk",
       "async",
       true,
-      "../../sdk/lib/async",
-      "../lib",
     ],
     [
       "core",
+      "sdk",
       "core",
       true,
-      "../../sdk/lib/core",
-      "../lib",
     ],
     [
       "collection",
+      "sdk",
       "collection",
       true,
-      "../../sdk/lib/collection",
-      "../lib",
     ],
     [
       "convert",
+      "sdk",
       "convert",
       true,
-      "../../sdk/lib/convert",
-      "../lib",
     ],
     [
       "developer",
+      "sdk",
       "developer",
       true,
-      "../../sdk/lib/developer",
-      "../lib",
     ],
     [
       "_internal",
+      "sdk",
       "internal",
       true,
-      "../../sdk/lib/internal",
-      "../lib",
     ],
     [
       "isolate",
+      "sdk",
       "isolate",
       true,
-      "../../sdk/lib/isolate",
-      "../lib",
     ],
     [
       "math",
+      "sdk",
       "math",
       true,
-      "../../sdk/lib/math",
-      "../lib",
     ],
     [
       "mirrors",
+      "sdk",
       "mirrors",
       true,
-      "../../sdk/lib/mirrors",
-      "../lib",
     ],
     [
       "profiler",
+      "sdk",
       "profiler",
       false,
-      "../../sdk/lib/profiler",
     ],
     [
       "typed_data",
+      "runtime",
       "typed_data",
       false,
-      "../lib",
     ],
     [
       "_vmservice",
+      "sdk",
       "vmservice",
       true,
-      "../../sdk/lib/vmservice",
-      "../lib",
     ],
   ]
 }
 
 template("concatenate_patch") {
   assert(defined(invoker.libname), "Need a name in $target_name")
   assert(defined(invoker.dir), "Need a dir in $target_name")
   assert(defined(invoker.output), "Need an output in $target_name")
 
   process_library_source(target_name) {
     output = invoker.output
     path = "../${invoker.dir}"
+    type = "sdk"
     filename = invoker.libname
     script = "../tools/concatenate_patches.py"
     args = [
       "--output",
       rebase_path(output, root_build_dir),
     ]
     inputs = []
   }
 }
 
@@ skipping 118 matching lines @@
       "vmservice",
     ],
     [
       "bin",
       "io",
     ],
   ]
 }
 
 }