Merge pull request swiftlang#30182 from scentini/enum-cxx

Author: swift-ci

test/ClangImporter/Inputs/custom-modules/EnumExhaustivity.apinotes renamed to test/CXXInterop/enum/Inputs/EnumExhaustivity.apinotes

@@ -0,0 +1,19 @@
+enum {
+  Constant1,
+  Constant2
+};
+
+enum {
+  VarConstant1,
+  VarConstant2
+} global;
+
+typedef struct SR2511 {
+    int x;
+
+    enum {
+      SR2511A = 0, SR2511B, SR2511C
+    } y;
+
+    int z;
+} SR2511;

test/ClangImporter/Inputs/custom-modules/EnumExhaustivity.h renamed to test/CXXInterop/enum/Inputs/EnumExhaustivity.h

@@ -0,0 +1,8 @@
+module EnumAnon {
+  header "enum-anon.h"
+}
+
+module EnumExhaustivity {
+  header "EnumExhaustivity.h"
+  export *
+}

test/CXXInterop/enum/Inputs/enum-anon.h

@@ -1,4 +1,7 @@
-// RUN: %target-swift-frontend -emit-ir %s -enable-objc-interop -import-objc-header %S/Inputs/enum-anon.h | %FileCheck -check-prefix=CHECK -check-prefix=CHECK-%target-runtime %s
+// RUN: %target-swift-frontend -emit-ir %s -I %S/Inputs | %FileCheck -check-prefix=CHECK %s
+// RUN: %target-swift-frontend -emit-ir %s -I %S/Inputs -enable-cxx-interop | %FileCheck -check-prefix=CHECK %s
+
+import EnumAnon
 
 func verifyIsInt(_: inout Int) { }
 func verifyIsInt32(_: inout Int32) { }
