Skip to content

call_stacks: rewrite for performance, add SQL implementation#116

Merged
mwootton merged 1 commit into
ROCm:masterfrom
mwootton:master
Jul 24, 2026
Merged

call_stacks: rewrite for performance, add SQL implementation#116
mwootton merged 1 commit into
ROCm:masterfrom
mwootton:master

Conversation

@mwootton

Copy link
Copy Markdown
Collaborator
 - Pre-compute GPU times into dict instead of per-event LEFT JOIN
 - Bulk-insert all rows at once via executemany
 - Replace StackFrame class with mutable lists
 - Add pure SQL implementation (--sql) using window functions + recursive CTE
 - Improve CLI: add --sql flag, WAL mode, cache tuning, post-insert indexes
 - Both implementations produce identical output (verified on 2.6M rows)

@mwootton
mwootton merged commit b24896a into ROCm:master Jul 24, 2026
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.

1 participant