Mesa (master): intel/compiler: Use ARRAY_SIZE()
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Wed Jan 22 00:51:23 UTC 2020
Module: Mesa
Branch: master
Commit: c69f3ece61d8d48ee97fad3555858a48b1214f02
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=c69f3ece61d8d48ee97fad3555858a48b1214f02
Author: Matt Turner <mattst88 at gmail.com>
Date: Thu Jan 16 14:22:14 2020 -0800
intel/compiler: Use ARRAY_SIZE()
Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira at intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/2635>
---
src/intel/compiler/test_eu_validate.cpp | 44 ++++++++++++++++-----------------
1 file changed, 22 insertions(+), 22 deletions(-)
diff --git a/src/intel/compiler/test_eu_validate.cpp b/src/intel/compiler/test_eu_validate.cpp
index 46dee15e271..7d7daa4581a 100644
--- a/src/intel/compiler/test_eu_validate.cpp
+++ b/src/intel/compiler/test_eu_validate.cpp
@@ -451,7 +451,7 @@ TEST_P(validation_test, vstride_on_align16_must_be_0_or_4)
brw_set_default_access_mode(p, BRW_ALIGN_16);
- for (unsigned i = 0; i < sizeof(vstride) / sizeof(vstride[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(vstride); i++) {
brw_ADD(p, g0, g0, g0);
brw_inst_set_src0_vstride(&devinfo, last_inst, vstride[i].vstride);
@@ -460,7 +460,7 @@ TEST_P(validation_test, vstride_on_align16_must_be_0_or_4)
clear_instructions(p);
}
- for (unsigned i = 0; i < sizeof(vstride) / sizeof(vstride[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(vstride); i++) {
brw_ADD(p, g0, g0, g0);
brw_inst_set_src1_vstride(&devinfo, last_inst, vstride[i].vstride);
@@ -795,7 +795,7 @@ TEST_P(validation_test, packed_byte_destination)
{ BRW_REGISTER_TYPE_B , BRW_REGISTER_TYPE_D , 0, 0, 0, false },
};
- for (unsigned i = 0; i < sizeof(move) / sizeof(move[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(move); i++) {
brw_MOV(p, retype(g0, move[i].dst_type), retype(g0, move[i].src_type));
brw_inst_set_src0_negate(&devinfo, last_inst, move[i].neg);
brw_inst_set_src0_abs(&devinfo, last_inst, move[i].abs);
@@ -892,7 +892,7 @@ TEST_P(validation_test, byte_64bit_conversion)
if (devinfo.gen < 8)
return;
- for (unsigned i = 0; i < sizeof(inst) / sizeof(inst[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(inst); i++) {
if (!devinfo.has_64bit_types && type_sz(inst[i].src_type) == 8)
continue;
@@ -988,7 +988,7 @@ TEST_P(validation_test, half_float_conversion)
if (devinfo.gen < 8)
return;
- for (unsigned i = 0; i < sizeof(inst) / sizeof(inst[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(inst); i++) {
if (!devinfo.has_64bit_types &&
(type_sz(inst[i].src_type) == 8 || type_sz(inst[i].dst_type) == 8)) {
continue;
@@ -1068,7 +1068,7 @@ TEST_P(validation_test, mixed_float_source_indirect_addressing)
if (devinfo.gen < 8)
return;
- for (unsigned i = 0; i < sizeof(inst) / sizeof(inst[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(inst); i++) {
brw_ADD(p, retype(g0, inst[i].dst_type),
retype(g0, inst[i].src0_type),
retype(g0, inst[i].src1_type));
@@ -1122,7 +1122,7 @@ TEST_P(validation_test, mixed_float_align1_simd16)
if (devinfo.gen < 8)
return;
- for (unsigned i = 0; i < sizeof(inst) / sizeof(inst[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(inst); i++) {
brw_ADD(p, retype(g0, inst[i].dst_type),
retype(g0, inst[i].src0_type),
retype(g0, inst[i].src1_type));
@@ -1189,7 +1189,7 @@ TEST_P(validation_test, mixed_float_align1_packed_fp16_dst_acc_read_offset_0)
if (devinfo.gen < 8)
return;
- for (unsigned i = 0; i < sizeof(inst) / sizeof(inst[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(inst); i++) {
brw_ADD(p, retype(g0, inst[i].dst_type),
retype(inst[i].read_acc ? acc0 : g0, inst[i].src0_type),
retype(g0, inst[i].src1_type));
@@ -1265,7 +1265,7 @@ TEST_P(validation_test, mixed_float_fp16_dest_with_acc)
if (devinfo.gen < 8)
return;
- for (unsigned i = 0; i < sizeof(inst) / sizeof(inst[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(inst); i++) {
if (inst[i].opcode == BRW_OPCODE_MAC) {
brw_MAC(p, retype(g0, inst[i].dst_type),
retype(g0, inst[i].src0_type),
@@ -1332,7 +1332,7 @@ TEST_P(validation_test, mixed_float_align1_math_strided_fp16_inputs)
if (devinfo.gen < 9)
return;
- for (unsigned i = 0; i < sizeof(inst) / sizeof(inst[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(inst); i++) {
gen6_math(p, retype(g0, inst[i].dst_type),
BRW_MATH_FUNCTION_POW,
retype(g0, inst[i].src0_type),
@@ -1407,7 +1407,7 @@ TEST_P(validation_test, mixed_float_align1_packed_fp16_dst)
if (devinfo.gen < 8)
return;
- for (unsigned i = 0; i < sizeof(inst) / sizeof(inst[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(inst); i++) {
brw_ADD(p, retype(g0, inst[i].dst_type),
retype(g0, inst[i].src0_type),
retype(g0, inst[i].src1_type));
@@ -1478,7 +1478,7 @@ TEST_P(validation_test, mixed_float_align16_packed_data)
brw_set_default_access_mode(p, BRW_ALIGN_16);
- for (unsigned i = 0; i < sizeof(inst) / sizeof(inst[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(inst); i++) {
brw_ADD(p, retype(g0, inst[i].dst_type),
retype(g0, inst[i].src0_type),
retype(g0, inst[i].src1_type));
@@ -1529,7 +1529,7 @@ TEST_P(validation_test, mixed_float_align16_no_simd16)
brw_set_default_access_mode(p, BRW_ALIGN_16);
- for (unsigned i = 0; i < sizeof(inst) / sizeof(inst[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(inst); i++) {
brw_ADD(p, retype(g0, inst[i].dst_type),
retype(g0, inst[i].src0_type),
retype(g0, inst[i].src1_type));
@@ -1580,7 +1580,7 @@ TEST_P(validation_test, mixed_float_align16_no_acc_read)
brw_set_default_access_mode(p, BRW_ALIGN_16);
- for (unsigned i = 0; i < sizeof(inst) / sizeof(inst[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(inst); i++) {
brw_ADD(p, retype(g0, inst[i].dst_type),
retype(inst[i].read_acc ? acc0 : g0, inst[i].src0_type),
retype(g0, inst[i].src1_type));
@@ -1635,7 +1635,7 @@ TEST_P(validation_test, mixed_float_align16_math_packed_format)
brw_set_default_access_mode(p, BRW_ALIGN_16);
- for (unsigned i = 0; i < sizeof(inst) / sizeof(inst[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(inst); i++) {
gen6_math(p, retype(g0, inst[i].dst_type),
BRW_MATH_FUNCTION_POW,
retype(g0, inst[i].src0_type),
@@ -1672,7 +1672,7 @@ TEST_P(validation_test, vector_immediate_destination_alignment)
{ BRW_REGISTER_TYPE_W, BRW_REGISTER_TYPE_UV, 1, BRW_EXECUTE_8, false },
};
- for (unsigned i = 0; i < sizeof(move) / sizeof(move[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(move); i++) {
/* UV type is Gen6+ */
if (devinfo.gen < 6 &&
move[i].src_type == BRW_REGISTER_TYPE_UV)
@@ -1714,7 +1714,7 @@ TEST_P(validation_test, vector_immediate_destination_stride)
{ BRW_REGISTER_TYPE_B, BRW_REGISTER_TYPE_UV, BRW_HORIZONTAL_STRIDE_2, true },
};
- for (unsigned i = 0; i < sizeof(move) / sizeof(move[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(move); i++) {
/* UV type is Gen6+ */
if (devinfo.gen < 6 &&
move[i].src_type == BRW_REGISTER_TYPE_UV)
@@ -1874,7 +1874,7 @@ TEST_P(validation_test, qword_low_power_align1_regioning_restrictions)
if (devinfo.gen >= 12)
return;
- for (unsigned i = 0; i < sizeof(inst) / sizeof(inst[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(inst); i++) {
if (!devinfo.has_64bit_types &&
(type_sz(inst[i].dst_type) == 8 || type_sz(inst[i].src_type) == 8))
continue;
@@ -1998,7 +1998,7 @@ TEST_P(validation_test, qword_low_power_no_indirect_addressing)
if (devinfo.gen < 8)
return;
- for (unsigned i = 0; i < sizeof(inst) / sizeof(inst[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(inst); i++) {
if (!devinfo.has_64bit_types &&
(type_sz(inst[i].dst_type) == 8 || type_sz(inst[i].src_type) == 8))
continue;
@@ -2138,7 +2138,7 @@ TEST_P(validation_test, qword_low_power_no_64bit_arf)
if (devinfo.gen < 8)
return;
- for (unsigned i = 0; i < sizeof(inst) / sizeof(inst[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(inst); i++) {
if (!devinfo.has_64bit_types &&
(type_sz(inst[i].dst_type) == 8 || type_sz(inst[i].src_type) == 8))
continue;
@@ -2241,7 +2241,7 @@ TEST_P(validation_test, align16_64_bit_integer)
brw_set_default_access_mode(p, BRW_ALIGN_16);
- for (unsigned i = 0; i < sizeof(inst) / sizeof(inst[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(inst); i++) {
if (inst[i].opcode == BRW_OPCODE_MOV) {
brw_MOV(p, retype(g0, inst[i].dst_type),
retype(g0, inst[i].src_type));
@@ -2347,7 +2347,7 @@ TEST_P(validation_test, qword_low_power_no_depctrl)
if (devinfo.gen >= 12)
return;
- for (unsigned i = 0; i < sizeof(inst) / sizeof(inst[0]); i++) {
+ for (unsigned i = 0; i < ARRAY_SIZE(inst); i++) {
if (!devinfo.has_64bit_types &&
(type_sz(inst[i].dst_type) == 8 || type_sz(inst[i].src_type) == 8))
continue;
More information about the mesa-commit
mailing list