summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJonathan Marek <[email protected]>2020-04-12 10:52:34 -0400
committerMarge Bot <[email protected]>2020-04-20 22:01:43 +0000
commit7870d714598f80305d9b44a1ed12e981c67feed2 (patch)
treea7d7fe1fb4923f256ea17d844a641cb360156d51
parent71820c6b02d71e40d413e2d080e87108e64cfeeb (diff)
anv: use common nir_convert_ycbcr
Signed-off-by: Jonathan Marek <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> Acked-by: Jason Ekstrand <[email protected]> Reviewed-by: D Scott Phillips <[email protected]> Reviewed-by: Lionel Landwerlin <[email protected]> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4528>
-rw-r--r--src/intel/vulkan/anv_nir_lower_ycbcr_textures.c131
1 files changed, 8 insertions, 123 deletions
diff --git a/src/intel/vulkan/anv_nir_lower_ycbcr_textures.c b/src/intel/vulkan/anv_nir_lower_ycbcr_textures.c
index c31c229ff40..e55e5317fc7 100644
--- a/src/intel/vulkan/anv_nir_lower_ycbcr_textures.c
+++ b/src/intel/vulkan/anv_nir_lower_ycbcr_textures.c
@@ -25,6 +25,7 @@
#include "anv_private.h"
#include "nir/nir.h"
#include "nir/nir_builder.h"
+#include "nir/nir_vulkan.h"
struct ycbcr_state {
nir_builder *builder;
@@ -34,127 +35,6 @@ struct ycbcr_state {
struct anv_ycbcr_conversion *conversion;
};
-static nir_ssa_def *
-y_range(nir_builder *b,
- nir_ssa_def *y_channel,
- int bpc,
- VkSamplerYcbcrRange range)
-{
- switch (range) {
- case VK_SAMPLER_YCBCR_RANGE_ITU_FULL:
- return y_channel;
- case VK_SAMPLER_YCBCR_RANGE_ITU_NARROW:
- return nir_fmul(b,
- nir_fadd(b,
- nir_fmul(b, y_channel,
- nir_imm_float(b, pow(2, bpc) - 1)),
- nir_imm_float(b, -16.0f * pow(2, bpc - 8))),
- nir_frcp(b, nir_imm_float(b, 219.0f * pow(2, bpc - 8))));
- default:
- unreachable("missing Ycbcr range");
- return NULL;
- }
-}
-
-static nir_ssa_def *
-chroma_range(nir_builder *b,
- nir_ssa_def *chroma_channel,
- int bpc,
- VkSamplerYcbcrRange range)
-{
- switch (range) {
- case VK_SAMPLER_YCBCR_RANGE_ITU_FULL:
- return nir_fadd(b, chroma_channel,
- nir_imm_float(b, -pow(2, bpc - 1) / (pow(2, bpc) - 1.0f)));
- case VK_SAMPLER_YCBCR_RANGE_ITU_NARROW:
- return nir_fmul(b,
- nir_fadd(b,
- nir_fmul(b, chroma_channel,
- nir_imm_float(b, pow(2, bpc) - 1)),
- nir_imm_float(b, -128.0f * pow(2, bpc - 8))),
- nir_frcp(b, nir_imm_float(b, 224.0f * pow(2, bpc - 8))));
- default:
- unreachable("missing Ycbcr range");
- return NULL;
- }
-}
-
-typedef struct nir_const_value_3_4 {
- nir_const_value v[3][4];
-} nir_const_value_3_4;
-
-static const nir_const_value_3_4 *
-ycbcr_model_to_rgb_matrix(VkSamplerYcbcrModelConversion model)
-{
- switch (model) {
- case VK_SAMPLER_YCBCR_MODEL_CONVERSION_YCBCR_601: {
- static const nir_const_value_3_4 bt601 = { {
- { { .f32 = 1.402f }, { .f32 = 1.0f }, { .f32 = 0.0f }, { .f32 = 0.0f } },
- { { .f32 = -0.714136286201022f }, { .f32 = 1.0f }, { .f32 = -0.344136286201022f }, { .f32 = 0.0f } },
- { { .f32 = 0.0f }, { .f32 = 1.0f }, { .f32 = 1.772f }, { .f32 = 0.0f } },
- } };
-
- return &bt601;
- }
- case VK_SAMPLER_YCBCR_MODEL_CONVERSION_YCBCR_709: {
- static const nir_const_value_3_4 bt709 = { {
- { { .f32 = 1.5748031496063f }, { .f32 = 1.0f }, { .f32 = 0.0f }, { .f32 = 0.0f } },
- { { .f32 = -0.468125209181067f }, { .f32 = 1.0f }, { .f32 = -0.187327487470334f }, { .f32 = 0.0f } },
- { { .f32 = 0.0f }, { .f32 = 1.0f }, { .f32 = 1.85563184264242f }, { .f32 = 0.0f } },
- } };
-
- return &bt709;
- }
- case VK_SAMPLER_YCBCR_MODEL_CONVERSION_YCBCR_2020: {
- static const nir_const_value_3_4 bt2020 = { {
- { { .f32 = 1.4746f }, { .f32 = 1.0f }, { .f32 = 0.0f }, { .f32 = 0.0f } },
- { { .f32 = -0.571353126843658f }, { .f32 = 1.0f }, { .f32 = -0.164553126843658f }, { .f32 = 0.0f } },
- { { .f32 = 0.0f }, { .f32 = 1.0f }, { .f32 = 1.8814f }, { .f32 = 0.0f } },
- } };
-
- return &bt2020;
- }
- default:
- unreachable("missing Ycbcr model");
- return NULL;
- }
-}
-
-static nir_ssa_def *
-convert_ycbcr(struct ycbcr_state *state,
- nir_ssa_def *raw_channels,
- uint32_t *bpcs)
-{
- nir_builder *b = state->builder;
- struct anv_ycbcr_conversion *conversion = state->conversion;
-
- nir_ssa_def *expanded_channels =
- nir_vec4(b,
- chroma_range(b, nir_channel(b, raw_channels, 0),
- bpcs[0], conversion->ycbcr_range),
- y_range(b, nir_channel(b, raw_channels, 1),
- bpcs[1], conversion->ycbcr_range),
- chroma_range(b, nir_channel(b, raw_channels, 2),
- bpcs[2], conversion->ycbcr_range),
- nir_imm_float(b, 1.0f));
-
- if (conversion->ycbcr_model == VK_SAMPLER_YCBCR_MODEL_CONVERSION_YCBCR_IDENTITY)
- return expanded_channels;
-
- const nir_const_value_3_4 *conversion_matrix =
- ycbcr_model_to_rgb_matrix(conversion->ycbcr_model);
-
- nir_ssa_def *converted_channels[] = {
- nir_fdot4(b, expanded_channels, nir_build_imm(b, 4, 32, conversion_matrix->v[0])),
- nir_fdot4(b, expanded_channels, nir_build_imm(b, 4, 32, conversion_matrix->v[1])),
- nir_fdot4(b, expanded_channels, nir_build_imm(b, 4, 32, conversion_matrix->v[2]))
- };
-
- return nir_vec4(b,
- converted_channels[0], converted_channels[1],
- converted_channels[2], nir_imm_float(b, 1.0f));
-}
-
/* TODO: we should probably replace this with a push constant/uniform. */
static nir_ssa_def *
get_texture_size(struct ycbcr_state *state, nir_deref_instr *texture)
@@ -436,8 +316,13 @@ try_lower_tex_ycbcr(const struct anv_pipeline_layout *layout,
}
nir_ssa_def *result = nir_vec(builder, swizzled_comp, 4);
- if (state.conversion->ycbcr_model != VK_SAMPLER_YCBCR_MODEL_CONVERSION_RGB_IDENTITY)
- result = convert_ycbcr(&state, result, swizzled_bpcs);
+ if (state.conversion->ycbcr_model != VK_SAMPLER_YCBCR_MODEL_CONVERSION_RGB_IDENTITY) {
+ result = nir_convert_ycbcr_to_rgb(builder,
+ state.conversion->ycbcr_model,
+ state.conversion->ycbcr_range,
+ result,
+ swizzled_bpcs);
+ }
nir_ssa_def_rewrite_uses(&tex->dest.ssa, nir_src_for_ssa(result));
nir_instr_remove(&tex->instr);