radv: only expose shaderFloat16 for chips with double rate fp16
authorSamuel Pitoiset <samuel.pitoiset@gmail.com>
Sun, 5 Apr 2020 07:25:18 +0000 (09:25 +0200)
committerSamuel Pitoiset <samuel.pitoiset@gmail.com>
Thu, 9 Apr 2020 11:34:36 +0000 (13:34 +0200)
commit1d74c6565d74188efea8bdef3dd33c1e4aa21f60
treed0249a89c1095716fb1903a2318ff5caa5f74fcd
parenta3113e07b90d56a09e53b5bf2f77171d13a049d6
radv: only expose shaderFloat16 for chips with double rate fp16

This disables shaderFloat16 on GFX8 because only GFX9+ supports
double rate packed math.

This improves consistency regarding other AMD Vulkan drivers and
it makes no sense to enable that feature without packed math.

This also reduces performance with Wolfeinstein Youngblood if
fp16 is forced enabled on GFX8, while it's similar on GFX9.

We might re-introduce that feature in the future with ACO support
if it ends up being faster and correct.

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/4453>
src/amd/vulkan/radv_device.c
src/amd/vulkan/radv_shader.c