nir+drivers: add helpers to get # of src/dest components
Add helpers to get the number of src/dest components for an intrinsic, and update spots that were open-coding this logic to use the helpers instead. Signed-off-by: Rob Clark <robdclark@gmail.com> Reviewed-by: Eric Anholt <eric@anholt.net>
This commit is contained in:
@@ -1146,6 +1146,28 @@ typedef struct {
|
||||
|
||||
extern const nir_intrinsic_info nir_intrinsic_infos[nir_num_intrinsics];
|
||||
|
||||
static inline unsigned
|
||||
nir_intrinsic_src_components(nir_intrinsic_instr *intr, unsigned srcn)
|
||||
{
|
||||
const nir_intrinsic_info *info = &nir_intrinsic_infos[intr->intrinsic];
|
||||
assert(srcn < info->num_srcs);
|
||||
if (info->src_components[srcn])
|
||||
return info->src_components[srcn];
|
||||
else
|
||||
return intr->num_components;
|
||||
}
|
||||
|
||||
static inline unsigned
|
||||
nir_intrinsic_dest_components(nir_intrinsic_instr *intr)
|
||||
{
|
||||
const nir_intrinsic_info *info = &nir_intrinsic_infos[intr->intrinsic];
|
||||
if (!info->has_dest)
|
||||
return 0;
|
||||
else if (info->dest_components)
|
||||
return info->dest_components;
|
||||
else
|
||||
return intr->num_components;
|
||||
}
|
||||
|
||||
#define INTRINSIC_IDX_ACCESSORS(name, flag, type) \
|
||||
static inline type \
|
||||
|
||||
@@ -257,10 +257,7 @@ copy_prop_instr(nir_instr *instr)
|
||||
nir_intrinsic_instr *intrin = nir_instr_as_intrinsic(instr);
|
||||
for (unsigned i = 0;
|
||||
i < nir_intrinsic_infos[intrin->intrinsic].num_srcs; i++) {
|
||||
unsigned num_components =
|
||||
nir_intrinsic_infos[intrin->intrinsic].src_components[i];
|
||||
if (!num_components)
|
||||
num_components = intrin->num_components;
|
||||
unsigned num_components = nir_intrinsic_src_components(intrin, i);
|
||||
|
||||
while (copy_prop_src(&intrin->src[i], instr, NULL, num_components))
|
||||
progress = true;
|
||||
|
||||
@@ -483,10 +483,7 @@ validate_intrinsic_instr(nir_intrinsic_instr *instr, validate_state *state)
|
||||
|
||||
unsigned num_srcs = nir_intrinsic_infos[instr->intrinsic].num_srcs;
|
||||
for (unsigned i = 0; i < num_srcs; i++) {
|
||||
unsigned components_read =
|
||||
nir_intrinsic_infos[instr->intrinsic].src_components[i];
|
||||
if (components_read == 0)
|
||||
components_read = instr->num_components;
|
||||
unsigned components_read = nir_intrinsic_src_components(instr, i);
|
||||
|
||||
validate_assert(state, components_read > 0);
|
||||
|
||||
@@ -499,10 +496,7 @@ validate_intrinsic_instr(nir_intrinsic_instr *instr, validate_state *state)
|
||||
}
|
||||
|
||||
if (nir_intrinsic_infos[instr->intrinsic].has_dest) {
|
||||
unsigned components_written =
|
||||
nir_intrinsic_infos[instr->intrinsic].dest_components;
|
||||
if (components_written == 0)
|
||||
components_written = instr->num_components;
|
||||
unsigned components_written = nir_intrinsic_dest_components(instr);
|
||||
|
||||
validate_assert(state, components_written > 0);
|
||||
|
||||
|
||||
Reference in New Issue
Block a user