Skip to content

Conversation

@bbiiggppiigg
Copy link
Member

No description provided.

@bbiiggppiigg bbiiggppiigg requested a review from hainest October 24, 2025 21:21
| :------------ | :----------: | :---------: | -----------------------------: |
| 1-gfx908.gpubin | gfx908 | John | Indirect Control Flow Analysis |
| 2-gfx90a.gpubin | gfx90a | John | Indirect Control Flow Analysis |
| 3-gfx940.gpubin | gfx940 | John | Indirect Control Flow Analysis |
Copy link
Contributor

Choose a reason for hiding this comment

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

  1. It's interesting how different the file sizes are. Is there something special about the 908 one to make it so much smaller?
  2. Are there tests to go into Dyninst to use these?
  3. Is 'John' Mellor-Crummey?

Copy link
Member Author

Choose a reason for hiding this comment

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

  1. It's interesting how different the file sizes are. Is there something special about the 908 one to make it so much smaller?

These were binaries which previously trigger bugs in Dyninst, reported by John.

  1. Are there tests to go into Dyninst to use these?

I personally have an example that prints out the indirect control flow targets, and I used them to check if anything breaks.

  1. Is 'John' Mellor-Crummey?

Yes, I was being lazy, I can probably fix this.
TBH, I can probably find the GitHub issues / PR that were associated with the fix for these binaries, when I’m not lazy.

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