[cxx-interop] Fix printing of namespaces declared in bridging headers

If a C++ namespace has redeclarations in a bridging header, printing AST for the namespace would crash the compiler. This is because such a redeclaration would not have an owning Clang module, and the AST printer did not account for that.

This change fixes the crash.

rdar://151715540

Author: egorzhdan

lib/AST/ASTPrinter.cpp

@@ -2740,6 +2740,10 @@ static void addNamespaceMembers(Decl *decl,
     declOwner = declOwner->getTopLevelModule();
   auto Redecls = llvm::SmallVector<clang::NamespaceDecl *, 2>(namespaceDecl->redecls());
   std::stable_sort(Redecls.begin(), Redecls.end(), [&](clang::NamespaceDecl *LHS, clang::NamespaceDecl *RHS) {
+    // A namespace redeclaration will not have an owning Clang module if it is
+    // declared in a bridging header.
+    if (!LHS->getOwningModule() || !RHS->getOwningModule())
+      return (bool)LHS->getOwningModule() < (bool)RHS->getOwningModule();
     return LHS->getOwningModule()->Name < RHS->getOwningModule()->Name;
   });
   for (auto redecl : Redecls) {

test/Interop/Cxx/namespace/Inputs/bridging-header.h

@@ -0,0 +1,15 @@
+namespace NS1 {
+namespace NS2 {
+
+struct InBridgingHeader1 {};
+
+} // namespace NS2
+} // namespace NS1
+
+namespace NS1 {
+namespace NS2 {
+
+struct InBridgingHeader2 {};
+
+} // namespace NS2
+} // namespace NS1

test/Interop/Cxx/namespace/bridging-header-module-interface.swift

@@ -0,0 +1,4 @@
+// RUN: %target-swift-ide-test -print-header -header-to-print %S/Inputs/bridging-header.h -import-objc-header %S/Inputs/bridging-header.h -source-filename=%s -cxx-interoperability-mode=upcoming-swift | %FileCheck %s
+
+// CHECK: struct InBridgingHeader1
+// CHECK: struct InBridgingHeader2