Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add debug ring support to normal ExecuteIndirect #1544

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

HansKristian-Work
Copy link
Owner

When debugging execute indirect faults, we need to know the input arguments to have any hope chasing things down.

@HansKristian-Work HansKristian-Work force-pushed the debug-ring-indirect branch 3 times, most recently from 3bbce3a to e456907 Compare May 16, 2023 09:49
@HansKristian-Work HansKristian-Work marked this pull request as ready for review July 3, 2023 16:33
@HansKristian-Work HansKristian-Work force-pushed the debug-ring-indirect branch 2 times, most recently from 17aecf8 to b542c17 Compare July 13, 2023 08:00
@HansKristian-Work HansKristian-Work force-pushed the debug-ring-indirect branch 3 times, most recently from acb85f0 to 33c0e08 Compare September 4, 2023 16:11
@HansKristian-Work HansKristian-Work force-pushed the debug-ring-indirect branch 4 times, most recently from ffcee2b to c1534e6 Compare January 31, 2024 12:35
When debugging hangs, intention is that we can dump
all execute indirect commands to debug ring.
Some crashes are caused by broken indirect buffers.

Signed-off-by: Hans-Kristian Arntzen <[email protected]>
When using breadcrumbs_trace_indirect, allow generic indirect commands
to go through debugger as well.

Signed-off-by: Hans-Kristian Arntzen <[email protected]>
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