[Dependency Scanner] Do not escape strings in the scanner output

When outputting strings for things like filenames, using `write_escaped` will result in Unicode characters being outputted with a full 3-character-octal or hex escape. Clients which expect a UTF-8 JSON output will not be able to parse such escape sequences.

Author: artemcm

lib/FrontendTool/ScanDependencies.cpp

@@ -307,7 +307,7 @@ namespace {
                       StringRef value,
                       unsigned indentLevel) {
     out << "\"";
-    out.write_escaped(value);
+    out << value;
     out << "\"";
   }
 
@@ -460,8 +460,6 @@ static void writeJSON(llvm::raw_ostream &out,
     writeJSONSingleField(out, "modulePath", modulePath, /*indentLevel=*/3,
                          /*trailingComma=*/true);
 
-    // Artem Refactoring
-    {
     // Source files.
     if (swiftTextualDeps) {
       writeJSONSingleField(out, "sourceFiles", swiftTextualDeps->sourceFiles, 3,
@@ -613,7 +611,6 @@ static void writeJSON(llvm::raw_ostream &out,
       out << ",";
     out << "\n";
   }
-  }
 }
 
 static bool diagnoseCycle(CompilerInstance &instance,

test/ScanDependencies/Inputs/unicode_filёnamё.swift

@@ -0,0 +1,3 @@
+func foo() {
+    print(1)
+}

test/ScanDependencies/unicode_filename.swift

@@ -0,0 +1,16 @@
+// RUN: %empty-directory(%t)
+// RUN: %target-swift-frontend -scan-dependencies %s %S/Inputs/unicode_filёnamё.swift -o %t/deps.json
+
+// Check the contents of the JSON output
+// RUN: %FileCheck %s < %t/deps.json
+
+print(foo())
+
+// CHECK:      "swift": "deps"
+// CHECK-NEXT:    },
+// CHECK-NEXT:    {
+// CHECK-NEXT:      "modulePath": "deps.swiftmodule",
+// CHECK-NEXT:      "sourceFiles": [
+// CHECK-NEXT:        "{{.*}}ScanDependencies{{/|\\}}unicode_filename.swift",
+// CHECK-NEXT:        "{{.*}}ScanDependencies{{/|\\}}Inputs{{/|\\}}unicode_filёnamё.swift"
+// CHECK-NEXT:      ],