Skip to content

Conversation

@b1llow
Copy link

@b1llow b1llow commented Oct 20, 2025

gdb-xml: add tricore.xml

link: BinaryAnalysisPlatform/bap-frames#29

@b1llow
Copy link
Author

b1llow commented Oct 23, 2025

Please take a look and see if there are any other areas that need revision.

@Rot127

Copy link

@Rot127 Rot127 left a comment

Choose a reason for hiding this comment

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

Just those nitpicks. Otherwise lgtm

static void plugin_exit(qemu_plugin_id_t id, void *udata) {
flush_all_frame_bufs();
qemu_plugin_outs("Exiting plugin, flushing data...\n");
// flush_all_frame_bufs();
Copy link

Choose a reason for hiding this comment

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

Please document why it is commented out. Or even better, open an issue and link it here.
Also change the log message, because it doesn't flush the data anymore.

@@ -0,0 +1,77 @@
<?xml version="1.0"?>
Copy link

Choose a reason for hiding this comment

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

Please add a comment here where you got it from or if you created it and with what reference. Also for the other xml files.

@Rot127
Copy link

Rot127 commented Oct 23, 2025

Would you mind fixing the build error? Should be just a simple install of dependencies.

@b1llow b1llow requested a review from Rot127 October 23, 2025 15:12
…(unused)) (#1)

Co-authored-by: b1llow <24983369+b1llow@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
@b1llow b1llow requested a review from Rot127 October 24, 2025 09:11
@Rot127
Copy link

Rot127 commented Oct 24, 2025

@b1llow Sorry, last thing. Can you please add TriCore to the "known to work" list in the main README.md?

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