expose @_transparent; don't expose @_alwaysEmitIntoClient and async

Author: WANGJIEKE

lib/PrintAsClang/DeclAndTypePrinter.cpp

@@ -1110,9 +1110,9 @@ class DeclAndTypePrinter::Implementation
 
   void visitFuncDecl(FuncDecl *FD) {
     if (outputLang == OutputLanguageMode::Cxx) {
-      // Don't expose async functions or transparent functions
+      // Don't expose async functions or @_alwaysEmitIntoClient functions
       // because they're currently unsupported
-      if (FD->hasAsync() || FD->isTransparent()) {
+      if (FD->hasAsync() || FD->getAttrs().hasAttribute<AlwaysEmitIntoClientAttr>()) {
         return;
       }
 

test/Interop/SwiftToCxx/functions/swift-no-expose-unsupported-func.swift

@@ -4,8 +4,8 @@
 
 // RUN: %check-interop-cxx-header-in-clang(%t/functions.h)
 
-// CHECK-NOT: SWIFT_EXTERN double $s9Functions10async_funcyS2dYaF(double x) SWIFT_NOEXCEPT SWIFT_CALL; // async_func(_:)
-// CHECK-NOT: SWIFT_EXTERN bool $s9Functions16transparent_funcyS2bF(bool x) SWIFT_NOEXCEPT SWIFT_CALL; // transparent_func(_:)
+// CHECK-NOT: SWIFT_EXTERN bool $s9Functions24alwaysEmitIntoClientFuncyS2bF(bool x) SWIFT_NOEXCEPT SWIFT_CALL; // alwaysEmitIntoClientFunc(_:)
+// CHECK-NOT: SWIFT_EXTERN double $s9Functions9asyncFuncyS2dYaF(double x) SWIFT_NOEXCEPT SWIFT_CALL; // asyncFunc(_:)
 
 // CHECK:       namespace Functions {
 // CHECK-EMPTY:
@@ -16,15 +16,15 @@
 // CHECK-EMPTY:
 // CHECK-NEXT:  } // namespace Functions
 
-// CHECK-NOT:  inline double async_func(double x) noexcept SWIFT_WARN_UNUSED_RESULT {
-// CHECK-NOT:   return _impl::$s9Functions10async_funcyS2dYaF(x);
+// CHECK-NOT: inline bool alwaysEmitIntoClientFunc(bool x) noexcept SWIFT_WARN_UNUSED_RESULT {
+// CHECK-NOT:   return _impl::$s9Functions24alwaysEmitIntoClientFuncyS2bF(x);
 // CHECK-NOT: }
 
-// CHECK-NOT:  inline bool transparent_func(bool x) noexcept SWIFT_WARN_UNUSED_RESULT {
-// CHECK-NOT:   return _impl::$s9Functions16transparent_funcyS2bF(x);
+// CHECK-NOT: inline double asyncFunc(double x) noexcept SWIFT_WARN_UNUSED_RESULT {
+// CHECK-NOT:   return _impl::$s9Functions9asyncFuncyS2dYaF(x);
 // CHECK-NOT: }
 
-public func async_func(_ x: Double) async -> Double { return 2 * x }
+public func asyncFunc(_ x: Double) async -> Double { return 2 * x }
 
-@_transparent
-public func transparent_func(_ x: Bool) -> Bool { return !x }
+@_alwaysEmitIntoClient
+public func alwaysEmitIntoClientFunc(_ x: Bool) -> Bool { return !x }

test/Interop/SwiftToCxx/functions/swift-transparent-functions-cxx-bridging.swift

@@ -0,0 +1,14 @@
+// RUN: %empty-directory(%t)
+// RUN: %target-swift-frontend %s -typecheck -module-name Functions -clang-header-expose-public-decls -emit-clang-header-path %t/functions.h
+// RUN: %FileCheck %s < %t/functions.h
+
+// RUN: %check-interop-cxx-header-in-clang(%t/functions.h)
+
+// CHECK: SWIFT_EXTERN ptrdiff_t $s9Functions24transparentPrimitiveFuncyS2iF(ptrdiff_t x) SWIFT_NOEXCEPT SWIFT_CALL; // transparentPrimitiveFunc(_:)
+
+// CHECK:      inline swift::Int transparentPrimitiveFunc(swift::Int x) noexcept SWIFT_WARN_UNUSED_RESULT {
+// CHECK-NEXT:   return _impl::$s9Functions24transparentPrimitiveFuncyS2iF(x);
+// CHECK-NEXT: }
+
+@_transparent
+public func transparentPrimitiveFunc(_ x: Int) -> Int { return x * x }

test/Interop/SwiftToCxx/functions/swift-transparent-functions-execution.cpp

@@ -0,0 +1,22 @@
+// RUN: %empty-directory(%t)
+
+// RUN: %target-swift-frontend %S/swift-transparent-functions-cxx-bridging.swift -typecheck -module-name Functions -clang-header-expose-public-decls -emit-clang-header-path %t/functions.h
+
+// RUN: %target-interop-build-clangxx -c %s -I %t -o %t/swift-functions-execution.o
+// RUN: %target-interop-build-swift %S/swift-transparent-functions-cxx-bridging.swift -o %t/swift-functions-execution -Xlinker %t/swift-functions-execution.o -module-name Functions -Xfrontend -entry-point-function-name -Xfrontend swiftMain
+
+// RUN: %target-codesign %t/swift-functions-execution
+// RUN: %target-run %t/swift-functions-execution
+
+// REQUIRES: executable_test
+
+#include <cassert>
+#include "functions.h"
+
+int main() {
+    using namespace Functions;
+    
+    assert(transparentPrimitiveFunc(12) == 144);
+
+    return 0;
+}