Parcourir la source

gl_rasterizer: Split VAO and VB setup functions.

Markus Wick il y a 7 ans
Parent
commit
0590dd2971

+ 14 - 4
src/video_core/renderer_opengl/gl_rasterizer.cpp

@@ -33,7 +33,8 @@ using Maxwell = Tegra::Engines::Maxwell3D::Regs;
 using PixelFormat = VideoCore::Surface::PixelFormat;
 using PixelFormat = VideoCore::Surface::PixelFormat;
 using SurfaceType = VideoCore::Surface::SurfaceType;
 using SurfaceType = VideoCore::Surface::SurfaceType;
 
 
-MICROPROFILE_DEFINE(OpenGL_VAO, "OpenGL", "Vertex Array Setup", MP_RGB(128, 128, 192));
+MICROPROFILE_DEFINE(OpenGL_VAO, "OpenGL", "Vertex Format Setup", MP_RGB(128, 128, 192));
+MICROPROFILE_DEFINE(OpenGL_VB, "OpenGL", "Vertex Buffer Setup", MP_RGB(128, 128, 192));
 MICROPROFILE_DEFINE(OpenGL_Shader, "OpenGL", "Shader Setup", MP_RGB(128, 128, 192));
 MICROPROFILE_DEFINE(OpenGL_Shader, "OpenGL", "Shader Setup", MP_RGB(128, 128, 192));
 MICROPROFILE_DEFINE(OpenGL_UBO, "OpenGL", "Const Buffer Setup", MP_RGB(128, 128, 192));
 MICROPROFILE_DEFINE(OpenGL_UBO, "OpenGL", "Const Buffer Setup", MP_RGB(128, 128, 192));
 MICROPROFILE_DEFINE(OpenGL_Index, "OpenGL", "Index Buffer Setup", MP_RGB(128, 128, 192));
 MICROPROFILE_DEFINE(OpenGL_Index, "OpenGL", "Index Buffer Setup", MP_RGB(128, 128, 192));
@@ -122,7 +123,7 @@ RasterizerOpenGL::RasterizerOpenGL(Core::Frontend::EmuWindow& window, ScreenInfo
 
 
 RasterizerOpenGL::~RasterizerOpenGL() {}
 RasterizerOpenGL::~RasterizerOpenGL() {}
 
 
-void RasterizerOpenGL::SetupVertexArrays() {
+void RasterizerOpenGL::SetupVertexFormat() {
     MICROPROFILE_SCOPE(OpenGL_VAO);
     MICROPROFILE_SCOPE(OpenGL_VAO);
     const auto& gpu = Core::System::GetInstance().GPU().Maxwell3D();
     const auto& gpu = Core::System::GetInstance().GPU().Maxwell3D();
     const auto& regs = gpu.regs;
     const auto& regs = gpu.regs;
@@ -175,8 +176,13 @@ void RasterizerOpenGL::SetupVertexArrays() {
         }
         }
     }
     }
     state.draw.vertex_array = VAO.handle;
     state.draw.vertex_array = VAO.handle;
-    state.draw.vertex_buffer = buffer_cache.GetHandle();
     state.Apply();
     state.Apply();
+}
+
+void RasterizerOpenGL::SetupVertexBuffer() {
+    MICROPROFILE_SCOPE(OpenGL_VB);
+    const auto& gpu = Core::System::GetInstance().GPU().Maxwell3D();
+    const auto& regs = gpu.regs;
 
 
     // Upload all guest vertex arrays sequentially to our buffer
     // Upload all guest vertex arrays sequentially to our buffer
     for (u32 index = 0; index < Maxwell::NumVertexArrays; ++index) {
     for (u32 index = 0; index < Maxwell::NumVertexArrays; ++index) {
@@ -203,6 +209,9 @@ void RasterizerOpenGL::SetupVertexArrays() {
             glVertexBindingDivisor(index, 0);
             glVertexBindingDivisor(index, 0);
         }
         }
     }
     }
+
+    // Implicit set by glBindVertexBuffer. Stupid glstate handling...
+    state.draw.vertex_buffer = buffer_cache.GetHandle();
 }
 }
 
 
 DrawParameters RasterizerOpenGL::SetupDraw() {
 DrawParameters RasterizerOpenGL::SetupDraw() {
@@ -621,7 +630,8 @@ void RasterizerOpenGL::DrawArrays() {
 
 
     buffer_cache.Map(buffer_size);
     buffer_cache.Map(buffer_size);
 
 
-    SetupVertexArrays();
+    SetupVertexFormat();
+    SetupVertexBuffer();
     DrawParameters params = SetupDraw();
     DrawParameters params = SetupDraw();
     SetupShaders(params.primitive_mode);
     SetupShaders(params.primitive_mode);
 
 

+ 2 - 1
src/video_core/renderer_opengl/gl_rasterizer.h

@@ -207,7 +207,8 @@ private:
 
 
     std::size_t CalculateIndexBufferSize() const;
     std::size_t CalculateIndexBufferSize() const;
 
 
-    void SetupVertexArrays();
+    void SetupVertexFormat();
+    void SetupVertexBuffer();
 
 
     DrawParameters SetupDraw();
     DrawParameters SetupDraw();