Add build time optimization documentation #3608
Open
+225
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
tenpercent
requested review from
a team,
Snektron,
ThomasNing,
afagaj,
andriy-ca,
aosewski,
asleepzzz,
bartekxk,
carlushuang,
cgmillette,
coderfeli,
ddembeckAMD,
geyyer,
illsilin,
poyenc,
qianfengz,
shumway,
vidyasagar-amd and
vpietila-amd
as code owners
tenpercent
force-pushed
the
mpodkory/build-time-docs
branch
2 times, most recently
from
b057073 to
5de5575
Compare
tenpercent
force-pushed
the
mpodkory/build-time-docs
branch
from
5de5575 to
52fa8f6
Compare
shumway
reviewed
Jan 19, 2026
shumway
reviewed
Jan 19, 2026
shumway
reviewed
Jan 19, 2026
shumway
reviewed
Jan 19, 2026
shumway
reviewed
Jan 19, 2026
shumway
reviewed
Jan 19, 2026
shumway
reviewed
Jan 19, 2026
shumway
reviewed
Jan 19, 2026
shumway
reviewed
Jan 19, 2026
tenpercent
force-pushed
the
mpodkory/build-time-docs
branch
11 times, most recently
from
80c4f97 to
71413bd
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds comprehensive documentation on build time optimization techniques for reducing C++ template instantiation overhead in the Composable Kernel codebase. The documentation targets the heavy use of template metaprogramming which can significantly impact compilation times.
Changes:
- Documents four key optimization strategies: pack expansion, named functors, constexpr loops, and fold expressions
- Provides before/after code examples with real metrics from previous optimization efforts
- Explains the computational complexity improvements and their practical impact on build times
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
tenpercent
force-pushed
the
mpodkory/build-time-docs
branch
5 times, most recently
from
e814786 to
ad15f55
Compare
tenpercent
force-pushed
the
mpodkory/find-transform-optimization
branch
2 times, most recently
from
ec8e794 to
83a76d7
Compare
This document describes techniques for reducing C++ template instantiation overhead in the Composable Kernel codebase, including: - Replacing recursive templates with pack expansion (O(N) → O(1) depth) - Using named functors instead of lambdas to share instantiations - Replacing template recursion with constexpr loops - Using fold expressions for accumulation operations These techniques can significantly reduce build times for template-heavy code.
tenpercent
force-pushed
the
mpodkory/build-time-docs
branch
from
ad15f55 to
b63bfd9
Compare
tenpercent
changed the base branch from
mpodkory/find-transform-optimization
to
develop
shumway
approved these changes
Jan 23, 2026
Collaborator
shumway
left a comment
shumway
left a comment
There was a problem hiding this comment.
This looks great! Thanks for the work rewriting this, it's really clear and helpful.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
BUILD_TIME_OPTIMIZATION.mddocumenting techniques for reducing C++ template instantiation overheadTracking issue: #3575