OpenHarmony/third_party_mesa3d
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

util: Remove tmp argument from BITSET_FOREACH_SET macro

Browse Source 
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3499>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3499>
This commit is contained in:
Matt Turner
2020-01-21 15:07:24 -08:00
committed by Marge Bot
parent d3eb2a0951
commit 4413537c80
9 changed files with 21 additions and 34 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/compiler/nir/nir_phi_builder.c
View File

@@ -132,8 +132,7 @@ nir_phi_builder_add_value(struct nir_phi_builder *pb, unsigned num_components,
pb->iter_count++; pb->iter_count++;
BITSET_WORD tmp; BITSET_FOREACH_SET(i, defs, pb->num_blocks) {
BITSET_FOREACH_SET(i, tmp, defs, pb->num_blocks) {
if (pb->work[i] < pb->iter_count) if (pb->work[i] < pb->iter_count)
pb->W[w_end++] = pb->blocks[i]; pb->W[w_end++] = pb->blocks[i];
pb->work[i] = pb->iter_count; pb->work[i] = pb->iter_count;

5
src/gallium/drivers/etnaviv/etnaviv_compiler_nir_emit.h
View File

@@ -814,13 +814,12 @@ live_defs(nir_function_impl *impl, struct live_def *defs, unsigned *live_map)
/* apply live_in/live_out to ranges */ /* apply live_in/live_out to ranges */
nir_foreach_block(block, impl) { nir_foreach_block(block, impl) {
BITSET_WORD tmp;
int i; int i;
BITSET_FOREACH_SET(i, tmp, block->live_in, state.num_defs) BITSET_FOREACH_SET(i, block->live_in, state.num_defs)
range_include(&state.defs[i], block_live_index[block->index]); range_include(&state.defs[i], block_live_index[block->index]);
BITSET_FOREACH_SET(i, tmp, block->live_out, state.num_defs) BITSET_FOREACH_SET(i, block->live_out, state.num_defs)
range_include(&state.defs[i], block_live_index[block->index + 1]); range_include(&state.defs[i], block_live_index[block->index + 1]);
} }

13
src/gallium/drivers/lima/ir/gp/regalloc.c
View File

@@ -191,14 +191,13 @@ static void add_all_interferences(struct regalloc_ctx *ctx,
BITSET_WORD *live_regs) BITSET_WORD *live_regs)
{ {
int live_node; int live_node;
BITSET_WORD tmp; BITSET_FOREACH_SET(live_node, live_nodes, ctx->comp->cur_index) {
BITSET_FOREACH_SET(live_node, tmp, live_nodes, ctx->comp->cur_index) {
add_interference(ctx, i, add_interference(ctx, i,
live_node + ctx->comp->cur_reg); live_node + ctx->comp->cur_reg);
} }
int live_reg; int live_reg;
BITSET_FOREACH_SET(live_reg, tmp, ctx->live, ctx->comp->cur_index) { BITSET_FOREACH_SET(live_reg, ctx->live, ctx->comp->cur_index) {
add_interference(ctx, i, live_reg); add_interference(ctx, i, live_reg);
} }
@@ -211,11 +210,10 @@ static void print_liveness(struct regalloc_ctx *ctx,
return; return;
int live_idx; int live_idx;
BITSET_WORD tmp; BITSET_FOREACH_SET(live_idx, live_reg, ctx->comp->cur_reg) {
BITSET_FOREACH_SET(live_idx, tmp, live_reg, ctx->comp->cur_reg) {
printf("reg%d ", live_idx); printf("reg%d ", live_idx);
} }
BITSET_FOREACH_SET(live_idx, tmp, live_val, ctx->comp->cur_index) { BITSET_FOREACH_SET(live_idx, live_val, ctx->comp->cur_index) {
printf("%d ", live_idx); printf("%d ", live_idx);
} }
printf("\n"); printf("\n");
@@ -446,8 +444,7 @@ static void assign_regs(struct regalloc_ctx *ctx)
block->live_out_phys = 0; block->live_out_phys = 0;
int reg_idx; int reg_idx;
BITSET_WORD tmp; BITSET_FOREACH_SET(reg_idx, block->live_out, ctx->comp->cur_reg) {
BITSET_FOREACH_SET(reg_idx, tmp, block->live_out, ctx->comp->cur_reg) {
if (BITSET_TEST(block->def_out, reg_idx)) { if (BITSET_TEST(block->def_out, reg_idx)) {
block->live_out_phys |= (1ull << ctx->registers[reg_idx].assigned_color); block->live_out_phys |= (1ull << ctx->registers[reg_idx].assigned_color);
} }

9
src/intel/compiler/test_eu_validate.cpp
View File

@@ -353,8 +353,7 @@ TEST_P(validation_test, invalid_type_encoding)
* encodings are rejected by the validator. * encodings are rejected by the validator.
*/ */
int e; int e;
BITSET_WORD tmp; BITSET_FOREACH_SET(e, invalid_encodings, num_encodings) {
BITSET_FOREACH_SET(e, tmp, invalid_encodings, num_encodings) {
if (file == FIXED_GRF) { if (file == FIXED_GRF) {
brw_MOV(p, g0, g0); brw_MOV(p, g0, g0);
brw_inst_set_src0_vstride(&devinfo, last_inst, BRW_VERTICAL_STRIDE_4); brw_inst_set_src0_vstride(&devinfo, last_inst, BRW_VERTICAL_STRIDE_4);
@@ -432,8 +431,7 @@ TEST_P(validation_test, invalid_type_encoding_3src_a16)
* encodings are rejected by the validator. * encodings are rejected by the validator.
*/ */
int e; int e;
BITSET_WORD tmp; BITSET_FOREACH_SET(e, invalid_encodings, num_encodings) {
BITSET_FOREACH_SET(e, tmp, invalid_encodings, num_encodings) {
for (unsigned i = 0; i < 2; i++) { for (unsigned i = 0; i < 2; i++) {
if (i == 0) { if (i == 0) {
brw_MAD(p, g0, g0, g0, g0); brw_MAD(p, g0, g0, g0, g0);
@@ -525,8 +523,7 @@ TEST_P(validation_test, invalid_type_encoding_3src_a1)
* encodings are rejected by the validator. * encodings are rejected by the validator.
*/ */
int e; int e;
BITSET_WORD tmp; BITSET_FOREACH_SET(e, invalid_encodings, num_encodings) {
BITSET_FOREACH_SET(e, tmp, invalid_encodings, num_encodings) {
const unsigned hw_type = e & 0x7; const unsigned hw_type = e & 0x7;
const unsigned exec_type = e >> 3; const unsigned exec_type = e >> 3;

3
src/mesa/main/performance_monitor.c
View File

@@ -568,9 +568,8 @@ perf_monitor_result_size(const struct gl_context *ctx,
for (group = 0; group < ctx->PerfMonitor.NumGroups; group++) { for (group = 0; group < ctx->PerfMonitor.NumGroups; group++) {
const struct gl_perf_monitor_group *g = &ctx->PerfMonitor.Groups[group]; const struct gl_perf_monitor_group *g = &ctx->PerfMonitor.Groups[group];
BITSET_WORD tmp;
BITSET_FOREACH_SET(counter, tmp, m->ActiveCounters[group], g->NumCounters) { BITSET_FOREACH_SET(counter, m->ActiveCounters[group], g->NumCounters) {
const struct gl_perf_monitor_counter *c = &g->Counters[counter]; const struct gl_perf_monitor_counter *c = &g->Counters[counter];
size += sizeof(uint32_t); /* Group ID */ size += sizeof(uint32_t); /* Group ID */

3
src/mesa/state_tracker/st_cb_perfmon.c
View File

@@ -88,9 +88,8 @@ init_perf_monitor(struct gl_context *ctx, struct gl_perf_monitor_object *m)
for (gid = 0; gid < ctx->PerfMonitor.NumGroups; gid++) { for (gid = 0; gid < ctx->PerfMonitor.NumGroups; gid++) {
const struct gl_perf_monitor_group *g = &ctx->PerfMonitor.Groups[gid]; const struct gl_perf_monitor_group *g = &ctx->PerfMonitor.Groups[gid];
const struct st_perf_monitor_group *stg = &st->perfmon[gid]; const struct st_perf_monitor_group *stg = &st->perfmon[gid];
BITSET_WORD tmp;
BITSET_FOREACH_SET(cid, tmp, m->ActiveCounters[gid], g->NumCounters) { BITSET_FOREACH_SET(cid, m->ActiveCounters[gid], g->NumCounters) {
const struct st_perf_monitor_counter *stc = &stg->counters[cid]; const struct st_perf_monitor_counter *stc = &stg->counters[cid];
struct st_perf_counter_object *cntr = struct st_perf_counter_object *cntr =
&stm->active_counters[stm->num_active_counters]; &stm->active_counters[stm->num_active_counters];

8
src/panfrost/midgard/midgard_schedule.c
View File

@@ -300,8 +300,7 @@ mir_update_worklist(
* where possible. */ * where possible. */
unsigned i; unsigned i;
BITSET_WORD tmp; BITSET_FOREACH_SET(i, done->dependents, count) {
BITSET_FOREACH_SET(i, tmp, done->dependents, count) {
assert(instructions[i]->nr_dependencies); assert(instructions[i]->nr_dependencies);
if (!(--instructions[i]->nr_dependencies)) if (!(--instructions[i]->nr_dependencies))
@@ -482,7 +481,6 @@ mir_choose_instruction(
/* Iterate to find the best instruction satisfying the predicate */ /* Iterate to find the best instruction satisfying the predicate */
unsigned i; unsigned i;
BITSET_WORD tmp;
signed best_index = -1; signed best_index = -1;
bool best_conditional = false; bool best_conditional = false;
@@ -494,11 +492,11 @@ mir_choose_instruction(
unsigned max_active = 0; unsigned max_active = 0;
unsigned max_distance = 6; unsigned max_distance = 6;
BITSET_FOREACH_SET(i, tmp, worklist, count) { BITSET_FOREACH_SET(i, worklist, count) {
max_active = MAX2(max_active, i); max_active = MAX2(max_active, i);
} }
BITSET_FOREACH_SET(i, tmp, worklist, count) { BITSET_FOREACH_SET(i, worklist, count) {
if ((max_active - i) >= max_distance) if ((max_active - i) >= max_distance)
continue; continue;

6
src/util/bitset.h
View File

@@ -133,12 +133,12 @@ __bitset_next_set(unsigned i, BITSET_WORD *tmp,
* Iterates over each set bit in a set * Iterates over each set bit in a set
* *
* @param __i iteration variable, bit number * @param __i iteration variable, bit number
* @param __tmp an internally-used temporary bitset
* @param __set the bitset to iterate (will not be modified) * @param __set the bitset to iterate (will not be modified)
* @param __size number of bits in the set to consider * @param __size number of bits in the set to consider
*/ */
#define BITSET_FOREACH_SET(__i, __tmp, __set, __size) \ #define BITSET_FOREACH_SET(__i, __set, __size) \
for (__tmp = *(__set), __i = 0; \ for (BITSET_WORD __tmp = *(__set), *__foo = &__tmp; __foo != NULL; __foo = NULL) \
for (__i = 0; \
(__i = __bitset_next_set(__i, &__tmp, __set, __size)) < __size;) (__i = __bitset_next_set(__i, &__tmp, __set, __size)) < __size;)
#ifdef __cplusplus #ifdef __cplusplus

3
src/util/register_allocate.c
View File

@@ -342,10 +342,9 @@ void
ra_make_reg_conflicts_transitive(struct ra_regs *regs, unsigned int r) ra_make_reg_conflicts_transitive(struct ra_regs *regs, unsigned int r)
{ {
struct ra_reg *reg = &regs->regs[r]; struct ra_reg *reg = &regs->regs[r];
BITSET_WORD tmp;
int c; int c;
BITSET_FOREACH_SET(c, tmp, reg->conflicts, regs->count) { BITSET_FOREACH_SET(c, reg->conflicts, regs->count) {
struct ra_reg *other = &regs->regs[c]; struct ra_reg *other = &regs->regs[c];
unsigned i; unsigned i;
for (i = 0; i < BITSET_WORDS(regs->count); i++) for (i = 0; i < BITSET_WORDS(regs->count); i++)