diff options
| author | Eric Anholt <[email protected]> | 2018-12-12 11:29:29 -0800 |
|---|---|---|
| committer | Eric Anholt <[email protected]> | 2018-12-12 16:09:48 -0800 |
| commit | 06fbcd2cd5cc5702c9039c26d20082a99bc157bf (patch) | |
| tree | 957a777142fd3dfe57efad854c5ada99e98793af /src/intel | |
| parent | d3e046e76c06978d92bc7311bf02926e888159dc (diff) | |
intel: Simplify the half-float packing in image load/store lowering.
This was noted by Jason in review when I tried to make a helper for the
old path.
Reviewed-by: Jason Ekstrand <[email protected]>
Diffstat (limited to 'src/intel')
| -rw-r--r-- | src/intel/compiler/brw_nir_lower_image_load_store.c | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/src/intel/compiler/brw_nir_lower_image_load_store.c b/src/intel/compiler/brw_nir_lower_image_load_store.c index 269dbf8e283..d1547c98c54 100644 --- a/src/intel/compiler/brw_nir_lower_image_load_store.c +++ b/src/intel/compiler/brw_nir_lower_image_load_store.c @@ -544,14 +544,8 @@ convert_color_for_store(nir_builder *b, const struct gen_device_info *devinfo, break; case ISL_SFLOAT: - if (image.bits[0] == 16) { - nir_ssa_def *f16comps[4]; - for (unsigned i = 0; i < image.chans; i++) { - f16comps[i] = nir_pack_half_2x16_split(b, nir_channel(b, color, i), - nir_imm_float(b, 0)); - } - color = nir_vec(b, f16comps, image.chans); - } + if (image.bits[0] == 16) + color = nir_pack_half_2x16_split(b, color, nir_imm_float(b, 0)); break; case ISL_UINT: |