radv: remove the LLVM version string when ACO is used
authorSamuel Pitoiset <samuel.pitoiset@gmail.com>
Wed, 6 May 2020 07:32:27 +0000 (09:32 +0200)
committerSamuel Pitoiset <samuel.pitoiset@gmail.com>
Fri, 8 May 2020 06:45:26 +0000 (08:45 +0200)
commitb1ef1c1211373f9336aaf87de865ef7f0c29fc44
tree039ecd0f98ed08e4a86b46b05a5d1e36afc2a6e0
parentee2aef3ea531a03dbd50a78c943ca6e7cc99c7a7
radv: remove the LLVM version string when ACO is used

Now that ACO supports all shader stages (the only exception is NGG
GS on Navi10 but it fallbacks to legacy GS) it makes sense to remove
the LLVM version string reported as part of the device name.

The LLVM version string was added in the past for some Feral games
to workaround LLVM issues by detecting the version. With ACO, this
is unecessary because the Mesa version is enough to eventually enable
specific shader workarounds.

When the LLVM version string is missing, it is assumed that an old
LLVM is used and workarounds are automatically applied. The only
Vulkan games that might be affected is Shadow of The Tomb Raider
but the impact should be fairly small.

Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Acked-by: Daniel Schürmann <daniel@schuermann.dev>
Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4911>
src/amd/vulkan/radv_device.c