diff options
author | Neil Roberts <[email protected]> | 2017-10-31 15:05:33 +0100 |
---|---|---|
committer | Neil Roberts <[email protected]> | 2017-10-31 23:28:00 +0100 |
commit | b697ece10aa041b8653eb184d73dcf5b846729a3 (patch) | |
tree | a64f86796c6a5cd7906b014b3b0a71ddf542174b /src/glx | |
parent | 9e849eb8bb97259136b40dc2b06f42a81cfd3dae (diff) |
nir/opt_intrinsics: Fix values for gl_SubGroupG{e,t}MaskARB
Previously the values were calculated by just shifting ~0 by the
invocation ID. This would end up including bits that are higher than
gl_SubGroupSizeARB. The corresponding CTS test effectively requires that
these high bits be zero so it was failing. There is a Piglit test as
well but this appears to checking the wrong values so it passes.
For the two greater-than bitmasks, this patch adds an extra mask with
(~0>>(64-gl_SubGroupSizeARB)) to force these bits to zero.
Fixes: KHR-GL45.shader_ballot_tests.ShaderBallotBitmasks
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102680#c3
Reviewed-by: Jason Ekstrand <[email protected]>
Cc: [email protected]
Signed-off-by: Neil Roberts <[email protected]>
Diffstat (limited to 'src/glx')
0 files changed, 0 insertions, 0 deletions