Skip to content

fix: include inferred stack usage in collection summaries#15

Merged
dylandreimerink merged 1 commit into
cilium:mainfrom
immanuwell:fix/collection-summary-inferred-stack-usage
Jul 17, 2026
Merged

fix: include inferred stack usage in collection summaries#15
dylandreimerink merged 1 commit into
cilium:mainfrom
immanuwell:fix/collection-summary-inferred-stack-usage

Conversation

@immanuwell

@immanuwell immanuwell commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

list and web could show 0 bytes when DWARF omits a stack location

reuse existing instruction-derived slots for collection summaries

Repro: stackwhere list testdata/noinline.o used to print 0 bytes - entry; now prints 8 bytes - entry

Signed-off-by: immanuwell <pchpr.00@list.ru>

@dylandreimerink dylandreimerink left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the regression, thanks for the fix!

@dylandreimerink
dylandreimerink merged commit fa03a92 into cilium:main Jul 17, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants