Downgrade placeholder error to warning for swiftparser

Author: kimdv

lib/ASTGen/Sources/ASTGen/Diagnostics.swift

@@ -89,14 +89,15 @@ func emitDiagnostic(
   diagEnginePtr: UnsafeMutablePointer<UInt8>,
   sourceFileBuffer: UnsafeMutableBufferPointer<UInt8>,
   diagnostic: Diagnostic,
+  diagnosticSeverity: DiagnosticSeverity,
   messageSuffix: String? = nil
 ) {
   // Emit the main diagnostic
   emitDiagnosticParts(
     diagEnginePtr: diagEnginePtr,
     sourceFileBuffer: sourceFileBuffer,
     message: diagnostic.diagMessage.message + (messageSuffix ?? ""),
-    severity: diagnostic.diagMessage.severity,
+    severity: diagnosticSeverity,
     position: diagnostic.position,
     highlights: diagnostic.highlights
   )

lib/ASTGen/Sources/ASTGen/SourceFile.swift

@@ -1,3 +1,4 @@
+import SwiftDiagnostics
 import SwiftParser
 import SwiftSyntax
 import SwiftParserDiagnostics
@@ -77,7 +78,8 @@ extension Syntax {
 public func emitParserDiagnostics(
   diagEnginePtr: UnsafeMutablePointer<UInt8>,
   sourceFilePtr: UnsafeMutablePointer<UInt8>,
-  emitOnlyErrors: CInt
+  emitOnlyErrors: CInt,
+  downgradePlaceholderErrors: CInt
 ) -> CInt {
   return sourceFilePtr.withMemoryRebound(
     to: ExportedSourceFile.self, capacity: 1
@@ -98,11 +100,19 @@ public func emitParserDiagnostics(
         continue
       }
 
+      let diagnosticSeverity: DiagnosticSeverity
+      if downgradePlaceholderErrors == 1 && diag.diagMessage.diagnosticID == StaticParserError.editorPlaceholderInSourceFile.diagnosticID {
+        diagnosticSeverity = .warning
+      } else {
+        diagnosticSeverity = diag.diagMessage.severity
+      }
+
       emitDiagnostic(
         diagEnginePtr: diagEnginePtr,
         sourceFileBuffer: UnsafeMutableBufferPointer(
           mutating: sourceFile.pointee.buffer),
-        diagnostic: diag
+        diagnostic: diag,
+        diagnosticSeverity: diagnosticSeverity
       )
       anyDiags = true
     }

lib/Parse/ParseDecl.cpp

@@ -188,7 +188,8 @@ extern "C" int swift_ASTGen_roundTripCheck(void *sourceFile);
 /// diagnostics were emitted.
 extern "C" int swift_ASTGen_emitParserDiagnostics(void *diagEngine,
                                                   void *sourceFile,
-                                                  int emitOnlyErrors);
+                                                  int emitOnlyErrors,
+                                                  int downgradePlaceholderErrors);
 
 // Build AST nodes for the top-level entities in the syntax.
 extern "C" void swift_ASTGen_buildTopLevelASTNodes(void *sourceFile,
@@ -276,7 +277,8 @@ void Parser::parseTopLevelItems(SmallVectorImpl<ASTNode> &items) {
                !Context.Diags.hadAnyError() &&
                swift_ASTGen_emitParserDiagnostics(&Context.Diags,
                                                   SF.exportedSourceFile,
-                                                  /*emitOnlyErrors=*/true)) {
+                                                  /*emitOnlyErrors=*/true,
+                                                  /*downgradePlaceholderErrors=*/true)) {
       // We might have emitted warnings in the C++ parser but no errors, in
       // which case we still have `hadAnyError() == false`. To avoid emitting
       // the same warnings from SwiftParser, only emit errors from SwiftParser
@@ -316,7 +318,7 @@ Parser::parseSourceFileViaASTGen(SmallVectorImpl<ASTNode> &items,
          Context.LangOpts.hasFeature(Feature::ParserASTGen)) &&
         !suppressDiagnostics &&
         swift_ASTGen_emitParserDiagnostics(
-            &Context.Diags, SF.exportedSourceFile, /*emitOnlyErrors=*/false) &&
+            &Context.Diags, SF.exportedSourceFile, /*emitOnlyErrors=*/false, /*downgradePlaceholderErrors=*/false) &&
         Context.Diags.hadAnyError() &&
         !Context.LangOpts.hasFeature(Feature::ParserASTGen)) {
       // Errors were emitted, and we're still using the C++ parser, so