Gardening: Migrate test suite to GH issues: ClangImporter

Author: AnthonyLatsis

test/ClangImporter/Inputs/enum-anon.h

@@ -8,15 +8,16 @@ enum {
   VarConstant2
 } global;
 
-typedef struct SR2511 {
-    int x;
+// NB: The order of fields here is important.
+typedef struct Struct {
+    int firstField;
 
     enum {
-      SR2511A = 0, SR2511B, SR2511C
-    } y;
+      NestedConstant1 = 0, NestedConstant2, NestedConstant3
+    } adhocAnonEnumField;
 
-    int z;
-} SR2511;
+    int lastField;
+} Struct;
 
 #if __OBJC__
 enum : unsigned short {

test/ClangImporter/MixedSource/submodule.swift

@@ -3,7 +3,8 @@
 @_exported import Mixed
 @_exported import Mixed.Submodule
 
-// SR-12265: Make sure we can perform a scoped import on a submodule in a mixed
+// https://github.com/apple/swift/issues/54693
+// Make sure we can perform a scoped import on a submodule in a mixed
 // source target.
 import func Mixed.Submodule.fromSubmodule
 

test/ClangImporter/availability_returns_twice.swift

@@ -1,8 +1,12 @@
 // RUN: %target-typecheck-verify-swift
+
 // UNSUPPORTED: OS=windows-msvc
+
 // `setjmp` is not available on WebAssembly/WASI
 // UNSUPPORTED: OS=wasi
-// In Android jmp_buf is int[16], which doesn't convert to &Int (SR-9136)
+
+// https://github.com/apple/swift/issues/51632
+// In Android jmp_buf is int[16], which doesn't convert to &Int
 // XFAIL: OS=linux-androideabi
 // XFAIL: OS=linux-android
 // XFAIL: OS=openbsd

test/ClangImporter/cfuncs_parse.swift

@@ -67,9 +67,10 @@ func test_pow() {
   pow(1.5, 2.5)
 }
 
-#if !((os(Android) || os(Linux)) && arch(arm64))
+// https://github.com/apple/swift/issues/51573
 // long doubles in AAPCS64 are 128 bits, which is not supported by
-// Swift, so don't test this. SR-9072.
+// Swift, so don't test this.
+#if !((os(Android) || os(Linux)) && arch(arm64))
 func test_powl() {
   powl(1.5, 2.5)
 }

test/ClangImporter/cxx_interop_ir.swift

@@ -1,6 +1,7 @@
 // RUN: %target-swiftxx-frontend -module-name cxx_ir -I %S/Inputs/custom-modules -emit-ir -o - -primary-file %s | %FileCheck %s
-//
-// We can't yet call member functions correctly on Windows (SR-13129).
+
+// https://github.com/apple/swift/issues/55575
+// We can't yet call member functions correctly on Windows.
 // XFAIL: OS=windows-msvc
 
 import CXXInterop

test/ClangImporter/enum-anon-sized.swift

@@ -6,7 +6,11 @@ func verifyIsUInt32(_: inout UInt32) { }
 
 var a = Constant2
 var b = VarConstant2
-var c = SR2511().y
+
+/// https://github.com/apple/swift/issues/45116
+/// Anonymous enums used as ad-hoc types should not be naively imported
+/// as `Int`.
+var c = Struct().adhocAnonEnumField
 
 verifyIsInt(&a)
 #if !os(Windows)
@@ -17,9 +21,9 @@ verifyIsInt32(&b)
 verifyIsInt32(&c)
 #endif
 
-// CHECK: %TSo6SR2511V = type <{ %Ts5Int32V, [[ENUM_TYPE:%Ts5Int32V|%Ts6UInt32V]], %Ts5Int32V }>
-// CHECK-LABEL: define{{.*}} i32 @"$s4main6testIR1xs5Int32VSPySo6SR2511VG_tF"(
-public func testIR(x: UnsafePointer<SR2511>) -> CInt {
+// CHECK: %TSo6StructV = type <{ %Ts5Int32V, [[ENUM_TYPE:%Ts5Int32V|%Ts6UInt32V]], %Ts5Int32V }>
+// CHECK-LABEL: define{{.*}} i32 @"$s4main6testIR1xs5Int32VSPySo6StructVG_tF"(
+public func testIR(x: UnsafePointer<Struct>) -> 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
 
@@ -30,6 +34,6 @@ public func testIR(x: UnsafePointer<SR2511>) -> CInt {
 
   // Force the definition of the type above.
   // CHECK: ret
-  return x.pointee.z
+  return x.pointee.lastField
 } // CHECK-NEXT: {{^}$}}
 

test/ClangImporter/enum-anon.swift

@@ -1,4 +1,4 @@
-// RUN: %target-swift-frontend -typecheck %s -enable-objc-interop -import-objc-header %S/Inputs/enum-anon.h -DDIAGS -verify
+// RUN: %target-swift-frontend -typecheck %s -enable-objc-interop -import-objc-header %S/Inputs/enum-anon.h -verify
 
 func testDiags() {
 #if _runtime(_ObjC)
@@ -17,6 +17,7 @@ func testDiags() {
 
   // 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'}}
+  let _: String = Struct.NestedConstant2
+  // expected-error@-1 {{type 'Struct' has no member 'NestedConstant2'}}
 }
 

test/ClangImporter/indirect_field_codegen.swift

@@ -1,6 +1,6 @@
 // RUN: %target-swift-frontend -emit-silgen -I %S/Inputs/custom-modules %s | %FileCheck %s
 
-// https://bugs.swift.org/browse/SR-10571
+// https://github.com/apple/swift/issues/52971
 
 import IndirectFields
 

test/ClangImporter/objc_async.swift

@@ -260,9 +260,10 @@ func testMirrored(instance: ClassWithAsync) async {
 @available(SwiftStdlib 5.5, *)
 @SomeGlobalActor func sgActorFn() {}
 
-// Check inferred isolation for overridden decls from ObjC.
-// Note that even if the override is not present, it
-// can have an affect. -- rdar://87217618 / SR-15694
+// rdar://87217618
+// https://github.com/apple/swift/issues/57973
+// Check inferred isolation for overridden decls from ObjC. Note that even if
+// the override is not present, it can have an affect.
 @MainActor
 @available(SwiftStdlib 5.5, *)
 class FooFrame: PictureFrame {

test/ClangImporter/objc_protocol_renaming.swift

@@ -15,7 +15,8 @@ import Foundation
 import ProtoRenaming
 import ImplementProtoRenaming
 
-// SR-3917: compiler crash when using an "old" name for an imported requirement
+// https://github.com/apple/swift/issues/46502
+// Compiler crash when using an "old" name for an imported requirement
 class MyGraphViewSubclass : MyGraphView {
 	@objc func doSomethingToGraphView(_ view: GraphView) { } // expected-error{{method 'doSomethingToGraphView' with Objective-C selector 'doSomethingToGraphView:' conflicts with method 'doSomething(to:)' from superclass 'MyGraphView' with the same Objective-C selector}}
 }

test/ClangImporter/objc_redeclared_properties.swift

@@ -33,6 +33,7 @@ func test(obj: RPFoo) {
   obj.accessorInProto = nil // okay
 }
 
-func sr8490(obj: RPSub) {
+// https://github.com/apple/swift/issues/51011
+func f_51011(obj: RPSub) {
   obj.accessorInProto = nil
 }