Commit fe3b7fe9 authored by Lionel Landwerlin's avatar Lionel Landwerlin

anv/lower_ycbcr: make sure to set 0s on all components

To play around with debugging, we might want to disable one or the
other component. Having 0s as default values makes this work.
Otherwise we might have NULL components, leading to crashes.
Signed-off-by: Lionel Landwerlin's avatarLionel Landwerlin <>
Reviewed-by: Jason Ekstrand's avatarJason Ekstrand <>
parent ee8d65c2
......@@ -373,11 +373,11 @@ try_lower_tex_ycbcr(struct anv_pipeline_layout *layout,
uint8_t y_bpc = y_isl_layout->channels_array[0].bits;
/* |ycbcr_comp| holds components in the order : Cr-Y-Cb */
nir_ssa_def *ycbcr_comp[5] = { NULL, NULL, NULL,
/* Use extra 2 channels for following swizzle */
nir_imm_float(builder, 1.0f),
nir_imm_float(builder, 0.0f),
nir_ssa_def *zero = nir_imm_float(builder, 0.0f);
nir_ssa_def *one = nir_imm_float(builder, 1.0f);
/* Use extra 2 channels for following swizzle */
nir_ssa_def *ycbcr_comp[5] = { zero, zero, zero, one, zero };
uint8_t ycbcr_bpcs[5];
memset(ycbcr_bpcs, y_bpc, sizeof(ycbcr_bpcs));
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment