Skip to content

Feat/comparative graph capture changes#797

Draft
kyle-hoffmeyer wants to merge 25 commits into
mainfrom
feat/comparative_graph_capture_changes
Draft

Feat/comparative graph capture changes#797
kyle-hoffmeyer wants to merge 25 commits into
mainfrom
feat/comparative_graph_capture_changes

Conversation

@kyle-hoffmeyer

Copy link
Copy Markdown
Collaborator

Pull Request Template

Note to AMDers:
This is a public repository. Please do not upload any confidential or customer data. Make sure all such data has been anonymized or removed before making this PR. If you need to attach any private files or links, please insert a Internal OneDrive Link or a Jira Ticket Link instead.

kyle-hoffmeyer and others added 25 commits June 3, 2026 14:47
…unts mismatch

AMD capture traces sometimes record extra HIP dispatch API calls
(hipExtModuleLaunchKernel, hipLaunchKernel) that get consolidated
during graph optimization, causing a count mismatch vs graph kernels.
Previously the merge was skipped entirely for that execution root,
leaving its kernels as flat children of hipGraphLaunch with no cpu_op
ancestry.

Adds _align_capture_to_graph which scans forward through capture
dispatch events to match each graph kernel by name, discarding any
unmatched extras. verify_subtree_events now returns a (success,
aligned_capture_events) tuple so the caller can proceed with the
aligned subset when counts differ but names align.

Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
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