util: rename list_empty() to list_is_empty()
authorTimothy Arceri <tarceri@itsqueeze.com>
Mon, 28 Oct 2019 10:27:52 +0000 (21:27 +1100)
committerTimothy Arceri <tarceri@itsqueeze.com>
Mon, 28 Oct 2019 11:24:38 +0000 (11:24 +0000)
This makes it clear that it's a boolean test and not an action
(eg. "empty the list").

Reviewed-by: Eric Engestrom <eric@engestrom.ch>
53 files changed:
src/amd/vulkan/radv_cmd_buffer.c
src/broadcom/compiler/nir_to_vir.c
src/broadcom/compiler/qpu_schedule.c
src/broadcom/compiler/vir.c
src/compiler/nir/nir.c
src/compiler/nir/nir_clone.c
src/compiler/nir/nir_deref.c
src/compiler/nir/nir_from_ssa.c
src/compiler/nir/nir_lower_io.c
src/compiler/nir/nir_lower_regs_to_ssa.c
src/compiler/nir/nir_lower_to_source_mods.c
src/compiler/nir/nir_lower_vec_to_movs.c
src/compiler/nir/nir_opt_combine_stores.c
src/compiler/nir/nir_opt_loop_unroll.c
src/compiler/nir/nir_opt_peephole_select.c
src/compiler/nir/nir_opt_vectorize.c
src/compiler/nir/nir_search_helpers.h
src/compiler/nir/nir_serialize.c
src/compiler/nir/nir_validate.c
src/compiler/nir/nir_worklist.h
src/compiler/spirv/vtn_cfg.c
src/freedreno/ir3/ir3_compiler_nir.c
src/freedreno/ir3/ir3_legalize.c
src/freedreno/ir3/ir3_sched.c
src/freedreno/vulkan/tu_cmd_buffer.c
src/gallium/auxiliary/driver_ddebug/dd_context.c
src/gallium/auxiliary/driver_ddebug/dd_draw.c
src/gallium/drivers/etnaviv/etnaviv_query_hw.c
src/gallium/drivers/freedreno/freedreno_query_acc.c
src/gallium/drivers/freedreno/freedreno_query_hw.c
src/gallium/drivers/lima/ir/gp/gpir.h
src/gallium/drivers/lima/ir/gp/reduce_scheduler.c
src/gallium/drivers/lima/ir/gp/scheduler.c
src/gallium/drivers/lima/ir/pp/codegen.c
src/gallium/drivers/lima/ir/pp/ppir.h
src/gallium/drivers/lima/ir/pp/regalloc.c
src/gallium/drivers/lima/ir/pp/scheduler.c
src/gallium/drivers/llvmpipe/lp_cs_tpool.c
src/gallium/drivers/v3d/v3d_bufmgr.c
src/gallium/drivers/vc4/vc4_bufmgr.c
src/gallium/drivers/vc4/vc4_program.c
src/gallium/drivers/vc4/vc4_qir.c
src/gallium/drivers/vc4/vc4_qir_schedule.c
src/gallium/drivers/vc4/vc4_qpu_schedule.c
src/gallium/winsys/amdgpu/drm/amdgpu_bo.c
src/intel/common/gen_aux_map.c
src/intel/compiler/brw_nir_opt_peephole_ffma.c
src/intel/vulkan/anv_batch_chain.c
src/intel/vulkan/anv_device.c
src/mesa/drivers/dri/i965/brw_bufmgr.c
src/util/list.h
src/vulkan/overlay-layer/overlay.cpp
src/vulkan/util/vk_debug_report.c

