Skip to content

certifier report: count number of inlining sites#7657

Merged
zliu41 merged 1 commit intomasterfrom
zliu41/count-inline-sites
Mar 11, 2026
Merged

certifier report: count number of inlining sites#7657
zliu41 merged 1 commit intomasterfrom
zliu41/count-inline-sites

Conversation

@zliu41
Copy link
Member

@zliu41 zliu41 commented Mar 9, 2026

Now the report includes the number of inlining sites, e.g:

──────────────────────────────────────────────────────
Pass 6: Inlining  ✅
──────────────────────────────────────────────────────
  Program Size Before: 14130
  Program Size After: 13396
  Optimization sites: 387

Each `↓ in the proof is an inlining site.

@zliu41 zliu41 added the No Changelog Required Add this to skip the Changelog Check label Mar 9, 2026
@basetunnel
Copy link
Collaborator

Could we also merge #7574 soon? Otherwise I need to keep rebasing that with every certifier change.

@zliu41 zliu41 merged commit ee241d6 into master Mar 11, 2026
10 checks passed
@zliu41 zliu41 deleted the zliu41/count-inline-sites branch March 11, 2026 14:34
@zliu41
Copy link
Member Author

zliu41 commented Mar 11, 2026

Could we also merge #7574 soon? Otherwise I need to keep rebasing that with every certifier change.

I think there are test failures? I'll take a final look once CI is green.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

No Changelog Required Add this to skip the Changelog Check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants