[clang] [clang-tools-extra] [clang] improve class type sugar preservation in pointers to members (PR #130537)

[Matheus Izvekov via cfe-commits]

================
@@ -5172,8 +5179,28 @@ QualType::DestructionKind QualType::isDestructedTypeImpl(QualType type) {
   return DK_none;
 }
 
+bool MemberPointerType::isSugared() const {
+  CXXRecordDecl *D1 = getMostRecentCXXRecordDecl(),
+                *D2 = getQualifier()->getAsRecordDecl();
+  return D1 != D2 && D1->getCanonicalDecl() != D2->getCanonicalDecl();
----------------
mizvekov wrote:

No, we expect these to be null for the dependent case.
But if one is null, the other must be null as well.

https://github.com/llvm/llvm-project/pull/130537