================ @@ -13462,13 +13462,114 @@ static ElaboratedTypeKeyword getCommonTypeKeyword(const T *X, const T *Y) { : ElaboratedTypeKeyword::None; } +static NestedNameSpecifier *getCommonNNS(ASTContext &Ctx, + NestedNameSpecifier *X, + NestedNameSpecifier *Y, bool IsSame) { + if (X == Y) + return X; + + NestedNameSpecifier *Canon = Ctx.getCanonicalNestedNameSpecifier(X); + if (Canon != Ctx.getCanonicalNestedNameSpecifier(Y)) { + assert(!IsSame && "Should be the same NestedNameSpecifier"); + return nullptr; ---------------- erichkeane wrote:
Since this happens as an assert, and it seems the users of this don't seem to reasonably handle a `nullptr`, would it be more reasonable to just return one of the names anyway? https://github.com/llvm/llvm-project/pull/131964 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits