firka/flutter
firka/flutter
4
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[Impeller] remove extra validation checks in GLES backend. (flutter/engine#56944)

Browse Source 
We're already doing a check if the pipeline is valid in RenderPass::AddCommand. The index checks are validated during binding, and the check for instancing is unecessary given that we don't expose.

While I'm at it do some minor cleanups.
This commit is contained in:
Jonah Williams 2024-12-04 15:24:09 -08:00 committed by GitHub
parent fd12f3489c
commit ae6555d5a9
1 changed files with 8 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
engine/src/flutter/impeller/renderer/backend/gles/render_pass_gles.cc
View File

@ -6,12 +6,12 @@
#include <cstdint> #include <cstdint>
#include "GLES3/gl3.h"
#include "flutter/fml/trace_event.h" #include "flutter/fml/trace_event.h"
#include "fml/closure.h" #include "fml/closure.h"
#include "fml/logging.h" #include "fml/logging.h"
#include "impeller/base/validation.h" #include "impeller/base/validation.h"
#include "impeller/core/formats.h" #include "impeller/core/formats.h"
#include "impeller/renderer/backend/gles/buffer_bindings_gles.h"
#include "impeller/renderer/backend/gles/context_gles.h" #include "impeller/renderer/backend/gles/context_gles.h"
#include "impeller/renderer/backend/gles/device_buffer_gles.h" #include "impeller/renderer/backend/gles/device_buffer_gles.h"
#include "impeller/renderer/backend/gles/formats_gles.h" #include "impeller/renderer/backend/gles/formats_gles.h"
@ -218,7 +218,7 @@ void RenderPassGLES::ResetGLState(const ProcTableGLES& gl) {
} }
} else { } else {
// Create and bind an offscreen FBO. // Create and bind an offscreen FBO.
auto cached_fbo = color_gles.GetCachedFBO(); GLuint cached_fbo = color_gles.GetCachedFBO();
if (cached_fbo != GL_NONE) { if (cached_fbo != GL_NONE) {
fbo = cached_fbo; fbo = cached_fbo;
gl.BindFramebuffer(GL_FRAMEBUFFER, fbo); gl.BindFramebuffer(GL_FRAMEBUFFER, fbo);
@ -289,7 +289,7 @@ void RenderPassGLES::ResetGLState(const ProcTableGLES& gl) {
// Both the viewport and scissor are specified in framebuffer coordinates. // Both the viewport and scissor are specified in framebuffer coordinates.
// Impeller's framebuffer coordinate system is top left origin, but OpenGL's // Impeller's framebuffer coordinate system is top left origin, but OpenGL's
// is bottom left origin, so we convert the coordinates here. // is bottom left origin, so we convert the coordinates here.
auto target_size = pass_data.color_attachment->GetSize(); ISize target_size = pass_data.color_attachment->GetSize();
//-------------------------------------------------------------------------- //--------------------------------------------------------------------------
/// Setup the viewport. /// Setup the viewport.
@ -310,16 +310,6 @@ void RenderPassGLES::ResetGLState(const ProcTableGLES& gl) {
} }
for (const auto& command : commands) { for (const auto& command : commands) {
if (command.instance_count != 1u) {
VALIDATION_LOG << "GLES backend does not support instanced rendering.";
return false;
}
if (!command.pipeline) {
VALIDATION_LOG << "Command has no pipeline specified.";
return false;
}
#ifdef IMPELLER_DEBUG #ifdef IMPELLER_DEBUG
fml::ScopedCleanupClosure pop_cmd_debug_marker( fml::ScopedCleanupClosure pop_cmd_debug_marker(
[&gl]() { gl.PopDebugGroup(); }); [&gl]() { gl.PopDebugGroup(); });
@ -426,11 +416,7 @@ void RenderPassGLES::ResetGLState(const ProcTableGLES& gl) {
break; break;
} }
auto vertex_desc_gles = pipeline.GetBufferBindings(); BufferBindingsGLES* vertex_desc_gles = pipeline.GetBufferBindings();
if (command.index_type == IndexType::kUnknown) {
return false;
}
//-------------------------------------------------------------------------- //--------------------------------------------------------------------------
/// Bind vertex buffers. /// Bind vertex buffers.
@ -471,7 +457,8 @@ void RenderPassGLES::ResetGLState(const ProcTableGLES& gl) {
// correct; full triangle outlines won't be drawn and disconnected // correct; full triangle outlines won't be drawn and disconnected
// geometry may appear connected. However this can still be useful for // geometry may appear connected. However this can still be useful for
// wireframe debug views. // wireframe debug views.
auto mode = pipeline.GetDescriptor().GetPolygonMode() == PolygonMode::kLine GLenum mode =
pipeline.GetDescriptor().GetPolygonMode() == PolygonMode::kLine
? GL_LINE_STRIP ? GL_LINE_STRIP
: ToMode(pipeline.GetDescriptor().GetPrimitiveType()); : ToMode(pipeline.GetDescriptor().GetPrimitiveType());