Skip to content

Conversation

@GuillaumeLagrange
Copy link
Contributor

@GuillaumeLagrange GuillaumeLagrange commented Jan 20, 2026

We used to resolve symbols to their innermost inlined frame, which would locate some user-written code that inlines something from the stdblib inside the stdlib files.

@codspeed-hq
Copy link

codspeed-hq bot commented Jan 20, 2026

CodSpeed Performance Report

Merging this PR will not alter performance

Comparing cod-2054-wrong-file-for-spans-in-walltime-flamegraphs (06d99f2) with main (e37aa06)

Summary

✅ 4 untouched benchmarks

@fargito fargito requested a review from art049 January 21, 2026 09:26
Copy link
Member

@art049 art049 left a comment

Choose a reason for hiding this comment

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

lgtm except the clarity in the comment

@art049
Copy link
Member

art049 commented Jan 21, 2026

but nice catch!

@GuillaumeLagrange GuillaumeLagrange force-pushed the cod-2054-wrong-file-for-spans-in-walltime-flamegraphs branch from 4d4872e to 06d99f2 Compare January 21, 2026 17:07
@GuillaumeLagrange GuillaumeLagrange merged commit 06d99f2 into main Jan 21, 2026
13 checks passed
@GuillaumeLagrange GuillaumeLagrange deleted the cod-2054-wrong-file-for-spans-in-walltime-flamegraphs branch January 21, 2026 17:11
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.

3 participants