Merge pull request #17084 from kitasuke/fix_duplicated_import_delc_for_sil-opt

[SIL] Don't print duplicated import decls

Author: slavapestov

lib/SIL/SILPrinter.cpp

@@ -2712,6 +2712,15 @@ void SILModule::print(SILPrintContext &PrintCtx, ModuleDecl *M,
           !D->isImplicit()) {
         if (isa<AccessorDecl>(D))
           continue;
+
+        // skip to visit ASTPrinter to avoid sil-opt prints duplicated import declarations
+        if (auto importDecl = dyn_cast<ImportDecl>(D)) {
+          StringRef importName = importDecl->getModule()->getName().str();
+          if (importName == BUILTIN_NAME ||
+              importName == STDLIB_NAME ||
+              importName == SWIFT_SHIMS_NAME)
+            continue;
+        }
         D->print(OS, Options);
         OS << "\n\n";
       }

test/SIL/import-decls.sil

@@ -0,0 +1,19 @@
+// RUN: %empty-directory(%t)
+// RUN: touch %t/empty.swift
+// RUN: %target-swift-frontend -emit-module -module-name TestModule %t/empty.swift -o %t
+// RUN: %target-sil-opt %s -I=%t | %FileCheck %s
+
+sil_stage raw
+
+import Builtin
+import Swift
+
+import TestModule
+
+func foo() {}
+
+// CHECK: import Builtin
+// CHECK: import Swift
+// CHECK-NOT: import Builtin
+// CHECK-NOT: import Swift{{$}}
+// CHECK: import TestModule