Minor fixes

Author: scentini

test/Interop/C/implementation-only-imports/lookup-visible-symbols-recursively-inversed.swift renamed to test/Interop/C/implementation-only-imports/lookup-visible-decls-recursively-inversed.swift

@@ -6,15 +6,15 @@
 // RUN: %target-swift-frontend -typecheck -swift-version 5 -I %t/use_module_a -I %t/use_module_b -I %S/Inputs -enable-cxx-interop %s
 
 
-// If a symbol comes from two modules, one of which is marked as
+// If a decl comes from two modules, one of which is marked as
 // @_implementationOnly, Swift may choose the @_implementationOnly source
-// and then error out due to the symbol being hidden.
+// and then error out due to the decl being hidden.
 
-// Swift should consider all sources for the symbol and recognize that the
-// symbol is not hidden behind @_implementationOnly in all modules.
+// Swift should consider all sources for the decl and recognize that the
+// decl is not hidden behind @_implementationOnly in all modules.
 
-// This test, as well as the `lookup-visible-symbols-recursively.swift` check
-// that the `getFortyTwo` symbol can be found when at least one of the
+// This test, as well as `lookup-visible-decls-recursively.swift` checks
+// that the `getFortyTwo` decl can be found when at least one of the
 // modules is not `@_implementationOnly`.
 
 @_implementationOnly import UseModuleA

test/Interop/C/implementation-only-imports/lookup-visible-symbols-recursively.swift renamed to test/Interop/C/implementation-only-imports/lookup-visible-decls-recursively.swift

@@ -6,15 +6,15 @@
 // RUN: %target-swift-frontend -typecheck -swift-version 5 -I %t/use_module_a -I %t/use_module_b -I %S/Inputs -enable-cxx-interop %s
 
 
-// If a symbol comes from two modules, one of which is marked as
+// If a decl comes from two modules, one of which is marked as
 // @_implementationOnly, Swift may choose the @_implementationOnly source
-// and then error out due to the symbol being hidden.
+// and then error out due to the decl being hidden.
 
-// Swift should consider all sources for the symbol and recognize that the
-// symbol is not hidden behind @_implementationOnly in all modules.
+// Swift should consider all sources for the decl and recognize that the
+// decl is not hidden behind @_implementationOnly in all modules.
 
-// This test, as well as the `lookup-visible-symbols-recursively-inversed.swift
-// check that the `getFortyTwo` symbol can be found when at least one of the
+// This test, as well as `lookup-visible-decls-recursively-inversed.swift`
+// checks that the `getFortyTwo` decl can be found when at least one of the
 // modules is not `@_implementationOnly`.
 
 import UseModuleA

test/Interop/C/implementation-only-imports/prefer-a-visible-decl-over-implementation-only-decls-inversed.swift

@@ -0,0 +1,22 @@
+// RUN: %empty-directory(%t)
+// RUN: %target-swift-frontend -emit-module -o %t/FortyTwo.swiftmodule -I %S/Inputs %s
+
+// If a decl comes from two modules, one of which is marked as
+// @_implementationOnly, Swift may choose the @_implementationOnly source
+// and then error out due to the decl being hidden.
+
+// Swift should consider all sources for the decl and recognize that the
+// decl is not hidden behind @_implementationOnly in all modules.
+
+// This test, as well as
+// `prefer-a-visible-decl-over-implementation-only-decls.swift`
+// checks that the `getFortyTwo` decl can be found when at least one of the
+// modules is not `@_implementationOnly`.
+
+import UserA
+@_implementationOnly import UserB
+
+@_inlineable
+public func callFortyTwo() -> CInt {
+    return getFortyTwo()
+}

test/Interop/C/implementation-only-imports/prefer-a-visible-decl-over-implementation-only-decls.swift

@@ -0,0 +1,22 @@
+// RUN: %empty-directory(%t)
+// RUN: %target-swift-frontend -emit-module -o %t/FortyTwo.swiftmodule -I %S/Inputs %s
+
+// If a decl comes from two modules, one of which is marked as
+// @_implementationOnly, Swift may choose the @_implementationOnly source
+// and then error out due to the decl being hidden.
+
+// Swift should consider all sources for the decl and recognize that the
+// decl is not hidden behind @_implementationOnly in all modules.
+
+// This test, as well as
+// `prefer-a-visible-decl-over-implementation-only-decls-inversed.swift`
+// checks that the `getFortyTwo` decl can be found when at least one of the
+// modules is not `@_implementationOnly`.
+
+@_implementationOnly import UserA
+import UserB
+
+@_inlineable
+public func callFortyTwo() -> CInt {
+    return getFortyTwo()
+}