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

Compiling a compute shader for GFX1010 shows incorrect 'wave_size' #58

Open
AdamJMiles opened this issue Mar 10, 2020 · 1 comment
Open

Comments

@AdamJMiles
Copy link

AdamJMiles commented Mar 10, 2020

If you compile a compute shader with [numthreads(64,1,1)] the top of the ISA output says "wave_size(64)". However it's apparent from the generated asm that the code generated was designed for wave32.

Evidence for this is manipulation of only exec_lo (exec_hi is never touched) and the fact that DX12 Wave Intrinsics such as "WaveActiveSum" generate code that only sums across 32 lanes.

If GFX1010 uses Wave32 now as its native wave size for Compute then the "wave_size(64)" at the top of the ISA output should read "wave_size(32)" instead.

@AmitBM
Copy link
Contributor

AmitBM commented Mar 25, 2020

Hi AdamJMiles,

Thanks for reporting this issue. A fix will be delivered in one of AMD's future driver releases. You do not need to update RGA to consume this fix.

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

No branches or pull requests

2 participants