@@ -23,11 +26,6 @@ public func testIR(x: UnsafePointer<SR2511>) -> CInt {
   // CHECK: store i32 1, i32* getelementptr inbounds ([[ENUM_TYPE]], [[ENUM_TYPE]]* bitcast (i32* @global to [[ENUM_TYPE]]*), i32 0, i32 0), align 4
   global = VarConstant2
 
-#if _runtime(_ObjC)
-  // CHECK-objc: store i16 1, i16* getelementptr inbounds (%Ts6UInt16V, %Ts6UInt16V* bitcast (i16* @usGlobal to %Ts6UInt16V*), i32 0, i32 0), align 2
-  usGlobal = USVarConstant2
-#endif
-
   // Force the definition of the type above.
   // CHECK: ret
   return x.pointee.z

test/CXXInterop/enum/Inputs/module.map

@@ -0,0 +1,17 @@
+// RUN: %target-swift-frontend -typecheck %s -I  %S/Inputs -DDIAGS -verify
+// RUN: %target-swift-frontend -typecheck %s -enable-cxx-interop -I %S/Inputs -DDIAGS -verify
+import EnumAnon
+
+func testDiags() {
+
+  let us2: UInt16 = 0
+  let _: String = us2 // expected-error {{cannot convert value of type 'UInt16' to specified type 'String'}}
+
+  let usVar2: UInt16 = 0
+  let _: String = usVar2 // expected-error {{cannot convert value of type 'UInt16' to specified type 'String'}}
+
+  // The nested anonymous enum value should still have top-level scope, because
+  // that's how C works. It should also have the same type as the field (above).
+  let _: String = SR2511.SR2511B // expected-error {{type 'SR2511' has no member 'SR2511B'}}
+}
+

test/ClangImporter/enum-anon-sized.swift renamed to test/CXXInterop/enum/enum-anon-sized.swift

@@ -1,4 +1,4 @@
-// RUN: %target-swift-frontend -emit-ir -primary-file %s -I %S/Inputs/custom-modules -module-cache-path %t -enable-cxx-interop -o - | %FileCheck %s
+// RUN: %target-swift-frontend -emit-ir -primary-file %s -I %S/../../ClangImporter/Inputs/ -module-cache-path %t -enable-cxx-interop -o - | %FileCheck %s
 
 import CXXInterop
 

test/CXXInterop/enum/enum-anon.swift

@@ -1,4 +1,4 @@
-// RUN: %target-swift-frontend -typecheck %s -Xcc -isystem -Xcc %S/Inputs/custom-modules -verify -enable-nonfrozen-enum-exhaustivity-diagnostics
+// RUN: %target-swift-frontend -typecheck %s -Xcc -isystem -Xcc %S/Inputs -verify -enable-nonfrozen-enum-exhaustivity-diagnostics
 
 import EnumExhaustivity
 

test/ClangImporter/enum-cxx.swift renamed to test/CXXInterop/enum/enum-cxx.swift

@@ -1,6 +1,6 @@
-// RUN: %target-swift-frontend -typecheck %s -I %S/Inputs/custom-modules -verify -warnings-as-errors -enable-nonfrozen-enum-exhaustivity-diagnostics
+// RUN: %target-swift-frontend -typecheck %s -I %S/Inputs -verify -warnings-as-errors -enable-nonfrozen-enum-exhaustivity-diagnostics
 
-// RUN: %target-swift-ide-test -source-filename %s -print-module -module-to-print EnumExhaustivity -I %S/Inputs/custom-modules | %FileCheck %s
+// RUN: %target-swift-ide-test -source-filename %s -print-module -module-to-print EnumExhaustivity -I %S/Inputs | %FileCheck %s
 
 // CHECK-LABEL: {{^}}enum RegularEnum : {{.+}} {
 // CHECK:      case A

test/ClangImporter/enum-exhaustivity-system.swift renamed to test/CXXInterop/enum/enum-exhaustivity-system.swift

@@ -45,11 +45,6 @@ module ctypes_bits_exported {
   export *
 }
 
-module EnumExhaustivity {
-  header "EnumExhaustivity.h"
-  export *
-}
-
 module ExternIntX { header "x.h" }
 
 module HasSubmodule {

test/ClangImporter/enum-exhaustivity.swift renamed to test/CXXInterop/enum/enum-exhaustivity.swift

@@ -1,24 +1,3 @@
-enum {
-  Constant1,
-  Constant2
-};
-
-enum {
-  VarConstant1,
-  VarConstant2
-} global;
-
-typedef struct SR2511 {
-    int x;
-
-    enum {
-      SR2511A = 0, SR2511B, SR2511C
-    } y;
-
-    int z;
-} SR2511;
-
-#if __OBJC__
 enum : unsigned short {
   USConstant1,
   USConstant2
@@ -28,4 +7,3 @@ enum : unsigned short {
   USVarConstant1,
   USVarConstant2
 } usGlobal;
-#endif // __OBJC__

test/ClangImporter/Inputs/custom-modules/module.map

@@ -1,22 +1,10 @@
 // RUN: %target-swift-frontend -typecheck %s -enable-objc-interop -import-objc-header %S/Inputs/enum-anon.h -DDIAGS -verify
 
+// REQUIRES: objc_interop
+
 func testDiags() {
-#if _runtime(_ObjC)
   let us2 = USConstant2
-#else
-  let us2: UInt16 = 0
-#endif
   let _: String = us2 // expected-error {{cannot convert value of type 'UInt16' to specified type 'String'}}
-
-#if _runtime(_ObjC)
   let usVar2 = USVarConstant2
-#else
-  let usVar2: UInt16 = 0
-#endif
   let _: String = usVar2 // expected-error {{cannot convert value of type 'UInt16' to specified type 'String'}}
-
-  // The nested anonymous enum value should still have top-level scope, because
-  // that's how C works. It should also have the same type as the field (above).
-  let _: String = SR2511.SR2511B // expected-error {{type 'SR2511' has no member 'SR2511B'}}
 }
-