Skip to content

Update hipFile documentation and add batch-roundtrip example - #564

Merged
j-stephan merged 1 commit into
amd-stagingfrom
jstephan/hipFile-update
Oct 30, 2026
Merged

j-stephan merged 1 commit into
amd-stagingfrom
jstephan/hipFile-update

Conversation

@j-stephan

Copy link
Copy Markdown
Contributor

Motivation

Upstream has changed the documentation of the existing hipFile examples in a few locations. Additionally, there is a new example available.

Technical Details

Sync'd the documentation with upstream, added the batch_roundtrip example.

Test Plan

Tested locally on a gfx1100 GPU.

Test Result

Example passed the test suite.

Added/Updated documentation?

  • Yes
  • No, does not apply to this PR.

Included Visual Studio files?

  • Yes
  • No, does not apply to this PR.

Submission Checklist

@j-stephan
j-stephan requested review from a team as code owners October 29, 2026 14:49
Co-authored-by: GLM-5.3-Flash <noreply@z.ai>
Signed-off-by: Jan Stephan <jan.stephan@amd.com>
@j-stephan
j-stephan force-pushed the jstephan/hipFile-update branch from 0970454 to 118dc8c Compare October 29, 2026 15:30
@j-stephan j-stephan self-assigned this Oct 29, 2026

@zichguan-amd zichguan-amd left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@j-stephan

Copy link
Copy Markdown
Contributor Author

The failing test is unrelated to this PR (another executable times out), so I'm merging this as-is.

@j-stephan
j-stephan merged commit b9a2888 into amd-staging Oct 30, 2026
22 of 23 checks passed
@j-stephan
j-stephan deleted the jstephan/hipFile-update branch October 30, 2026 09:16
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