diff options
| author | Jason Ekstrand <jason.ekstrand@intel.com> | 2018-11-12 15:58:18 -0600 |
|---|---|---|
| committer | Jason Ekstrand <jason.ekstrand@intel.com> | 2018-11-15 19:59:27 -0600 |
| commit | b77d68b78e2bf623bf9ba3f28945cf2080185a9d (patch) | |
| tree | 129ec21f92efd1dda3a795bfa5892318fff2c702 /src/compiler/nir/nir_lower_io.c | |
| parent | 1f29f4db1e867357a119c0c7c34fb54dc27fb682 (diff) | |
nir/builder: Add iadd_imm and imul_imm helpers
The pattern of adding or multiplying an integer by an immediate is
fairly common especially in deref chain handling. This adds a helper
for it and uses it a few places. The advantage to the helper is that
it automatically handles bit sizes for you.
Reviewed-by: Samuel Iglesias Gonsálvez <siglesias@igalia.com>
Reviewed-by: Karol Herbst <kherbst@redhat.com>
Diffstat (limited to 'src/compiler/nir/nir_lower_io.c')
| -rw-r--r-- | src/compiler/nir/nir_lower_io.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/compiler/nir/nir_lower_io.c b/src/compiler/nir/nir_lower_io.c index b3595bb19d5..f3377eaec8f 100644 --- a/src/compiler/nir/nir_lower_io.c +++ b/src/compiler/nir/nir_lower_io.c @@ -127,8 +127,7 @@ get_io_offset(nir_builder *b, nir_deref_instr *deref, unsigned size = type_size((*p)->type); nir_ssa_def *mul = - nir_imul(b, nir_imm_int(b, size), - nir_ssa_for_src(b, (*p)->arr.index, 1)); + nir_imul_imm(b, nir_ssa_for_src(b, (*p)->arr.index, 1), size); offset = nir_iadd(b, offset, mul); } else if ((*p)->deref_type == nir_deref_type_struct) { @@ -139,7 +138,7 @@ get_io_offset(nir_builder *b, nir_deref_instr *deref, for (unsigned i = 0; i < (*p)->strct.index; i++) { field_offset += type_size(glsl_get_struct_field(parent->type, i)); } - offset = nir_iadd(b, offset, nir_imm_int(b, field_offset)); + offset = nir_iadd_imm(b, offset, field_offset); } else { unreachable("Unsupported deref type"); } |