From: Marek Olšák <marek.olsak@amd.com>
Date: Thu, 27 Feb 2020 21:06:47 +0000 (-0500)
Subject: Revert "mesa: check for z=0 in _mesa_Vertex3dv()"
X-Git-Tag: upstream/20.1.8~2899
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=df3891e74a72d275aceba91adc94a9e7dc9aa029;p=platform%2Fupstream%2Fmesa.git

Revert "mesa: check for z=0 in _mesa_Vertex3dv()"

This reverts commit f04d7439a0ad6e13ff2912ff824553b6bcf511a4.

It no longer helps performance and the current vbo implementation is
faster anyway.

The app that hit this was a CAD program called Spazio3D. It made pretty
terrible use of the OpenGL API and we sent them some tips for improvements.
I'm assuming they've fixed this by now.

Reviewed-by: Mathias Fröhlich <Mathias.Froehlich@web.de>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4052>
---

diff --git a/src/mesa/main/api_loopback.c b/src/mesa/main/api_loopback.c
index c03c33e..53873ea 100644
--- a/src/mesa/main/api_loopback.c
+++ b/src/mesa/main/api_loopback.c
@@ -631,10 +631,7 @@ _mesa_Vertex2sv( const GLshort *v )
 void GLAPIENTRY
 _mesa_Vertex3dv( const GLdouble *v )
 {
-   if (v[2] == 0.0)
-      VERTEX2( (GLfloat) v[0], (GLfloat) v[1] );
-   else
-      VERTEX3( (GLfloat) v[0], (GLfloat) v[1], (GLfloat) v[2] );
+   VERTEX3( (GLfloat) v[0], (GLfloat) v[1], (GLfloat) v[2] );
 }
 
 void GLAPIENTRY