I'm disappointed that the compiler didn't warn me about use of
uninitialized uc in these paths. Just use the incoming clear color
instead of the packing temporary if we're doing our own packing.
Fixes GTF-GLES3.gtf.GL3Tests.color_buffer_float.color_buffer_float_clamp_*
break;
case V3D_INTERNAL_TYPE_8I:
case V3D_INTERNAL_TYPE_8UI:
break;
case V3D_INTERNAL_TYPE_8I:
case V3D_INTERNAL_TYPE_8UI:
- job->clear_color[i][0] = ((uc.ui[0] & 0xff) |
- (uc.ui[1] & 0xff) << 8 |
- (uc.ui[2] & 0xff) << 16 |
- (uc.ui[3] & 0xff) << 24);
+ job->clear_color[i][0] = ((color->ui[0] & 0xff) |
+ (color->ui[1] & 0xff) << 8 |
+ (color->ui[2] & 0xff) << 16 |
+ (color->ui[3] & 0xff) << 24);
break;
case V3D_INTERNAL_TYPE_16F:
util_pack_color(color->f, PIPE_FORMAT_R16G16B16A16_FLOAT,
break;
case V3D_INTERNAL_TYPE_16F:
util_pack_color(color->f, PIPE_FORMAT_R16G16B16A16_FLOAT,
break;
case V3D_INTERNAL_TYPE_16I:
case V3D_INTERNAL_TYPE_16UI:
break;
case V3D_INTERNAL_TYPE_16I:
case V3D_INTERNAL_TYPE_16UI:
- job->clear_color[i][0] = ((uc.ui[0] & 0xffff) |
- uc.ui[1] << 16);
- job->clear_color[i][1] = ((uc.ui[2] & 0xffff) |
- uc.ui[3] << 16);
+ job->clear_color[i][0] = ((color->ui[0] & 0xffff) |
+ color->ui[1] << 16);
+ job->clear_color[i][1] = ((color->ui[2] & 0xffff) |
+ color->ui[3] << 16);
break;
case V3D_INTERNAL_TYPE_32F:
case V3D_INTERNAL_TYPE_32I:
break;
case V3D_INTERNAL_TYPE_32F:
case V3D_INTERNAL_TYPE_32I: