mesa icon indicating copy to clipboard operation
mesa copied to clipboard

visual corruption in Heroes of the Storm - supposedly game bug, but fixed in proprietary compilers

Open aufkrawall opened this issue 6 years ago • 4 comments

Hello, there is this kind of corruption: hots

The issue occurs with: dxvk radv-aco dxvk radv-llvm dxvk amdvlk-llvm wined3d11 radeonsi-llvm

The issue does not occur with: AMD, Nvidia and Intel native D3D11 dxvk amdvlk-proprietary dxvk nvidia

Quoting Rhys Perry from here: https://bugs.freedesktop.org/show_bug.cgi?id=109394

I think this is a game bug: it's using implicit derivatives when sampling textures in non-uniform control flow. In the GLSL and SPIR-V spec, the results are undefined. I think it's the same with HLSL. Since the control flow is non-uniform and (IIRC) LLVM seems to move the operations creating the texture coordinates into it, I think it's causing discontinuities and messing up the LOD calculation.

Would it be possible to fix it in ACO?

aufkrawall avatar Jul 04 '19 21:07 aufkrawall

It is probably best to report the issue to Blizzard first.

baryluk avatar Oct 16 '19 14:10 baryluk

Somebody may do so, but nothing will happen. They don't even fix undefined cubemap behavior that badly affects AMD and Intel D3D11 on native Windows.

aufkrawall avatar Oct 16 '19 15:10 aufkrawall

Somebody may do so, but nothing will happen. They don't even fix undefined cubemap behavior that badly affects AMD and Intel D3D11 on native Windows.

I understand that. But it is worth a try nevertheless. Even if likely hood of fixing it 2%.

baryluk avatar Oct 16 '19 15:10 baryluk

Since the game receives only content updates and no engine fixes: Any chance to address this in between of the other goodness? The game is not playable with amdvlk-pro driver due to missing shader cache, while Nvidia users don't have the artifacts, but have a shader cache at the same time.

aufkrawall avatar Dec 14 '19 20:12 aufkrawall

Hi,

This repo is no longer being maintained since ACO was merged in upstream Mesa since late 2019. If you have issues with it, please open an issue in upstream Mesa here: https://gitlab.freedesktop.org/mesa/mesa/-/issues - select the Radeon Vulkan bug report template and fill out the details.

Venemo avatar May 17 '23 15:05 Venemo

This is fixed in RADV/ACO, though can't say since when. Funny thing is that since AMD have switched to LLPC also for their Windows Vulkan driver with RDNA3 (and PRO driver on Linux uses it too with RDNA3), it shows the issue as well. Unlike RDNA2 driver, that is still using the proprietary compiler (apart from amdvlk-open).

aufkrawall avatar Jun 19 '23 03:06 aufkrawall

This is fixed in RADV/ACO, though can't say since when.

Probably https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/22636

pendingchaos avatar Jun 19 '23 10:06 pendingchaos