[clang] [NFC][Clang] Adopt simplified `getTrailingObjects` in ASTConcept (PR #139974)

via cfe-commits cfe-commits at lists.llvm.org
Wed May 14 17:51:16 PDT 2025


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


More information about the cfe-commits mailing list