[DepScan] Add missing field in JSON output

SwiftSourceModule can contain "commandLine" field and add that into the
JSON output format.

Author: cachemeifyoucan

lib/DependencyScan/ScanDependencies.cpp

@@ -1016,20 +1016,6 @@ static void writeJSON(llvm::raw_ostream &out,
                              5,
                              /*trailingComma=*/true);
         out.indent(5 * 2);
-        out << "\"commandLine\": [\n";
-        for (int i = 0, count = swiftTextualDeps->command_line->count;
-             i < count; ++i) {
-          const auto &arg =
-              get_C_string(swiftTextualDeps->command_line->strings[i]);
-          out.indent(6 * 2);
-          out << "\"" << quote(arg) << "\"";
-          if (i != count - 1)
-            out << ",";
-          out << "\n";
-        }
-        out.indent(5 * 2);
-        out << "],\n";
-        out.indent(5 * 2);
         out << "\"compiledModuleCandidates\": [\n";
         for (int i = 0,
                  count = swiftTextualDeps->compiled_module_candidates->count;
@@ -1045,6 +1031,20 @@ static void writeJSON(llvm::raw_ostream &out,
         out.indent(5 * 2);
         out << "],\n";
       }
+      out.indent(5 * 2);
+      out << "\"commandLine\": [\n";
+      for (int i = 0, count = swiftTextualDeps->command_line->count; i < count;
+           ++i) {
+        const auto &arg =
+            get_C_string(swiftTextualDeps->command_line->strings[i]);
+        out.indent(6 * 2);
+        out << "\"" << quote(arg) << "\"";
+        if (i != count - 1)
+          out << ",";
+        out << "\n";
+      }
+      out.indent(5 * 2);
+      out << "],\n";
       bool hasBridgingHeaderPath =
           swiftTextualDeps->bridging_header_path.data &&
           get_C_string(swiftTextualDeps->bridging_header_path)[0] != '\0';

test/CAS/module_deps.swift

@@ -66,7 +66,8 @@ import SubE
 // CHECK-DAG:     "swift": "_Concurrency"
 // CHECK-DAG:     "swift": "_cross_import_E"
 // CHECK: ],
-
+// CHECK:      "commandLine":
+// CHECK:      "casFSRootID":
 // CHECK:      "extraPcmArgs": [
 // CHECK-NEXT:    "-Xcc",
 // CHECK-NEXT:    "-target",

test/ModuleInterface/clang-session-transitive.swift

@@ -33,6 +33,9 @@ import TestModule
  // CHECK-NEXT:        "swift": {
  // CHECK-NEXT:          "moduleInterfacePath": 
  // CHECK-NEXT:          "contextHash":
+ // CHECK-NEXT:          "compiledModuleCandidates": [
+ // CHECK-NEXT:            TestModule.swiftmodule
+ // CHECK-NEXT:          ],
  // CHECK-NEXT:          "commandLine": [
  // CHECK-NEXT:            "-frontend",
  // CHECK-NEXT:            "-compile-module-from-interface",

test/ModuleInterface/extension-transitive-availability.swift

@@ -42,6 +42,8 @@ func foo() {
 // CHECK-NEXT:        "swift": {
 // CHECK-NEXT:          "moduleInterfacePath": 
 // CHECK-NEXT:          "contextHash":
+// CHECK-NEXT:          "compiledModuleCandidates": [
+// CHECK-NEXT:          ],
 // CHECK-NEXT:          "commandLine": [
 // CHECK-NEXT:            "-frontend",
 // CHECK-NEXT:            "-compile-module-from-interface",