gallium: remove depth.occlusion_count flag
authorKeith Whitwell <keithw@vmware.com>
Thu, 1 Oct 2009 13:34:23 +0000 (14:34 +0100)
committerKeith Whitwell <keithw@vmware.com>
Thu, 1 Oct 2009 13:34:23 +0000 (14:34 +0100)
This was redundant as drivers can just keep track of whether they are
inside a begin/end query pair.  We want to add more query types later
and also support nested queries, none of which map well onto a flag like
this.  No driver appeared to be using the flag.

src/gallium/drivers/llvmpipe/lp_bld_depth.c
src/gallium/drivers/llvmpipe/lp_state_fs.c
src/gallium/drivers/softpipe/sp_video_context.c
src/gallium/include/pipe/p_state.h
src/gallium/state_trackers/g3dvl/vl_context.c
src/mesa/state_tracker/st_atom_depth.c

index 21c665c4d4c2511c392019f0a0b96b979782d89f..98ec1cb1b9dca428118830ae8e8f70173a4d9648 100644 (file)
@@ -210,7 +210,4 @@ lp_build_depth_test(LLVMBuilderRef builder,
       dst = lp_build_select(&bld, z_bitmask, src, dst);
       LLVMBuildStore(builder, dst, dst_ptr);
    }
-
-   /* FIXME */
-   assert(!state->occlusion_count);
 }
index d5ce6993c5d1fbdc55de381c5796de7de0f7b5c7..b00be0cc32af5635c1a40028be93457397c3587e 100644 (file)
@@ -401,7 +401,6 @@ generate_fragment(struct llvmpipe_context *lp,
    if(key->depth.enabled) {
       debug_printf("depth.func = %s\n", debug_dump_func(key->depth.func, TRUE));
       debug_printf("depth.writemask = %u\n", key->depth.writemask);
-      debug_printf("depth.occlusion_count = %u\n", key->depth.occlusion_count);
    }
    if(key->alpha.enabled) {
       debug_printf("alpha.func = %s\n", debug_dump_func(key->alpha.func, TRUE));
index ccb29726b6a9d3a32dce9c921ed8a26fb33abf1c..7e9136d8e09fd96503820a12a1409b06cfa05cc1 100644 (file)
@@ -167,7 +167,6 @@ init_pipe_state(struct sp_mpeg12_context *ctx)
    dsa.depth.enabled = 0;
    dsa.depth.writemask = 0;
    dsa.depth.func = PIPE_FUNC_ALWAYS;
-   dsa.depth.occlusion_count = 0;
    for (i = 0; i < 2; ++i) {
       dsa.stencil[i].enabled = 0;
       dsa.stencil[i].func = PIPE_FUNC_ALWAYS;
index b59d6b7ae32ac5b4b8a32c250e016aab2533fc4b..287b424e4aca6abd6c783a0dcdfe02236cb02634 100644 (file)
@@ -197,7 +197,6 @@ struct pipe_depth_state
    unsigned enabled:1;         /**< depth test enabled? */
    unsigned writemask:1;       /**< allow depth buffer writes? */
    unsigned func:3;            /**< depth test func (PIPE_FUNC_x) */
-   unsigned occlusion_count:1; /**< do occlusion counting? */
 };
 
 
index 5cfd233c4c100cbde0cc4aa587bffe774c426719..cfbf618d742adc28d37f7247d7bf6f14f6b42e5f 100644 (file)
@@ -69,7 +69,6 @@ static int vlInitCommon(struct vlContext *context)
        dsa.depth.enabled = 0;
        dsa.depth.writemask = 0;
        dsa.depth.func = PIPE_FUNC_ALWAYS;
-       dsa.depth.occlusion_count = 0;
        for (i = 0; i < 2; ++i)
        {
                dsa.stencil[i].enabled = 0;
index 0aa128f94793af443677c4342720367b344ea70d..88b80a07fc98402fd89a6c7022eb7995ecb6910c 100644 (file)
@@ -104,10 +104,6 @@ update_depth_stencil_alpha(struct st_context *st)
       dsa->depth.func = st_compare_func_to_pipe(ctx->Depth.Func);
    }
 
-   if (ctx->Query.CurrentOcclusionObject &&
-       ctx->Query.CurrentOcclusionObject->Active)
-      dsa->depth.occlusion_count = 1;
-
    if (ctx->Stencil.Enabled && ctx->DrawBuffer->Visual.stencilBits > 0) {
       dsa->stencil[0].enabled = 1;
       dsa->stencil[0].func = st_compare_func_to_pipe(ctx->Stencil.Function[0]);