[clang] [clang] Reduce TemplateDeclInstantiator size. (PR #142983)
Haojian Wu via cfe-commits
cfe-commits at lists.llvm.org
Thu Jun 5 07:48:53 PDT 2025
https://github.com/hokein created https://github.com/llvm/llvm-project/pull/142983
This gives us another ~1.85% improvement (1617->1647 for the `instantiation-depth-default.cpp`) on clang's template instantiation depths,
No performance regressions have been observed: https://llvm-compile-time-tracker.com/compare.php?from=702e228249906d43687952d9a2f3d2f90d8024c6&to=61be4bfea92d52cfc3e48a3cabb1bc80cbebb7fa&stat=instructions:u
>From 61be4bfea92d52cfc3e48a3cabb1bc80cbebb7fa Mon Sep 17 00:00:00 2001
From: Haojian Wu <hokein.wu at gmail.com>
Date: Thu, 5 Jun 2025 15:38:58 +0200
Subject: [PATCH] [clang] Reduce TemplateDeclInstantiator size.
---
clang/include/clang/Sema/Template.h | 4 ++--
clang/lib/Sema/SemaTemplateInstantiateDecl.cpp | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/clang/include/clang/Sema/Template.h b/clang/include/clang/Sema/Template.h
index af0c1e8506cf3..0c68ad9b9d90a 100644
--- a/clang/include/clang/Sema/Template.h
+++ b/clang/include/clang/Sema/Template.h
@@ -587,7 +587,7 @@ enum class TemplateSubstitutionKind : char {
/// specializations that will need to be instantiated after the
/// enclosing class's instantiation is complete.
SmallVector<std::pair<ClassTemplateDecl *,
- ClassTemplatePartialSpecializationDecl *>, 4>
+ ClassTemplatePartialSpecializationDecl *>, 1>
OutOfLinePartialSpecs;
/// A list of out-of-line variable template partial
@@ -595,7 +595,7 @@ enum class TemplateSubstitutionKind : char {
/// enclosing variable's instantiation is complete.
/// FIXME: Verify that this is needed.
SmallVector<
- std::pair<VarTemplateDecl *, VarTemplatePartialSpecializationDecl *>, 4>
+ std::pair<VarTemplateDecl *, VarTemplatePartialSpecializationDecl *>, 1>
OutOfLineVarPartialSpecs;
public:
diff --git a/clang/lib/Sema/SemaTemplateInstantiateDecl.cpp b/clang/lib/Sema/SemaTemplateInstantiateDecl.cpp
index bcad815e1587f..7da09b149446a 100644
--- a/clang/lib/Sema/SemaTemplateInstantiateDecl.cpp
+++ b/clang/lib/Sema/SemaTemplateInstantiateDecl.cpp
@@ -2394,7 +2394,7 @@ Decl *TemplateDeclInstantiator::VisitVarTemplateDecl(VarTemplateDecl *D) {
// Queue up any out-of-line partial specializations of this member
// variable template; the client will force their instantiation once
// the enclosing class has been instantiated.
- SmallVector<VarTemplatePartialSpecializationDecl *, 4> PartialSpecs;
+ SmallVector<VarTemplatePartialSpecializationDecl *, 1> PartialSpecs;
D->getPartialSpecializations(PartialSpecs);
for (unsigned I = 0, N = PartialSpecs.size(); I != N; ++I)
if (PartialSpecs[I]->getFirstDecl()->isOutOfLine())
More information about the cfe-commits
mailing list