Skip to content

Remove perf numbers from docs - #3702

Merged
tenpercent merged 3 commits into
developfrom
remove-perf-numbers-from-docs
Feb 3, 2026
Merged

Remove perf numbers from docs#3702
tenpercent merged 3 commits into
developfrom
remove-perf-numbers-from-docs

Conversation

@tenpercent

@tenpercent tenpercent commented Feb 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Remove concrete performance improvement numbers from BUILD_TIME_OPTIMIZATION.md
  • Replace specific benchmark figures (e.g., "388 to 32 (92% reduction)") with qualitative descriptions
  • Measurements vary across environments and may become outdated, so qualitative statements are more maintainable

Test plan

  • Documentation-only change, no functional impact

@tenpercent @claude
Remove concrete performance numbers from BUILD_TIME_OPTIMIZATION.md
615a720 
Replace specific benchmark numbers with qualitative descriptions since
measurements vary across environments and may become outdated.

Co-Authored-By: Claude <noreply@anthropic.com>
@tenpercent
tenpercent force-pushed the remove-perf-numbers-from-docs branch from 80e3086 to 615a720 Compare February 2, 2026 18:54
@tenpercent
tenpercent enabled auto-merge (squash) February 2, 2026 19:01
@tenpercent
tenpercent merged commit 3f04d27 into develop Feb 3, 2026
15 checks passed
@tenpercent
tenpercent deleted the remove-perf-numbers-from-docs branch February 3, 2026 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AviralGoelAMD AviralGoelAMD AviralGoelAMD approved these changes
@vidyasagar-amd vidyasagar-amd vidyasagar-amd approved these changes
@illsilin illsilin Awaiting requested review from illsilin illsilin is a code owner
@carlushuang carlushuang Awaiting requested review from carlushuang carlushuang is a code owner
@qianfengz qianfengz Awaiting requested review from qianfengz qianfengz is a code owner
@aosewski aosewski Awaiting requested review from aosewski aosewski is a code owner
@poyenc poyenc Awaiting requested review from poyenc poyenc is a code owner
@geyyer geyyer Awaiting requested review from geyyer geyyer is a code owner
@bartekxk bartekxk Awaiting requested review from bartekxk bartekxk is a code owner
@andriy-ca andriy-ca Awaiting requested review from andriy-ca andriy-ca is a code owner
@afagaj afagaj Awaiting requested review from afagaj afagaj is a code owner
@asleepzzz asleepzzz Awaiting requested review from asleepzzz
@ThomasNing ThomasNing Awaiting requested review from ThomasNing
@coderfeli coderfeli Awaiting requested review from coderfeli coderfeli is a code owner
@cgmillette cgmillette Awaiting requested review from cgmillette cgmillette is a code owner
@shumway shumway Awaiting requested review from shumway shumway is a code owner
@ddembeckAMD ddembeckAMD Awaiting requested review from ddembeckAMD ddembeckAMD is a code owner
@vpietila-amd vpietila-amd Awaiting requested review from vpietila-amd vpietila-amd is a code owner
@Snektron Snektron Awaiting requested review from Snektron Snektron is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tenpercent @AviralGoelAMD @vidyasagar-amd