[nfc][cxx-interop] Add more info to request dumps and add a few pretty stack traces.

Author: zoecarver

lib/ClangImporter/ClangImporter.cpp

@@ -5809,6 +5809,10 @@ static bool hasIteratorAPIAttr(const clang::Decl *decl) {
 /// Recursively checks that there are no pointers in any fields or base classes.
 /// Does not check C++ records with specific API annotations.
 static bool hasPointerInSubobjects(const clang::CXXRecordDecl *decl) {
+  clang::PrettyStackTraceDecl trace(decl, clang::SourceLocation(),
+                                    decl->getASTContext().getSourceManager(),
+                                    "checking if pointer in subobjects");
+
   // Probably a class template that has not yet been specialized:
   if (!decl->getDefinition())
     return false;
@@ -5868,6 +5872,10 @@ static bool copyAssignOperatorIsDefaulted(const clang::CXXRecordDecl *decl) {
 /// destructors in any fields or base classes.
 /// Does not check C++ records with specific API annotations.
 static bool isSufficientlyTrivial(const clang::CXXRecordDecl *decl) {
+  clang::PrettyStackTraceDecl trace(decl, clang::SourceLocation(),
+                                    decl->getASTContext().getSourceManager(),
+                                    "checking if sufficently trivial");
+
   // Probably a class template that has not yet been specialized:
   if (!decl->getDefinition())
     return true;
@@ -5912,6 +5920,10 @@ static bool isSufficientlyTrivial(const clang::CXXRecordDecl *decl) {
 /// Checks if a record provides the required value type lifetime operations
 /// (copy and destroy).
 static bool hasRequiredValueTypeOperations(const clang::CXXRecordDecl *decl) {
+  clang::PrettyStackTraceDecl trace(decl, clang::SourceLocation(),
+                                    decl->getASTContext().getSourceManager(),
+                                    "checking if has required lifetime ops");
+
   if (auto dtor = decl->getDestructor()) {
     if (dtor->isDeleted() || dtor->getAccess() != clang::AS_public) {
       return false;
@@ -6037,6 +6049,8 @@ void swift::simple_display(llvm::raw_ostream &out,
                            CxxRecordSemanticsDescriptor desc) {
   out << "Matching API semantics of C++ record '"
       << desc.decl->getNameAsString() << "'.\n";
+
+  desc.decl->dump();
 }
 
 SourceLoc swift::extractNearestSourceLoc(CxxRecordSemanticsDescriptor desc) {
@@ -6051,6 +6065,8 @@ void swift::simple_display(llvm::raw_ostream &out,
   else
     out << "<invalid decl>";
   out << "' is safe to use in context.\n";
+
+  desc.decl->dump();
 }
 
 SourceLoc swift::extractNearestSourceLoc(SafeUseOfCxxDeclDescriptor desc) {