From eb25dfd399d481ce770c8851416b70b077892afc Mon Sep 17 00:00:00 2001 From: Tom Englund Date: Sat, 27 Sep 2025 18:46:26 +0200 Subject: opengl: move from unordered_set to array setCapStatus is a a heavy used function in hot rendering paths, it shows up in profiling as using a bit of cpu just because of unordered_set hashing etc, move to a enum and array and cache only the heavily used ones. --- src/render/OpenGL.cpp | 34 ++++++++++++++++++++++++++-------- 1 file changed, 26 insertions(+), 8 deletions(-) (limited to 'src/render/OpenGL.cpp') diff --git a/src/render/OpenGL.cpp b/src/render/OpenGL.cpp index e1ecdd2f..fd5ce164 100644 --- a/src/render/OpenGL.cpp +++ b/src/render/OpenGL.cpp @@ -3170,18 +3170,36 @@ void CHyprOpenGLImpl::setViewport(GLint x, GLint y, GLsizei width, GLsizei heigh } void CHyprOpenGLImpl::setCapStatus(int cap, bool status) { - // check if the capability status is already set to the desired status - auto it = m_capStatus.find(cap); - bool currentStatus = (it != m_capStatus.end()) ? it->second : false; // default to 'false' if not found + const auto getCapIndex = [cap]() { + switch (cap) { + case GL_BLEND: return CAP_STATUS_BLEND; + case GL_SCISSOR_TEST: return CAP_STATUS_SCISSOR_TEST; + case GL_STENCIL_TEST: return CAP_STATUS_STENCIL_TEST; + default: return CAP_STATUS_END; + } + }; + + auto idx = getCapIndex(); + + if (idx == CAP_STATUS_END) { + if (status) + glEnable(cap); + else + glDisable(cap); - if (currentStatus == status) return; + } - m_capStatus[cap] = status; + if (m_capStatus[idx] == status) + return; - // Enable or disable the capability based on status - auto func = status ? [](int c) { glEnable(c); } : [](int c) { glDisable(c); }; - func(cap); + if (status) { + m_capStatus[idx] = status; + glEnable(cap); + } else { + m_capStatus[idx] = status; + glDisable(cap); + } } uint32_t CHyprOpenGLImpl::getPreferredReadFormat(PHLMONITOR pMonitor) { -- cgit v1.2.3