Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net> On Tue, Mar 13, 2018 at 2:39 PM, Rob Clark <robdcl...@gmail.com> wrote:
> Signed-off-by: Rob Clark <robdcl...@gmail.com> > --- > Now with switch (and also u64<-u64 copy instead of f64<-f64 for 64b).. > > src/compiler/nir/nir_lower_load_const_to_scalar.c | 19 > +++++++++++++++---- > 1 file changed, 15 insertions(+), 4 deletions(-) > > diff --git a/src/compiler/nir/nir_lower_load_const_to_scalar.c > b/src/compiler/nir/nir_lower_load_const_to_scalar.c > index e494facfd21..39447d42c23 100644 > --- a/src/compiler/nir/nir_lower_load_const_to_scalar.c > +++ b/src/compiler/nir/nir_lower_load_const_to_scalar.c > @@ -50,11 +50,22 @@ lower_load_const_instr_scalar(nir_load_const_instr > *lower) > for (unsigned i = 0; i < lower->def.num_components; i++) { > nir_load_const_instr *load_comp = > nir_load_const_instr_create(b.shader, 1, lower->def.bit_size); > - if (lower->def.bit_size == 64) > - load_comp->value.f64[0] = lower->value.f64[i]; > - else > + switch (lower->def.bit_size) { > + case 64: > + load_comp->value.u64[0] = lower->value.u64[i]; > + break; > + case 32: > load_comp->value.u32[0] = lower->value.u32[i]; > - assert(lower->def.bit_size == 64 || lower->def.bit_size == 32); > + break; > + case 16: > + load_comp->value.u16[0] = lower->value.u16[i]; > + break; > + case 8: > + load_comp->value.u8[0] = lower->value.u8[i]; > + break; > + default: > + assert(!"invalid bit size"); > + } > nir_builder_instr_insert(&b, &load_comp->instr); > loads[i] = &load_comp->def; > } > -- > 2.14.3 > >
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev