On Sat, May 05, 2012 at 09:01:42PM +0200, Kai Wasserbäch wrote:
> 
> Signed-off-by: Kai Wasserbäch <k...@dev.carbon-project.org>
> ---
>  This patch should enable CEIL on r600g for r600 to Evergreen. Not sure if
>  Cayman's registers are already covered by the EG definitions.

The opcode for CEIL is the same on Evergreen and Cayman, so you can
enable it in cm_shader_tgsi_instruction as well.

With that change, the patch is:

Reviewed-by: Tom Stellard <thomas.stell...@amd.com>

> 
>  As I have never touched any core driver part before, please review carefully.
> 
>  src/gallium/drivers/r600/r600_shader.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/gallium/drivers/r600/r600_shader.c 
> b/src/gallium/drivers/r600/r600_shader.c
> index fa29b36..4e1f6b6 100644
> --- a/src/gallium/drivers/r600/r600_shader.c
> +++ b/src/gallium/drivers/r600/r600_shader.c
> @@ -5154,7 +5154,7 @@ static struct r600_shader_tgsi_instruction 
> r600_shader_tgsi_instruction[] = {
>       {80,                    0, V_SQ_ALU_WORD1_OP2_SQ_OP2_INST_NOP, 
> tgsi_unsupported},
>       {TGSI_OPCODE_PUSHA,     0, V_SQ_ALU_WORD1_OP2_SQ_OP2_INST_NOP, 
> tgsi_unsupported},
>       {TGSI_OPCODE_POPA,      0, V_SQ_ALU_WORD1_OP2_SQ_OP2_INST_NOP, 
> tgsi_unsupported},
> -     {TGSI_OPCODE_CEIL,      0, V_SQ_ALU_WORD1_OP2_SQ_OP2_INST_NOP, 
> tgsi_unsupported},
> +     {TGSI_OPCODE_CEIL,      0, V_SQ_ALU_WORD1_OP2_SQ_OP2_INST_CEIL, 
> tgsi_op2},
>       {TGSI_OPCODE_I2F,       0, V_SQ_ALU_WORD1_OP2_SQ_OP2_INST_INT_TO_FLT, 
> tgsi_op2_trans},
>       {TGSI_OPCODE_NOT,       0, V_SQ_ALU_WORD1_OP2_SQ_OP2_INST_NOT_INT, 
> tgsi_op2},
>       {TGSI_OPCODE_TRUNC,     0, V_SQ_ALU_WORD1_OP2_SQ_OP2_INST_TRUNC, 
> tgsi_op2},
> @@ -5328,7 +5328,7 @@ static struct r600_shader_tgsi_instruction 
> eg_shader_tgsi_instruction[] = {
>       {80,                    0, EG_V_SQ_ALU_WORD1_OP2_SQ_OP2_INST_NOP, 
> tgsi_unsupported},
>       {TGSI_OPCODE_PUSHA,     0, EG_V_SQ_ALU_WORD1_OP2_SQ_OP2_INST_NOP, 
> tgsi_unsupported},
>       {TGSI_OPCODE_POPA,      0, EG_V_SQ_ALU_WORD1_OP2_SQ_OP2_INST_NOP, 
> tgsi_unsupported},
> -     {TGSI_OPCODE_CEIL,      0, EG_V_SQ_ALU_WORD1_OP2_SQ_OP2_INST_NOP, 
> tgsi_unsupported},
> +     {TGSI_OPCODE_CEIL,      0, EG_V_SQ_ALU_WORD1_OP2_SQ_OP2_INST_CEIL, 
> tgsi_op2},
>       {TGSI_OPCODE_I2F,       0, 
> EG_V_SQ_ALU_WORD1_OP2_SQ_OP2_INST_INT_TO_FLT, tgsi_op2_trans},
>       {TGSI_OPCODE_NOT,       0, EG_V_SQ_ALU_WORD1_OP2_SQ_OP2_INST_NOT_INT, 
> tgsi_op2},
>       {TGSI_OPCODE_TRUNC,     0, EG_V_SQ_ALU_WORD1_OP2_SQ_OP2_INST_TRUNC, 
> tgsi_op2},
> -- 
> 1.7.10
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to