From 7a75b55a01d355090d186357896e3cb141b9775e Mon Sep 17 00:00:00 2001 From: Jason Ekstrand Date: Wed, 1 Apr 2015 18:15:42 -0700 Subject: i965/fs_inst: Get rid of the effective_width field The effective_width field was an ill-concieved hack to get around issues in the LOAD_PAYLOAD instruction. Now that the LOAD_PAYLOAD instruction is far more sane, this field can die. Reviewed-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_fs.cpp | 25 ------------------------- 1 file changed, 25 deletions(-) (limited to 'src/mesa/drivers/dri/i965/brw_fs.cpp') diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp index d0a3bdd9264..3bf58668601 100644 --- a/src/mesa/drivers/dri/i965/brw_fs.cpp +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp @@ -90,31 +90,6 @@ fs_inst::init(enum opcode opcode, uint8_t exec_size, const fs_reg &dst, } assert(this->exec_size != 0); - for (unsigned i = 0; i < sources; ++i) { - switch (this->src[i].file) { - case BAD_FILE: - this->src[i].effective_width = 8; - break; - case GRF: - case HW_REG: - case ATTR: - assert(this->src[i].width > 0); - if (this->src[i].width == 1) { - this->src[i].effective_width = this->exec_size; - } else { - this->src[i].effective_width = this->src[i].width; - } - break; - case IMM: - case UNIFORM: - this->src[i].effective_width = this->exec_size; - break; - default: - unreachable("Invalid source register file"); - } - } - this->dst.effective_width = this->exec_size; - this->conditional_mod = BRW_CONDITIONAL_NONE; /* This will be the case for almost all instructions. */ -- cgit v1.2.3