Skip to content

Add KA versioninfo interface#933

Draft
luraess wants to merge 2 commits into
mainfrom
lr/ka-vinfo
Draft

Add KA versioninfo interface#933
luraess wants to merge 2 commits into
mainfrom
lr/ka-vinfo

Conversation

@luraess

@luraess luraess commented Jun 19, 2026

Copy link
Copy Markdown
Member

Follows up on #932 and will require KA and AK bump as well as merging AMDGPU main into this branch.

Needs JuliaGPU/KernelAbstractions.jl#712

@christiangnrd

Copy link
Copy Markdown
Member

The @info statements redirected to io.

Also, you could probably merge the new io argument to versioninfo right away and I'll include the KA integration with #868 once it's merged.

@luraess

luraess commented Jun 19, 2026

Copy link
Copy Markdown
Member Author

Thanks!

The @info statements redirected to io.

I updated the @info bit upon code review suggestion.

Also, you could probably merge the new io argument to versioninfo right away and I'll include the KA integration with #868 once it's merged.

You're then suggesting that upon #932 being merged you'll take care of handling this PR together with #868 ?

@christiangnrd

Copy link
Copy Markdown
Member

You're then suggesting that upon #932 being merged you'll take care of handling this PR together with #868 ?

I somehow completely missed that #932 existed and that the @info issue had been brought up over there.

And yes I'll add KA.versioninfo(io::IO, ::ROCBackend) = AMDGPU.versioninfo(io) to #868 as soon as JuliaGPU/KernelAbstractions.jl#712 is in main

@luraess

luraess commented Jun 19, 2026

Copy link
Copy Markdown
Member Author

Great. I'll ping on 868 when 932 gets merged and let you then close this in favour of the latter.

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