[Build] Handle unparsable messages during message size parsing

<rdar://problem/49235252>

Looks like swift compiler can emit a non-parsable message in some cases.

Author: aciidgh

Sources/Build/BuildDelegate.swift

@@ -432,7 +432,7 @@ fileprivate struct CommandTaskTracker {
             }
 
             finishedCount += 1
-        case .signalled, .skipped:
+        case .unparsableOutput, .signalled, .skipped:
             break
         }
     }
@@ -482,9 +482,10 @@ fileprivate struct CommandTaskTracker {
 
 extension SwiftCompilerMessage {
     fileprivate var verboseProgressText: String? {
-        if case .began(let info) = kind {
+        switch kind {
+        case .began(let info):
             return ([info.commandExecutable] + info.commandArguments).joined(separator: " ")
-        } else {
+        case .skipped, .finished, .signalled, .unparsableOutput:
             return nil
         }
     }
@@ -494,7 +495,9 @@ extension SwiftCompilerMessage {
         case .finished(let info),
              .signalled(let info):
             return info.output
-        default:
+        case .unparsableOutput(let output):
+            return output
+        case .skipped, .began:
             return nil
         }
     }

Sources/Build/SwiftCompilerOutputParser.swift

@@ -41,6 +41,7 @@ struct SwiftCompilerMessage {
         case skipped(SkippedInfo)
         case finished(OutputInfo)
         case signalled(OutputInfo)
+        case unparsableOutput(String)
     }
 
     let name: String
@@ -166,7 +167,12 @@ private extension SwiftCompilerOutputParser {
         }
 
         guard let messageSize = Int(string) else {
-            throw ParsingError(reason: "invalid message size")
+            // Non-parseable chunks are *assumed* to be output. E.g., you get
+            // a "remark" if you build with SWIFTC_MAXIMUM_DETERMINISM env variable.
+            let message = SwiftCompilerMessage(name: "unknown", kind: .unparsableOutput(string))
+            delegate?.swiftCompilerOutputParser(self, didParse: message)
+            buffer.removeAll()
+            return
         }
 
         buffer.removeAll()

Tests/BuildTests/SwiftCompilerOutputParserTests.swift

@@ -211,7 +211,9 @@ class SwiftCompilerOutputParserTests: XCTestCase {
             2A
 
             """.utf8)
-        delegate.assert(messages: [], errorDescription: "invalid message size")
+        delegate.assert(messages: [
+            SwiftCompilerMessage(name: "unknown", kind: .unparsableOutput("2A"))
+        ], errorDescription: nil)
 
         parser.parse(bytes: """
             119
@@ -223,7 +225,9 @@ class SwiftCompilerOutputParserTests: XCTestCase {
               "signal": 4
             }
             """.utf8)
-        delegate.assert(messages: [], errorDescription: nil)
+        delegate.assert(messages: [
+            SwiftCompilerMessage(name: "link", kind: .signalled(.init(pid: 22699, output: nil)))
+        ], errorDescription: nil)
     }
 
     func testInvalidMessageBytes() {