From 7092c1218a04be9369750a28501cfa44bc2334fd Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Tue, 10 Jan 2023 15:09:12 -0800 Subject: [PATCH] intel/compiler: Use more symbolic source names in components_read() Rather than hardcoding source 1, source 2, etc. Reviewed-by: Lionel Landwerlin Reviewed-by: Rohan Garg Part-of: --- src/intel/compiler/brw_fs.cpp | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/src/intel/compiler/brw_fs.cpp b/src/intel/compiler/brw_fs.cpp index a816ad9d438..50191448d3d 100644 --- a/src/intel/compiler/brw_fs.cpp +++ b/src/intel/compiler/brw_fs.cpp @@ -747,13 +747,13 @@ fs_inst::components_read(unsigned i) const case SHADER_OPCODE_A64_UNTYPED_READ_LOGICAL: case SHADER_OPCODE_A64_OWORD_BLOCK_READ_LOGICAL: case SHADER_OPCODE_A64_UNALIGNED_OWORD_BLOCK_READ_LOGICAL: - assert(src[2].file == IMM); + assert(src[A64_LOGICAL_ARG].file == IMM); return 1; case SHADER_OPCODE_A64_OWORD_BLOCK_WRITE_LOGICAL: - assert(src[2].file == IMM); - if (i == 1) { /* data to write */ - const unsigned comps = src[2].ud / exec_size; + assert(src[A64_LOGICAL_ARG].file == IMM); + if (i == A64_LOGICAL_SRC) { /* data to write */ + const unsigned comps = src[A64_LOGICAL_ARG].ud / exec_size; assert(comps > 0); return comps; } else { @@ -775,12 +775,13 @@ fs_inst::components_read(unsigned i) const } case SHADER_OPCODE_A64_UNTYPED_WRITE_LOGICAL: - assert(src[2].file == IMM); - return i == 1 ? src[2].ud : 1; + assert(src[A64_LOGICAL_ARG].file == IMM); + return i == A64_LOGICAL_SRC ? src[A64_LOGICAL_ARG].ud : 1; case SHADER_OPCODE_A64_UNTYPED_ATOMIC_LOGICAL: - assert(src[2].file == IMM); - return i == 1 ? lsc_op_num_data_values(src[2].ud) : 1; + assert(src[A64_LOGICAL_ARG].file == IMM); + return i == A64_LOGICAL_SRC ? + lsc_op_num_data_values(src[A64_LOGICAL_ARG].ud) : 1; case SHADER_OPCODE_BYTE_SCATTERED_READ_LOGICAL: case SHADER_OPCODE_DWORD_SCATTERED_READ_LOGICAL: