Merge pull request swiftlang#82627 from swiftlang/egorzhdan/void-begin-crash

[cxx-interop] Do not crash for `void begin()`

Author: egorzhdan

lib/Sema/CSDiagnostics.cpp

@@ -4134,11 +4134,11 @@ void MissingMemberFailure::diagnoseUnsafeCxxMethod(SourceLoc loc,
                                                           scratch);
     };
 
-    auto returnTypeStr = cast<FuncDecl>(found)
-                             ->getResultInterfaceType()
-                             ->getAnyNominal()
-                             ->getName()
-                             .str();
+    auto returnTy =
+        cast<FuncDecl>(found)->getResultInterfaceType()->getAnyNominal();
+    if (!returnTy)
+      continue;
+    auto returnTypeStr = returnTy->getName().str();
 
     auto methodClangLoc = cxxMethod->getLocation();
     auto methodSwiftLoc =

test/Interop/Cxx/class/invalid-unsafe-projection-errors.swift

@@ -23,6 +23,11 @@ struct M {
   StringLiteral stringLiteral() const { return StringLiteral{"M"}; }
 };
 
+struct HasNonIteratorBeginMethod {
+  void begin() const;
+  void end() const;
+};
+
 //--- test.swift
 
 import Test
@@ -48,3 +53,8 @@ public func test(x: M) {
   // CHECK-NOT: error: value of type 'M' has no member 'stringLiteral'
   x.stringLiteral()
 }
+
+public func test(_ x: HasNonIteratorBeginMethod) {
+  x.begin()
+  x.end()
+}