diff options
| author | Damien Lespiau <damien.lespiau@intel.com> | 2013-01-31 00:26:51 +0000 |
|---|---|---|
| committer | Damien Lespiau <damien.lespiau@intel.com> | 2013-03-04 15:54:41 +0000 |
| commit | 49861a03b6b3c9316ca329dba231623e64930816 (patch) | |
| tree | afb83adeeb469d1f3a9995567b7bf1750295624b /assembler | |
| parent | b21c2e60e9d9c3a2a8482d77a7f7e29f997d7dda (diff) | |
assembler: Introduce set_instruction_saturate()
Also simplify the logic that was setting the saturate bit in the math
instruction.
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Diffstat (limited to 'assembler')
| -rw-r--r-- | assembler/gram.y | 26 |
1 files changed, 14 insertions, 12 deletions
diff --git a/assembler/gram.y b/assembler/gram.y index 917bccf3..43c34f65 100644 --- a/assembler/gram.y +++ b/assembler/gram.y @@ -100,6 +100,8 @@ static int set_instruction_src1_three_src(struct brw_program_instruction *instr, struct src_operand *src); static int set_instruction_src2_three_src(struct brw_program_instruction *instr, struct src_operand *src); +static void set_instruction_saturate(struct brw_program_instruction *instr, + int saturate); static void set_instruction_options(struct brw_program_instruction *instr, struct options options); static void set_instruction_predicate(struct brw_program_instruction *instr, @@ -996,7 +998,7 @@ unaryinstruction: { memset(&$$, 0, sizeof($$)); set_instruction_opcode(&$$, $2); - GEN(&$$)->header.saturate = $4; + set_instruction_saturate(&$$, $4); $6.width = $5; set_instruction_options(&$$, $8); set_instruction_pred_cond(&$$, &$1, &$3, &@3); @@ -1022,7 +1024,7 @@ binaryinstruction: { memset(&$$, 0, sizeof($$)); set_instruction_opcode(&$$, $2); - GEN(&$$)->header.saturate = $4; + set_instruction_saturate(&$$, $4); set_instruction_options(&$$, $9); set_instruction_pred_cond(&$$, &$1, &$3, &@3); $6.width = $5; @@ -1050,7 +1052,7 @@ binaryaccinstruction: { memset(&$$, 0, sizeof($$)); set_instruction_opcode(&$$, $2); - GEN(&$$)->header.saturate = $4; + set_instruction_saturate(&$$, $4); $6.width = $5; set_instruction_options(&$$, $9); set_instruction_pred_cond(&$$, &$1, &$3, &@3); @@ -1083,7 +1085,7 @@ trinaryinstruction: set_instruction_pred_cond(&$$, &$1, &$3, &@3); set_instruction_opcode(&$$, $2); - GEN(&$$)->header.saturate = $4; + set_instruction_saturate(&$$, $4); GEN(&$$)->header.execution_size = $5; if (set_instruction_dest_three_src(&$$, &$6)) @@ -1485,20 +1487,14 @@ msgtarget: NULL_TOKEN GEN(&$$)->bits2.send_gen5.sfid = BRW_SFID_MATH; GEN(&$$)->bits3.generic_gen5.header_present = 0; GEN(&$$)->bits3.math_gen5.function = $2; - if ($3 == BRW_INSTRUCTION_SATURATE) - GEN(&$$)->bits3.math_gen5.saturate = 1; - else - GEN(&$$)->bits3.math_gen5.saturate = 0; + set_instruction_saturate(&$$, $3); GEN(&$$)->bits3.math_gen5.int_type = $4; GEN(&$$)->bits3.math_gen5.precision = BRW_MATH_PRECISION_FULL; GEN(&$$)->bits3.math_gen5.data_type = $5; } else { GEN(&$$)->bits3.generic.msg_target = BRW_SFID_MATH; GEN(&$$)->bits3.math.function = $2; - if ($3 == BRW_INSTRUCTION_SATURATE) - GEN(&$$)->bits3.math.saturate = 1; - else - GEN(&$$)->bits3.math.saturate = 0; + set_instruction_saturate(&$$, $3); GEN(&$$)->bits3.math.int_type = $4; GEN(&$$)->bits3.math.precision = BRW_MATH_PRECISION_FULL; GEN(&$$)->bits3.math.data_type = $5; @@ -2990,6 +2986,12 @@ static int set_instruction_src2_three_src(struct brw_program_instruction *instr, return 0; } +static void set_instruction_saturate(struct brw_program_instruction *instr, + int saturate) +{ + GEN(instr)->header.saturate = saturate; +} + static void set_instruction_options(struct brw_program_instruction *instr, struct options options) { |