index d22bc84c897636c69ac5d5164a2b66d35703b7f3..a172767e3dde1dae4c273335a96c75b240e42576 100644 (file)
@@ -3180,7 +3180,7 @@ VkResult radv_AllocateCommandBuffers(
 
        for (i = 0; i < pAllocateInfo->commandBufferCount; i++) {
 
-               if (!list_empty(&pool->free_cmd_buffers)) {
+               if (!list_is_empty(&pool->free_cmd_buffers)) {
                        struct radv_cmd_buffer *cmd_buffer = list_first_entry(&pool->free_cmd_buffers, struct radv_cmd_buffer, pool_link);
 
                        list_del(&cmd_buffer->pool_link);
index a457b7ad85f90b4a9434da364fc087e513ebff50..86b87837f8ed4423b297409052aeb09f0631a410 100644 (file)
@@ -429,7 +429,7 @@ ntq_store_dest(struct v3d_compile *c, nir_dest *dest, int chan,
                struct qreg result)
 {
         struct qinst *last_inst = NULL;
-        if (!list_empty(&c->cur_block->instructions))
+        if (!list_is_empty(&c->cur_block->instructions))
                 last_inst = (struct qinst *)c->cur_block->instructions.prev;
 
         assert((result.file == QFILE_TEMP &&
index c15218e267e213a6cb2d195b0b0b0c23d8eed51b..9f11fe27eb869dc5631cb7adc5434cd5bf93a632 100644 (file)
@@ -1299,7 +1299,7 @@ schedule_instructions(struct v3d_compile *c,
         const struct v3d_device_info *devinfo = c->devinfo;
         uint32_t time = 0;
 
-        while (!list_empty(&scoreboard->dag->heads)) {
+        while (!list_is_empty(&scoreboard->dag->heads)) {
                 struct schedule_node *chosen =
                         choose_instruction_to_schedule(devinfo,
                                                        scoreboard,
@@ -1439,7 +1439,7 @@ qpu_schedule_instructions_block(struct v3d_compile *c,
         list_inithead(&setup_list);
 
         /* Wrap each instruction in a scheduler structure. */
-        while (!list_empty(&block->instructions)) {
+        while (!list_is_empty(&block->instructions)) {
                 struct qinst *qinst = (struct qinst *)block->instructions.next;
                 struct schedule_node *n =
                         rzalloc(mem_ctx, struct schedule_node);
index 99168847adf2b97847f717d0c4b9dd3fc9900d77..da4234042eafa7439fedcbdbffb33806db83f5a2 100644 (file)
@@ -1034,7 +1034,7 @@ vir_compile_destroy(struct v3d_compile *c)
         c->cursor.link = NULL;
 
         vir_for_each_block(block, c) {
-                while (!list_empty(&block->instructions)) {
+                while (!list_is_empty(&block->instructions)) {
                         struct qinst *qinst =
                                 list_first_entry(&block->instructions,
                                                  struct qinst, link);
index 1cec8171a7fb0502e8efb661bed6113b6fdead27..1d7a7479d6c7f1d73fc977fc5cc3ac0ce11442dd 100644 (file)
@@ -1416,7 +1416,7 @@ nir_instr_rewrite_dest(nir_instr *instr, nir_dest *dest, nir_dest new_dest)
 {
    if (dest->is_ssa) {
       /* We can only overwrite an SSA destination if it has no uses. */
-      assert(list_empty(&dest->ssa.uses) && list_empty(&dest->ssa.if_uses));
+      assert(list_is_empty(&dest->ssa.uses) && list_is_empty(&dest->ssa.if_uses));
    } else {
       list_del(&dest->reg.def_link);
       if (dest->reg.indirect)
@@ -1547,7 +1547,7 @@ nir_ssa_def_components_read(const nir_ssa_def *def)
       }
    }
 
-   if (!list_empty(&def->if_uses))
+   if (!list_is_empty(&def->if_uses))
       read_mask |= 1;
 
    return read_mask;
@@ -1888,7 +1888,7 @@ nir_function_impl_lower_instructions(nir_function_impl *impl,
          list_for_each_entry_safe(nir_src, use_src, &old_if_uses, use_link)
             nir_if_rewrite_condition(use_src->parent_if, new_src);
 
-         if (list_empty(&old_def->uses) && list_empty(&old_def->if_uses)) {
+         if (list_is_empty(&old_def->uses) && list_is_empty(&old_def->if_uses)) {
             iter = nir_instr_remove(instr);
          } else {
             iter = nir_after_instr(instr);
index 71303086db72a1969a0ce83885fb6710fe5d1175..682a1a274a720c2f5fdba1f971d85797b6d0b507 100644 (file)
@@ -629,7 +629,7 @@ fixup_phi_srcs(clone_state *state)
          list_addtail(&src->src.use_link, &src->src.reg.reg->uses);
       }
    }
-   assert(list_empty(&state->phi_srcs));
+   assert(list_is_empty(&state->phi_srcs));
 }
 
 void
@@ -669,7 +669,7 @@ clone_function_impl(clone_state *state, const nir_function_impl *fi)
    clone_reg_list(state, &nfi->registers, &fi->registers);
    nfi->reg_alloc = fi->reg_alloc;
 
-   assert(list_empty(&state->phi_srcs));
+   assert(list_is_empty(&state->phi_srcs));
 
    clone_cf_list(state, &nfi->body, &fi->body);
 
index 7ec460126e38693bc07830139a6711d49fa90b37..b81edde8dac1ffdf738c523cf81a073b60ac788e 100644 (file)
@@ -92,7 +92,7 @@ nir_deref_instr_remove_if_unused(nir_deref_instr *instr)
    for (nir_deref_instr *d = instr; d; d = nir_deref_instr_parent(d)) {
       /* If anyone is using this deref, leave it alone */
       assert(d->dest.is_ssa);
-      if (!list_empty(&d->dest.ssa.uses))
+      if (!list_is_empty(&d->dest.ssa.uses))
          break;
 
       nir_instr_remove(&d->instr);
@@ -855,7 +855,7 @@ opt_deref_cast(nir_builder *b, nir_deref_instr *cast)
    }
 
    /* If uses would be a bit crazy */
-   assert(list_empty(&cast->dest.ssa.if_uses));
+   assert(list_is_empty(&cast->dest.ssa.if_uses));
 
    nir_deref_instr_remove_if_unused(cast);
    return progress;
index b5d79b374598cf40a95cab165d0cc8093afdf6a4..acb758584a6dbdf62ab5d2b1e6d28c6becaa44de 100644 (file)
@@ -495,7 +495,7 @@ rewrite_ssa_def(nir_ssa_def *def, void *void_state)
    }
 
    nir_ssa_def_rewrite_uses(def, nir_src_for_reg(reg));
-   assert(list_empty(&def->uses) && list_empty(&def->if_uses));
+   assert(list_is_empty(&def->uses) && list_is_empty(&def->if_uses));
 
    if (def->parent_instr->type == nir_instr_type_ssa_undef) {
       /* If it's an ssa_undef instruction, remove it since we know we just got
@@ -961,7 +961,7 @@ ssa_def_is_local_to_block(nir_ssa_def *def, UNUSED void *state)
       }
    }
 
-   if (!list_empty(&def->if_uses))
+   if (!list_is_empty(&def->if_uses))
       return false;
 
    return true;
index 6f257d94cc459cbea574d38d7f84de6c28e36890..745ef92ddf3316fb23e154154b2cac14a404fb62 100644 (file)
@@ -1191,8 +1191,8 @@ lower_explicit_io_deref(nir_builder *b, nir_deref_instr *deref,
     * one deref which could break our list walking since we walk the list
     * backwards.
     */
-   assert(list_empty(&deref->dest.ssa.if_uses));
-   if (list_empty(&deref->dest.ssa.uses)) {
+   assert(list_is_empty(&deref->dest.ssa.if_uses));
+   if (list_is_empty(&deref->dest.ssa.uses)) {
       nir_instr_remove(&deref->instr);
       return;
    }
index b755d1df99e0ec79b18e787b4ca40d706f1fa155..2e83c80af18a6d867e8ef9feafafb288f06cc5e5 100644 (file)
@@ -300,9 +300,9 @@ nir_lower_regs_to_ssa_impl(nir_function_impl *impl)
 
    nir_foreach_register_safe(reg, &impl->registers) {
       if (state.values[reg->index]) {
-         assert(list_empty(&reg->uses));
-         assert(list_empty(&reg->if_uses));
-         assert(list_empty(&reg->defs));
+         assert(list_is_empty(&reg->uses));
+         assert(list_is_empty(&reg->if_uses));
+         assert(list_is_empty(&reg->defs));
          exec_node_remove(&reg->node);
       }
    }
index f3d004ccdd2c2ff32f268fa107ab1273c083bc7c..ab9581dcc3d98f46b4d94cb6a1a8e3481728ab80 100644 (file)
@@ -122,8 +122,8 @@ nir_lower_to_source_mods_block(nir_block *block,
             alu->src[i].swizzle[j] = parent->src[0].swizzle[alu->src[i].swizzle[j]];
          }
 
-         if (list_empty(&parent->dest.dest.ssa.uses) &&
-             list_empty(&parent->dest.dest.ssa.if_uses))
+         if (list_is_empty(&parent->dest.dest.ssa.uses) &&
+             list_is_empty(&parent->dest.dest.ssa.if_uses))
             nir_instr_remove(&parent->instr);
 
          progress = true;
@@ -144,7 +144,7 @@ nir_lower_to_source_mods_block(nir_block *block,
       if (!(options & nir_lower_float_source_mods))
          continue;
 
-      if (!list_empty(&alu->dest.dest.ssa.if_uses))
+      if (!list_is_empty(&alu->dest.dest.ssa.if_uses))
          continue;
 
       bool all_children_are_sat = true;
index 8c286117d55e8302675f0b3da13dfa689115f052..eec994f15e53881580659ecfcbce91713b0f5db2 100644 (file)
@@ -140,7 +140,7 @@ try_coalesce(nir_alu_instr *vec, unsigned start_idx)
          return 0;
    }
 
-   if (!list_empty(&vec->src[start_idx].src.ssa->if_uses))
+   if (!list_is_empty(&vec->src[start_idx].src.ssa->if_uses))
       return 0;
 
    if (vec->src[start_idx].src.ssa->parent_instr->type != nir_instr_type_alu)
index b3e5cb3947fc8810912b173bc10f8158455fa6e7..c59f3ed99365dbab48e1d7717efb21f6d61db34c 100644 (file)
@@ -84,7 +84,7 @@ static struct combined_store *
 alloc_combined_store(struct combine_stores_state *state)
 {
    struct combined_store *result;
-   if (list_empty(&state->freelist)) {
+   if (list_is_empty(&state->freelist)) {
       result = linear_zalloc_child(state->lin_ctx, sizeof(*result));
    } else {
       result = list_first_entry(&state->freelist,
index 0681518d9606bc9544f16561b394ac97a8634c13..e1c37d24eb71ea1d6dd48c32d9827032d3703ccd 100644 (file)
@@ -514,7 +514,7 @@ complex_unroll_single_terminator(nir_loop *loop)
 static bool
 wrapper_unroll(nir_loop *loop)
 {
-   if (!list_empty(&loop->info->loop_terminator_list)) {
+   if (!list_is_empty(&loop->info->loop_terminator_list)) {
 
       /* Unrolling a loop with a large number of exits can result in a
        * large inrease in register pressure. For now we just skip
index 09ae3d5632f9d281e2d76a596a9652d3e2caa7f2..14f36e799de8ba00a002548976333c85afb295da 100644 (file)
@@ -156,7 +156,7 @@ block_check_for_allowed_instrs(nir_block *block, unsigned *count,
                return false;
 
             /* It cannot have any if-uses */
-            if (!list_empty(&mov->dest.dest.ssa.if_uses))
+            if (!list_is_empty(&mov->dest.dest.ssa.if_uses))
                return false;
 
             /* The only uses of this definition must be phis in the successor */
index a3681610a59dfbe1b6b7e7108d60c10a40d905bb..20e03db0d38f1d87166b6736040a6b90c373674d 100644 (file)
@@ -248,8 +248,8 @@ instr_try_combine(nir_instr *instr1, nir_instr *instr2)
       nir_if_rewrite_condition(src->parent_if, nir_src_for_ssa(new_alu1));
    }
 
-   assert(list_empty(&alu1->dest.dest.ssa.uses));
-   assert(list_empty(&alu1->dest.dest.ssa.if_uses));
+   assert(list_is_empty(&alu1->dest.dest.ssa.uses));
+   assert(list_is_empty(&alu1->dest.dest.ssa.if_uses));
 
    nir_foreach_use_safe(src, &alu2->dest.dest.ssa) {
       if (src->parent_instr->type == nir_instr_type_alu) {
@@ -285,8 +285,8 @@ instr_try_combine(nir_instr *instr1, nir_instr *instr2)
       nir_if_rewrite_condition(src->parent_if, nir_src_for_ssa(new_alu2));
    }
 
-   assert(list_empty(&alu2->dest.dest.ssa.uses));
-   assert(list_empty(&alu2->dest.dest.ssa.if_uses));
+   assert(list_is_empty(&alu2->dest.dest.ssa.uses));
+   assert(list_is_empty(&alu2->dest.dest.ssa.if_uses));
 
    nir_instr_remove(instr1);
    nir_instr_remove(instr2);
index 4849aef052d069a54ebc86ea4819bfaebaf6b8c6..0c2f631ac39029026a45127801bd907af610cd8b 100644 (file)
@@ -226,8 +226,8 @@ is_not_const_and_not_fsign(struct hash_table *ht, nir_alu_instr *instr, unsigned
 static inline bool
 is_used_once(nir_alu_instr *instr)
 {
-   bool zero_if_use = list_empty(&instr->dest.dest.ssa.if_uses);
-   bool zero_use = list_empty(&instr->dest.dest.ssa.uses);
+   bool zero_if_use = list_is_empty(&instr->dest.dest.ssa.if_uses);
+   bool zero_use = list_is_empty(&instr->dest.dest.ssa.uses);
 
    if (zero_if_use && zero_use)
       return false;
@@ -248,13 +248,13 @@ is_used_once(nir_alu_instr *instr)
 static inline bool
 is_used_by_if(nir_alu_instr *instr)
 {
-   return !list_empty(&instr->dest.dest.ssa.if_uses);
+   return !list_is_empty(&instr->dest.dest.ssa.if_uses);
 }
 
 static inline bool
 is_not_used_by_if(nir_alu_instr *instr)
 {
-   return list_empty(&instr->dest.dest.ssa.if_uses);
+   return list_is_empty(&instr->dest.dest.ssa.if_uses);
 }
 
 static inline bool
index b7b44f6af888e608823af40759eca72bece28d63..558bd6c97a3fdf42e7136df8a850da4e2d648d4a 100644 (file)
@@ -762,7 +762,7 @@ read_fixup_phis(read_ctx *ctx)
 
       list_addtail(&src->src.use_link, &src->src.ssa->uses);
    }
-   assert(list_empty(&ctx->phi_srcs));
+   assert(list_is_empty(&ctx->phi_srcs));
 }
 
 static void
index 204cfdbbe3cddd6c524272237e557b3a33d5e8e5..22effa0c5e6aef30dffb94878774a1140d1b0cb4 100644 (file)
@@ -498,7 +498,7 @@ validate_deref_instr(nir_deref_instr *instr, validate_state *state)
     * conditions expect well-formed Booleans.  If you want to compare with
     * NULL, an explicit comparison operation should be used.
     */
-   validate_assert(state, list_empty(&instr->dest.ssa.if_uses));
+   validate_assert(state, list_is_empty(&instr->dest.ssa.if_uses));
 
    /* Only certain modes can be used as sources for phi instructions. */
    nir_foreach_use(use, &instr->dest.ssa) {
index 05aa757eb7903546cfcee97e6cf5c08662637596..36014cc7752470bf0c58acb49482f6c92be7afaa 100644 (file)
@@ -124,7 +124,7 @@ nir_instr_worklist_length(nir_instr_worklist *wl)
 }
 
 static inline bool
-nir_instr_worklist_empty(nir_instr_worklist *wl)
+nir_instr_worklist_is_empty(nir_instr_worklist *wl)
 {
    return nir_instr_worklist_length(wl) == 0;
 }
index 843966f86b5fc80627225de41bbff9be03271939..a645149008125af87499d6d9f7d6831ca0d0a6f6 100644 (file)
@@ -1002,7 +1002,7 @@ vtn_emit_cf_list(struct vtn_builder *b, struct list_head *cf_list,
 
          vtn_emit_cf_list(b, &vtn_loop->body, NULL, NULL, handler);
 
-         if (!list_empty(&vtn_loop->cont_body)) {
+         if (!list_is_empty(&vtn_loop->cont_body)) {
             /* If we have a non-trivial continue body then we need to put
              * it at the beginning of the loop with a flag to ensure that
              * it doesn't get executed in the first iteration.
index 1228fea5534122ec909bdbaf46ffe02ee9baa452..e0bf9280975c96d162dea6ebb6cb323eab295bc8 100644 (file)
@@ -2506,7 +2506,7 @@ emit_function(struct ir3_context *ctx, nir_function_impl *impl)
        /* at this point, we should have a single empty block,
         * into which we emit the 'end' instruction.
         */
-       compile_assert(ctx, list_empty(&ctx->block->instr_list));
+       compile_assert(ctx, list_is_empty(&ctx->block->instr_list));
 
        /* If stream-out (aka transform-feedback) enabled, emit the
         * stream-out instructions, followed by a new empty block (into
index 76362c4468cd3924489d2a2b20722ed85b7e1cbf..53fd4f919ee1a47d30eaed6f4735c2349fd12d19 100644 (file)
@@ -200,10 +200,10 @@ legalize_block(struct ir3_legalize_ctx *ctx, struct ir3_block *block)
                }
 
                /* need to be able to set (ss) on first instruction: */
-               if (list_empty(&block->instr_list) && (opc_cat(n->opc) >= 5))
+               if (list_is_empty(&block->instr_list) && (opc_cat(n->opc) >= 5))
                        ir3_NOP(block);
 
-               if (is_nop(n) && !list_empty(&block->instr_list)) {
+               if (is_nop(n) && !list_is_empty(&block->instr_list)) {
                        struct ir3_instruction *last = list_last_entry(&block->instr_list,
                                        struct ir3_instruction, node);
                        if (is_nop(last) && (last->repeat < 5)) {
@@ -410,7 +410,7 @@ resolve_dest_block(struct ir3_block *block)
         *   (2) (block-is-empty || only-instr-is-jump)
         */
        if (block->successors[1] == NULL) {
-               if (list_empty(&block->instr_list)) {
+               if (list_is_empty(&block->instr_list)) {
                        return block->successors[0];
                } else if (list_length(&block->instr_list) == 1) {
                        struct ir3_instruction *instr = list_first_entry(
index 635c693095afa7e42eac1a25d06d4b182085f5a3..34c648a6748b92640ba49a05b7a657df2cf3eecb 100644 (file)
@@ -796,7 +796,7 @@ sched_block(struct ir3_sched_ctx *ctx, struct ir3_block *block)
                }
        }
 
-       while (!list_empty(&ctx->depth_list)) {
+       while (!list_is_empty(&ctx->depth_list)) {
                struct ir3_sched_notes notes = {0};
                struct ir3_instruction *instr;
 
index fdc3a8d1436f7d5efb15e64fcb3208597a6cbc7d..200ba002939bdcdf787e0dec60e17d00435226fa 100644 (file)
@@ -1576,7 +1576,7 @@ tu_AllocateCommandBuffers(VkDevice _device,
 
    for (i = 0; i < pAllocateInfo->commandBufferCount; i++) {
 
-      if (!list_empty(&pool->free_cmd_buffers)) {
+      if (!list_is_empty(&pool->free_cmd_buffers)) {
          struct tu_cmd_buffer *cmd_buffer = list_first_entry(
             &pool->free_cmd_buffers, struct tu_cmd_buffer, pool_link);
 
index 311eca7c1ae05a17f558070996ea699d682c4354..40da6cf41c7073bb59f5106d3779d6ee7280119d 100644 (file)
@@ -623,7 +623,7 @@ dd_context_destroy(struct pipe_context *_pipe)
    mtx_destroy(&dctx->mutex);
    cnd_destroy(&dctx->cond);
 
-   assert(list_empty(&dctx->records));
+   assert(list_is_empty(&dctx->records));
 
    if (pipe->set_log_context) {
       pipe->set_log_context(pipe, NULL);
index 1d8e57767edf0db9a5572e91c0b9731b8542c631..b2ed5d5f13b40656efa69bbc0a6987556563bf3f 100644 (file)
@@ -1103,7 +1103,7 @@ dd_thread_main(void *input)
       if (dctx->api_stalled)
          cnd_signal(&dctx->cond);
 
-      if (list_empty(&records)) {
+      if (list_is_empty(&records)) {
          if (dctx->kill_thread)
             break;
 
@@ -1184,7 +1184,7 @@ dd_add_record(struct dd_context *dctx, struct dd_draw_record *record)
       dctx->api_stalled = false;
    }
 
-   if (list_empty(&dctx->records))
+   if (list_is_empty(&dctx->records))
       cnd_signal(&dctx->cond);
 
    list_addtail(&record->list, &dctx->records);
index e60d90dbb05d7a150f167c9d7ef5df4fd8d2a965..9324071f41cf917da99a1d9e25e8ee0775337dad 100644 (file)
@@ -150,7 +150,7 @@ etna_hw_begin_query(struct etna_context *ctx, struct etna_query *q)
    p->start(hq, ctx);
 
    /* add to active list */
-   assert(list_empty(&hq->node));
+   assert(list_is_empty(&hq->node));
    list_addtail(&hq->node, &ctx->active_hw_queries);
 
    return true;
index b0be16c233e3130c795ccf85a7f5f29a9e6d8db6..dd8665e0ca6513a09cc693843a00d524b7b131c3 100644 (file)
@@ -91,7 +91,7 @@ fd_acc_begin_query(struct fd_context *ctx, struct fd_query *q)
                p->resume(aq, batch);
 
        /* add to active list: */
-       assert(list_empty(&aq->node));
+       assert(list_is_empty(&aq->node));
        list_addtail(&aq->node, &ctx->acc_active_queries);
 
        return true;
index b37126dc0da90eb2c9566813d96259d82098eaee..6699b98062cc2e9eb20760db3d9907af36ea9895 100644 (file)
@@ -147,7 +147,7 @@ fd_hw_begin_query(struct fd_context *ctx, struct fd_query *q)
                resume_query(batch, hq, batch->draw);
 
        /* add to active list: */
-       assert(list_empty(&hq->list));
+       assert(list_is_empty(&hq->list));
        list_addtail(&hq->list, &ctx->hw_active_queries);
 
        return true;
index 4d11fb1eaa70912c1507d07bd991c2c18e71e796..26d0fd9ffa78d2e4751c8ea7e735fb3d5c847e1e 100644 (file)
@@ -454,12 +454,12 @@ void gpir_node_print_prog_seq(gpir_compiler *comp);
 
 static inline bool gpir_node_is_root(gpir_node *node)
 {
-   return list_empty(&node->succ_list);
+   return list_is_empty(&node->succ_list);
 }
 
 static inline bool gpir_node_is_leaf(gpir_node *node)
 {
-   return list_empty(&node->pred_list);
+   return list_is_empty(&node->pred_list);
 }
 
 #define gpir_node_to_alu(node) ((gpir_alu_node *)(node))
index d51fc355f2b9c6fb99bbe2bd6f28d52d47f026c4..8b4f559330d87d7d49d6c673fe6b8ebc5a97d212 100644 (file)
@@ -128,7 +128,7 @@ static void schedule_insert_ready_list(struct list_head *ready_list,
 
 static void schedule_ready_list(gpir_block *block, struct list_head *ready_list)
 {
-   if (list_empty(ready_list))
+   if (list_is_empty(ready_list))
       return;
 
    gpir_node *node = list_first_entry(ready_list, gpir_node, list);
index bf8bd63e57cbc12fb2d6745b9ca3cc0f13f91235..471696d31b027222d80141a30ddb155671ba007b 100644 (file)
@@ -1604,7 +1604,7 @@ static bool schedule_block(gpir_block *block)
    }
 
    list_inithead(&block->node_list);
-   while (!list_empty(&ctx.ready_list)) {
+   while (!list_is_empty(&ctx.ready_list)) {
       if (!schedule_one_instr(&ctx))
          return false;
    }
index 47e25d3f886f4253a319f4386ff0ee6b3375ee8b..b6ce500750f7acea04d6e271e2201730052836ed 100644 (file)
@@ -596,13 +596,13 @@ static void ppir_codegen_encode_branch(ppir_node *node, void *code)
    }
 
    target = branch->target;
-   while (list_empty(&target->instr_list)) {
+   while (list_is_empty(&target->instr_list)) {
       if (!target->list.next)
          break;
       target = LIST_ENTRY(ppir_block, target->list.next, list);
    }
 
-   assert(!list_empty(&target->instr_list));
+   assert(!list_is_empty(&target->instr_list));
 
    target_instr = list_first_entry(&target->instr_list, ppir_instr, list);
    b->branch.target = target_instr->offset - node->instr->offset;
index 96ae1e0df2cc30c352231513141db73f535aebc8..4dc48350fa412c88c1f96759afdbe4e9738353a4 100644 (file)
@@ -398,12 +398,12 @@ ppir_node *ppir_node_insert_mov(ppir_node *node);
 
 static inline bool ppir_node_is_root(ppir_node *node)
 {
-   return list_empty(&node->succ_list);
+   return list_is_empty(&node->succ_list);
 }
 
 static inline bool ppir_node_is_leaf(ppir_node *node)
 {
-   return list_empty(&node->pred_list);
+   return list_is_empty(&node->pred_list);
 }
 
 static inline bool ppir_node_has_single_succ(ppir_node *node)
@@ -652,12 +652,12 @@ void ppir_instr_insert_mul_node(ppir_node *add, ppir_node *mul);
 
 static inline bool ppir_instr_is_root(ppir_instr *instr)
 {
-   return list_empty(&instr->succ_list);
+   return list_is_empty(&instr->succ_list);
 }
 
 static inline bool ppir_instr_is_leaf(ppir_instr *instr)
 {
-   return list_empty(&instr->pred_list);
+   return list_is_empty(&instr->pred_list);
 }
 
 bool ppir_lower_prog(ppir_compiler *comp);
index 0daeb5c526d87ae838d8136afb76c97cbee144af..d0fe5f9d68e2ce59c48a92021e10e7e3c802713e 100644 (file)
@@ -688,7 +688,7 @@ bool ppir_regalloc_prog(ppir_compiler *comp)
    ppir_regalloc_update_reglist_ssa(comp);
 
    /* No registers? Probably shader consists of discard instruction */
-   if (list_empty(&comp->reg_list))
+   if (list_is_empty(&comp->reg_list))
       return true;
 
    /* this will most likely succeed in the first
index 721504a810e264e810f5170f0860523d0cd88902..5e7a17c9bb90e77bc50f64d45a1aab51ab76691d 100644 (file)
@@ -118,7 +118,7 @@ static void ppir_insert_ready_list(struct list_head *ready_list,
 static void ppir_schedule_ready_list(ppir_block *block,
                                      struct list_head *ready_list)
 {
-   if (list_empty(ready_list))
+   if (list_is_empty(ready_list))
       return;
 
    ppir_instr *instr = list_first_entry(ready_list, ppir_instr, list);
index 26e82ab1285c52f71c7bfe187ea91b53c81fcc19..4a23b6a2a950e93521af4579ecba598dc9031b7c 100644 (file)
@@ -44,7 +44,7 @@ lp_cs_tpool_worker(void *data)
    while (!pool->shutdown) {
       struct lp_cs_tpool_task *task;
 
-      while (list_empty(&pool->workqueue) && !pool->shutdown)
+      while (list_is_empty(&pool->workqueue) && !pool->shutdown)
          cnd_wait(&pool->new_work, &pool->m);
 
       if (pool->shutdown)
index f0df1b98373789ac276f2a8109bbcc28af6cfdb9..b3e4d053cc030e662d07a6a59fdba583ee63b254 100644 (file)
@@ -65,7 +65,7 @@ v3d_bo_dump_stats(struct v3d_screen *screen)
         fprintf(stderr, "  BOs cached:      %d\n", cache_count);
         fprintf(stderr, "  BOs cached size: %dkb\n", cache_size / 1024);
 
-        if (!list_empty(&cache->time_list)) {
+        if (!list_is_empty(&cache->time_list)) {
                 struct v3d_bo *first = list_first_entry(&cache->time_list,
                                                         struct v3d_bo,
                                                         time_list);
@@ -103,7 +103,7 @@ v3d_bo_from_cache(struct v3d_screen *screen, uint32_t size, const char *name)
 
         struct v3d_bo *bo = NULL;
         mtx_lock(&cache->lock);
-        if (!list_empty(&cache->size_list[page_index])) {
+        if (!list_is_empty(&cache->size_list[page_index])) {
                 bo = list_first_entry(&cache->size_list[page_index],
                                       struct v3d_bo, size_list);
 
@@ -170,7 +170,7 @@ v3d_bo_alloc(struct v3d_screen *screen, uint32_t size, const char *name)
         bo->offset = create.offset;
 
         if (ret != 0) {
-                if (!list_empty(&screen->bo_cache.time_list) &&
+                if (!list_is_empty(&screen->bo_cache.time_list) &&
                     !cleared_and_retried) {
                         cleared_and_retried = true;
                         v3d_bo_cache_free_all(&screen->bo_cache);
@@ -300,7 +300,7 @@ v3d_bo_last_unreference_locked_timed(struct v3d_bo *bo, time_t time)
                  */
                 for (int i = 0; i < cache->size_list_size; i++) {
                         struct list_head *old_head = &cache->size_list[i];
-                        if (list_empty(old_head))
+                        if (list_is_empty(old_head))
                                 list_inithead(&new_list[i]);
                         else {
                                 new_list[i].next = old_head->next;
index cc32862d3fe58b4f3f5302284cb35596b3387317..5ec360934c071692952faa5d5032b70b96d769bc 100644 (file)
@@ -92,7 +92,7 @@ vc4_bo_dump_stats(struct vc4_screen *screen)
         fprintf(stderr, "  BOs cached:      %d\n", cache->bo_count);
         fprintf(stderr, "  BOs cached size: %dkb\n", cache->bo_size / 1024);
 
-        if (!list_empty(&cache->time_list)) {
+        if (!list_is_empty(&cache->time_list)) {
                 struct vc4_bo *first = LIST_ENTRY(struct vc4_bo,
                                                   cache->time_list.next,
                                                   time_list);
@@ -263,7 +263,7 @@ vc4_bo_alloc(struct vc4_screen *screen, uint32_t size, const char *name)
         bo->handle = create.handle;
 
         if (ret != 0) {
-                if (!list_empty(&screen->bo_cache.time_list) &&
+                if (!list_is_empty(&screen->bo_cache.time_list) &&
                     !cleared_and_retried) {
                         cleared_and_retried = true;
                         vc4_bo_cache_free_all(&screen->bo_cache);
index 6ea1667ca281e95f201c61fa103f52ffc73235ba..3fabf4c7dc9743ead05e7462c86cc8ec310fe9be 100644 (file)
@@ -185,7 +185,7 @@ ntq_store_dest(struct vc4_compile *c, nir_dest *dest, int chan,
                struct qreg result)
 {
         struct qinst *last_inst = NULL;
-        if (!list_empty(&c->cur_block->instructions))
+        if (!list_is_empty(&c->cur_block->instructions))
                 last_inst = (struct qinst *)c->cur_block->instructions.prev;
 
         assert(result.file == QFILE_UNIF ||
@@ -832,7 +832,7 @@ ntq_src_is_only_ssa_def_user(nir_src *src)
         if (!src->is_ssa)
                 return false;
 
-        if (!list_empty(&src->ssa->if_uses))
+        if (!list_is_empty(&src->ssa->if_uses))
                 return false;
 
         return (src->ssa->uses.next == &src->use_link &&
index 71f06aebfa04503eb88bae47c42283badda87455..0d104c0615057e8989649755cc8c646812f86c6d 100644 (file)
@@ -738,7 +738,7 @@ void
 qir_compile_destroy(struct vc4_compile *c)
 {
         qir_for_each_block(block, c) {
-                while (!list_empty(&block->instructions)) {
+                while (!list_is_empty(&block->instructions)) {
                         struct qinst *qinst =
                                 list_first_entry(&block->instructions,
                                                  struct qinst, link);
@@ -798,7 +798,7 @@ qir_SF(struct vc4_compile *c, struct qreg src)
 {
         struct qinst *last_inst = NULL;
 
-        if (!list_empty(&c->cur_block->instructions))
+        if (!list_is_empty(&c->cur_block->instructions))
                 last_inst = (struct qinst *)c->cur_block->instructions.prev;
 
         /* We don't have any way to guess which kind of MOV is implied. */
index cbd993ceb32b32003c37b2acb510063ea9c4efcd..fdd922fde4d5fca5cf4feef8b7d8618ace0123ee 100644 (file)
@@ -622,7 +622,7 @@ schedule_instructions(struct vc4_compile *c,
         }
 
         state->time = 0;
-        while (!list_empty(&state->dag->heads)) {
+        while (!list_is_empty(&state->dag->heads)) {
                 struct schedule_node *chosen = choose_instruction(state);
                 struct qinst *inst = chosen->inst;
 
index c150e951664627c26c77ad8b209e54e0b07309ca..5f01f8c7093a58c0a4644580cfe493f03266be75 100644 (file)
@@ -874,7 +874,7 @@ schedule_instructions(struct vc4_compile *c,
 {
         uint32_t time = 0;
 
-        while (!list_empty(&scoreboard->dag->heads)) {
+        while (!list_is_empty(&scoreboard->dag->heads)) {
                 struct schedule_node *chosen =
                         choose_instruction_to_schedule(scoreboard,
                                                        schedule_list,
@@ -995,7 +995,7 @@ qpu_schedule_instructions_block(struct vc4_compile *c,
 
         /* Wrap each instruction in a scheduler structure. */
         uint32_t next_sched_uniform = *next_uniform;
-        while (!list_empty(&block->qpu_inst_list)) {
+        while (!list_is_empty(&block->qpu_inst_list)) {
                 struct queued_qpu_inst *inst =
                         (struct queued_qpu_inst *)block->qpu_inst_list.next;
                 struct schedule_node *n = rzalloc(scoreboard->dag,
index 53d4356b1a1e2a9764533a8bd142040ba8b40fef..32ac276d2b6355673873ddb0a8924fb4ae3ca16f 100644 (file)
@@ -961,7 +961,7 @@ static void amdgpu_bo_sparse_destroy(struct pb_buffer *_buf)
       fprintf(stderr, "amdgpu: clearing PRT VA region on destroy failed (%d)\n", r);
    }
 
-   while (!list_empty(&bo->u.sparse.backing)) {
+   while (!list_is_empty(&bo->u.sparse.backing)) {
       struct amdgpu_sparse_backing *dummy = NULL;
       sparse_free_backing_buffer(bo,
                                  container_of(bo->u.sparse.backing.next,
index 722d85238a9c2bdf35bbb5797cd57b7571360c38..7acf03cbb2ea4e4df72c9ea81239ac2254bdc0a4 100644 (file)
@@ -165,7 +165,7 @@ align_and_verify_space(struct gen_aux_map_context *ctx, uint32_t size,
 static void
 get_current_pos(struct gen_aux_map_context *ctx, uint64_t *gpu, uint64_t **map)
 {
-   assert(!list_empty(&ctx->buffers));
+   assert(!list_is_empty(&ctx->buffers));
    struct aux_map_buffer *tail =
       list_last_entry(&ctx->buffers, struct aux_map_buffer, link);
    if (gpu)
index 58fabb179239ca7891616271ec7a8c1cb988b16f..45a5f56c8aab6475e8df9d5a9c625fd17a5b75a9 100644 (file)
@@ -36,7 +36,7 @@
 static inline bool
 are_all_uses_fadd(nir_ssa_def *def)
 {
-   if (!list_empty(&def->if_uses))
+   if (!list_is_empty(&def->if_uses))
       return false;
 
    nir_foreach_use(use_src, def) {
@@ -153,7 +153,7 @@ any_alu_src_is_a_constant(nir_alu_src srcs[])
             nir_instr_as_load_const (srcs[i].src.ssa->parent_instr);
 
          if (list_is_singular(&load_const->def.uses) &&
-             list_empty(&load_const->def.if_uses)) {
+             list_is_empty(&load_const->def.if_uses)) {
             return true;
          }
       }
@@ -256,7 +256,7 @@ brw_nir_opt_peephole_ffma_block(nir_builder *b, nir_block *block)
                                nir_src_for_ssa(&ffma->dest.dest.ssa));
 
       nir_builder_instr_insert(b, &ffma->instr);
-      assert(list_empty(&add->dest.dest.ssa.uses));
+      assert(list_is_empty(&add->dest.dest.ssa.uses));
       nir_instr_remove(&add->instr);
 
       progress = true;
index 3b1f713ef5b1ae170be27ba4bc6a07d24fb8c9a5..0d4da389da1b23c2c677d6ede821cf90ae17d5a7 100644 (file)
@@ -812,13 +812,13 @@ void
 anv_cmd_buffer_reset_batch_bo_chain(struct anv_cmd_buffer *cmd_buffer)
 {
    /* Delete all but the first batch bo */
-   assert(!list_empty(&cmd_buffer->batch_bos));
+   assert(!list_is_empty(&cmd_buffer->batch_bos));
    while (cmd_buffer->batch_bos.next != cmd_buffer->batch_bos.prev) {
       struct anv_batch_bo *bbo = anv_cmd_buffer_current_batch_bo(cmd_buffer);
       list_del(&bbo->link);
       anv_batch_bo_destroy(bbo, cmd_buffer);
    }
-   assert(!list_empty(&cmd_buffer->batch_bos));
+   assert(!list_is_empty(&cmd_buffer->batch_bos));
 
    anv_batch_bo_start(anv_cmd_buffer_current_batch_bo(cmd_buffer),
                       &cmd_buffer->batch,
index 9e6eaab8c80991cd4f7fad568bf0b46088c531e6..75654da8f24bc68e684746150194d8bde2a2aeac 100644 (file)
@@ -68,7 +68,7 @@ compiler_debug_log(void *data, const char *fmt, ...)
    char str[MAX_DEBUG_MESSAGE_LENGTH];
    struct anv_device *device = (struct anv_device *)data;
 
-   if (list_empty(&device->instance->debug_report_callbacks.callbacks))
+   if (list_is_empty(&device->instance->debug_report_callbacks.callbacks))
       return;
 
    va_list args;
index eefa548552ad19f041d00d29409feaef483cc69f..536fe36e5e43902e47f741deb0b86d9ac932d656 100644 (file)
@@ -531,7 +531,7 @@ bo_alloc_internal(struct brw_bufmgr *bufmgr,
    /* Get a buffer out of the cache if available */
 retry:
    alloc_from_cache = false;
-   if (bucket != NULL && !list_empty(&bucket->head)) {
+   if (bucket != NULL && !list_is_empty(&bucket->head)) {
       if (busy && !zeroed) {
          /* Allocate new render-target BOs from the tail (MRU)
           * of the list, as it will likely be hot in the GPU
index a4502285f8ce34e892707a9d1510e628e5434c5e..7165f15f4ec3263fc14f2621436daf2e2e0593d9 100644 (file)
@@ -77,11 +77,11 @@ static inline void list_addtail(struct list_head *item, struct list_head *list)
     list->prev = item;
 }
 
-static inline bool list_empty(const struct list_head *list);
+static inline bool list_is_empty(const struct list_head *list);
 
 static inline void list_replace(struct list_head *from, struct list_head *to)
 {
-    if (list_empty(from)) {
+    if (list_is_empty(from)) {
         list_inithead(to);
     } else {
         to->prev = from->prev;
@@ -106,7 +106,7 @@ static inline void list_delinit(struct list_head *item)
     item->prev = item;
 }
 
-static inline bool list_empty(const struct list_head *list)
+static inline bool list_is_empty(const struct list_head *list)
 {
    return list->next == list;
 }
@@ -130,7 +130,7 @@ static inline unsigned list_length(const struct list_head *list)
 
 static inline void list_splice(struct list_head *src, struct list_head *dst)
 {
-   if (list_empty(src))
+   if (list_is_empty(src))
       return;
 
    src->next->prev = dst;
@@ -141,7 +141,7 @@ static inline void list_splice(struct list_head *src, struct list_head *dst)
 
 static inline void list_splicetail(struct list_head *src, struct list_head *dst)
 {
-   if (list_empty(src))
+   if (list_is_empty(src))
       return;
 
    src->prev->next = dst;
index 2a3c63e69c91273f74ff72e9bd58d755c4691f6f..652478407e7ac073b8dd50c1b67b45f8330cb40d 100644 (file)
@@ -493,7 +493,7 @@ static void destroy_swapchain_data(struct swapchain_data *data)
 struct overlay_draw *get_overlay_draw(struct swapchain_data *data)
 {
    struct device_data *device_data = data->device;
-   struct overlay_draw *draw = list_empty(&data->draws) ?
+   struct overlay_draw *draw = list_is_empty(&data->draws) ?
       NULL : list_first_entry(&data->draws, struct overlay_draw, link);
 
    VkSemaphoreCreateInfo sem_info = {};
@@ -2137,7 +2137,7 @@ static VkResult overlay_QueueSubmit(
              !cmd_buffer_data->timestamp_query_pool)
             continue;
 
-         if (list_empty(&cmd_buffer_data->link)) {
+         if (list_is_empty(&cmd_buffer_data->link)) {
             list_addtail(&cmd_buffer_data->link,
                          &queue_data->running_command_buffer);
          } else {
index 22ae4a7d931664bebb9e4c7217110426e8de2d12..c4884b963e052c59dda03a01a33943b2d10e8d55 100644 (file)
@@ -99,7 +99,7 @@ vk_debug_report(struct vk_debug_report_instance *instance,
                 const char *pMessage)
 {
    /* Allow NULL for convinience, return if no callbacks registered. */
-   if (!instance || list_empty(&instance->callbacks))
+   if (!instance || list_is_empty(&instance->callbacks))
       return;
 
    pthread_mutex_lock(&instance->callbacks_mutex);