Hi,
On Friday, 9 February 2018 02:18:44 CET Marek Olšák wrote:
> From: Marek Olšák <marek.ol...@amd.com>
>
> All drivers convert these to float, so there is no reason to use double.
> The piglit test that expects double precision from glGet will be adjusted
> not to require it (there is a piglit patch).
I am not aware of any additional requirement for the glGet precision beyond
the general standard float like guarantee from paragraph 2.3.3.
Reviewed-by: Mathias Fröhlich <mathias.froehl...@web.de>
best
Mathias
>
> gl_context::ViewportArray: 512 -> 384 bytes
> ---
> src/mesa/drivers/common/meta.h | 2 +-
> src/mesa/main/mtypes.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/mesa/drivers/common/meta.h b/src/mesa/drivers/common/meta.h
> index 1c35ea3..dbe8a9a 100644
> --- a/src/mesa/drivers/common/meta.h
> +++ b/src/mesa/drivers/common/meta.h
> @@ -151,21 +151,21 @@ struct save_state
> /** mask of TEXTURE_2D_BIT, etc */
> GLbitfield TexEnabled[MAX_TEXTURE_UNITS];
> GLbitfield TexGenEnabled[MAX_TEXTURE_UNITS];
> GLuint EnvMode; /* unit[0] only */
>
> /** MESA_META_VERTEX */
> struct gl_vertex_array_object *VAO;
>
> /** MESA_META_VIEWPORT */
> GLfloat ViewportX, ViewportY, ViewportW, ViewportH;
> - GLclampd DepthNear, DepthFar;
> + GLclampf DepthNear, DepthFar;
>
> /** MESA_META_CLAMP_FRAGMENT_COLOR */
> GLenum ClampFragmentColor;
>
> /** MESA_META_CLAMP_VERTEX_COLOR */
> GLenum ClampVertexColor;
>
> /** MESA_META_CONDITIONAL_RENDER */
> struct gl_query_object *CondRenderQuery;
> GLenum CondRenderMode;
> diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h
> index d8cb8f8..a589bd4 100644
> --- a/src/mesa/main/mtypes.h
> +++ b/src/mesa/main/mtypes.h
> @@ -1400,21 +1400,21 @@ struct gl_transform_attrib
> };
>
>
> /**
> * Viewport attribute group (GL_VIEWPORT_BIT).
> */
> struct gl_viewport_attrib
> {
> GLfloat X, Y; /**< position */
> GLfloat Width, Height; /**< size */
> - GLdouble Near, Far; /**< Depth buffer range */
> + GLfloat Near, Far; /**< Depth buffer range */
> };
>
>
> typedef enum
> {
> MAP_USER,
> MAP_INTERNAL,
> MAP_COUNT
> } gl_map_buffer_index;
>
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
