diff options
| author | Paulo Zanoni <[email protected]> | 2019-07-10 16:48:01 -0700 |
|---|---|---|
| committer | Caio Marcelo de Oliveira Filho <[email protected]> | 2019-08-12 15:16:23 -0700 |
| commit | 6ba4717924f2bdf1e77c62f117b4f6b98ea2f75e (patch) | |
| tree | b6caaab39b17cf8620483bafda3dc45e82e48c54 /src/intel/compiler/brw_fs.h | |
| parent | 75b3868dcc81e9b629e997f354aae691b7933f32 (diff) | |
intel/compiler: don't instantiate a builder for each instruction
Don't instantiate a builder for each instruction during
lower_integer_multiplication(). Instantiate one only when needed.
On the other hand, these unneeded builders don't seem to cost much to
init, so I don't expect any significant difference in performance:
this is mostly about code organization.
Reviewed-by: Matt Turner <[email protected]>
Reviewed-by: Caio Marcelo de Oliveira Filho <[email protected]>
Signed-off-by: Paulo Zanoni <[email protected]>
Diffstat (limited to 'src/intel/compiler/brw_fs.h')
| -rw-r--r-- | src/intel/compiler/brw_fs.h | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/intel/compiler/brw_fs.h b/src/intel/compiler/brw_fs.h index 82d572916ce..fb78fa829a0 100644 --- a/src/intel/compiler/brw_fs.h +++ b/src/intel/compiler/brw_fs.h @@ -406,10 +406,8 @@ private: void resolve_inot_sources(const brw::fs_builder &bld, nir_alu_instr *instr, fs_reg *op); - void lower_mul_dword_inst(fs_inst *inst, bblock_t *block, - const brw::fs_builder &ibld); - void lower_mulh_inst(fs_inst *inst, bblock_t *block, - const brw::fs_builder &ibld); + void lower_mul_dword_inst(fs_inst *inst, bblock_t *block); + void lower_mulh_inst(fs_inst *inst, bblock_t *block); }; /** |