[PATCH 1/2] Mark swapbuffers as being a rendering command.

[José Fonseca]

Applied. Thanks.

On Tue, Sep 4, 2012 at 8:28 PM, Eric Anholt <eric at anholt.net> wrote:
> It affects the contents of framebuffers, is something you want to
> include in a "/draw" callset, and is something you want to do gpu-side
> profiling on.
> ---
>  common/trace_parser_flags.cpp |    8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/common/trace_parser_flags.cpp b/common/trace_parser_flags.cpp
> index dcc846b..a03e9cf 100644
> --- a/common/trace_parser_flags.cpp
> +++ b/common/trace_parser_flags.cpp
> @@ -176,14 +176,14 @@ callFlagTable[] = {
>      { "glXQueryExtension",                             CALL_FLAG_NO_SIDE_EFFECTS | CALL_FLAG_VERBOSE },
>      { "glXQueryExtensionsString",                      CALL_FLAG_NO_SIDE_EFFECTS | CALL_FLAG_VERBOSE },
>      { "glXQueryVersion",                               CALL_FLAG_NO_SIDE_EFFECTS | CALL_FLAG_VERBOSE },
> -    { "glXSwapBuffers",                                CALL_FLAG_SWAPBUFFERS },
> +    { "glXSwapBuffers",                                CALL_FLAG_RENDER | CALL_FLAG_SWAPBUFFERS},
>      { "wglGetCurrentContext",                          CALL_FLAG_NO_SIDE_EFFECTS | CALL_FLAG_VERBOSE },
>      { "wglGetCurrentDC",                               CALL_FLAG_NO_SIDE_EFFECTS | CALL_FLAG_VERBOSE },
>      { "wglGetDefaultProcAddress",                      CALL_FLAG_NO_SIDE_EFFECTS | CALL_FLAG_VERBOSE },
>      { "wglGetProcAddress",                             CALL_FLAG_NO_SIDE_EFFECTS | CALL_FLAG_VERBOSE },
> -    { "wglSwapBuffers",                                CALL_FLAG_SWAPBUFFERS },
> -    { "wglSwapLayerBuffers",                           CALL_FLAG_SWAPBUFFERS },
> -    { "wglSwapMultipleBuffers",                        CALL_FLAG_SWAPBUFFERS },
> +    { "wglSwapBuffers",                                CALL_FLAG_RENDER | CALL_FLAG_SWAPBUFFERS },
> +    { "wglSwapLayerBuffers",                           CALL_FLAG_RENDER | CALL_FLAG_SWAPBUFFERS },
> +    { "wglSwapMultipleBuffers",                        CALL_FLAG_RENDER | CALL_FLAG_SWAPBUFFERS },
>      // NOTE: New entries must be sorted alphabetically
>  };
>
> --
> 1.7.10.4
>
> _______________________________________________
> apitrace mailing list
> apitrace at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/apitrace