llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Rahul Joshi (jurahul) <details> <summary>Changes</summary> Use non-templated form of `getTrailingObjects` when using a single trailing type in `TrailingObjects`. --- Full diff: https://github.com/llvm/llvm-project/pull/139974.diff 2 Files Affected: - (modified) clang/include/clang/AST/ASTConcept.h (+2-2) - (modified) clang/lib/AST/ASTConcept.cpp (+4-6) ``````````diff diff --git a/clang/include/clang/AST/ASTConcept.h b/clang/include/clang/AST/ASTConcept.h index 078e1e848f393..c8f6330a73bb1 100644 --- a/clang/include/clang/AST/ASTConcept.h +++ b/clang/include/clang/AST/ASTConcept.h @@ -93,11 +93,11 @@ struct ASTConstraintSatisfaction final : bool ContainsErrors : 1; const UnsatisfiedConstraintRecord *begin() const { - return getTrailingObjects<UnsatisfiedConstraintRecord>(); + return getTrailingObjects(); } const UnsatisfiedConstraintRecord *end() const { - return getTrailingObjects<UnsatisfiedConstraintRecord>() + NumRecords; + return getTrailingObjects() + NumRecords; } ASTConstraintSatisfaction(const ASTContext &C, diff --git a/clang/lib/AST/ASTConcept.cpp b/clang/lib/AST/ASTConcept.cpp index f7ee0fb3ee92d..c9adccdbc77ef 100644 --- a/clang/lib/AST/ASTConcept.cpp +++ b/clang/lib/AST/ASTConcept.cpp @@ -40,9 +40,8 @@ ASTConstraintSatisfaction::ASTConstraintSatisfaction( IsSatisfied{Satisfaction.IsSatisfied}, ContainsErrors{ Satisfaction.ContainsErrors} { for (unsigned I = 0; I < NumRecords; ++I) - CreateUnsatisfiedConstraintRecord( - C, Satisfaction.Details[I], - getTrailingObjects<UnsatisfiedConstraintRecord>() + I); + CreateUnsatisfiedConstraintRecord(C, Satisfaction.Details[I], + getTrailingObjects() + I); } ASTConstraintSatisfaction::ASTConstraintSatisfaction( @@ -51,9 +50,8 @@ ASTConstraintSatisfaction::ASTConstraintSatisfaction( IsSatisfied{Satisfaction.IsSatisfied}, ContainsErrors{Satisfaction.ContainsErrors} { for (unsigned I = 0; I < NumRecords; ++I) - CreateUnsatisfiedConstraintRecord( - C, *(Satisfaction.begin() + I), - getTrailingObjects<UnsatisfiedConstraintRecord>() + I); + CreateUnsatisfiedConstraintRecord(C, *(Satisfaction.begin() + I), + getTrailingObjects() + I); } ASTConstraintSatisfaction * `````````` </details> https://github.com/llvm/llvm-project/pull/139974 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
