Merge pull request #69776 from apple/egorzhdan/reland-nsnotificationname-lookup

Revert "Revert "[cxx-interop] Look up `NSNotificationName` in C++ language mode properly""

Author: egorzhdan

lib/ClangImporter/ClangAdapter.cpp

@@ -560,7 +560,9 @@ clang::TypedefNameDecl *importer::findSwiftNewtype(const clang::NamedDecl *decl,
     clang::LookupResult lookupResult(clangSema, notificationName,
                                      clang::SourceLocation(),
                                      clang::Sema::LookupOrdinaryName);
-    if (!clangSema.LookupName(lookupResult, clangSema.TUScope))
+    if (!clangSema.LookupQualifiedName(
+            lookupResult,
+            /*LookupCtx*/ clangSema.getASTContext().getTranslationUnitDecl()))
       return nullptr;
     auto nsDecl = lookupResult.getAsSingle<clang::TypedefNameDecl>();
     if (!nsDecl)

test/Interop/Cxx/objc-correctness/Inputs/nsnotification-bridging.h

@@ -1,3 +1,4 @@
 #import <Foundation/Foundation.h>
 
 extern NSString * const SpaceShipNotification;
+extern "C" NSString * const CExternNotification;

test/Interop/Cxx/objc-correctness/nsnotification-bridging-ide-test.swift

@@ -5,3 +5,4 @@
 // CHECK: import Foundation
 
 // CHECK: let SpaceShipNotification: String
+// CHECK: let CExternNotification: String

test/Interop/Cxx/objc-correctness/nsnotification-typechecker.swift

@@ -0,0 +1,10 @@
+// RUN: %target-typecheck-verify-swift -verify-ignore-unknown -I %S/Inputs -enable-objc-interop -enable-experimental-cxx-interop
+// REQUIRES: objc_interop
+
+import Foundation
+import NSNotificationBridging
+
+func test(_ n: NSNotification.Name) {}
+
+test(NSNotification.Name.SpaceShip)
+test(NSNotification.Name.CExtern)