OGL: support palette texture decoding #2085
Conversation
degasus
force-pushed
the
ogl_paletted
branch
5 times, most recently
from
cec6142
to
5a55ca3
Compare
| } | ||
|
|
||
|
|
||
| TextureCache::~TextureCache() | ||
| { | ||
| DeleteShaders(); | ||
| delete s_palette_stream_buffer; | ||
| s_palette_stream_buffer = nullptr; | ||
| glDeleteTextures(1, &s_palette_resolv_texture); |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
degasus
force-pushed
the
ogl_paletted
branch
from
5a55ca3
to
66cc062
Compare
| { | ||
| // TODO: Implement. | ||
| return; | ||
| g_renderer->ResetAPIState(); |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
degasus
force-pushed
the
ogl_paletted
branch
from
66cc062
to
b0e483f
Compare
|
LGTM 👍 |
| @@ -465,6 +465,7 @@ Renderer::Renderer() | |||
| g_Config.backend_info.bSupportsBBox = GLExtensions::Supports("GL_ARB_shader_storage_buffer_object"); | |||
| g_Config.backend_info.bSupportsGSInstancing = GLExtensions::Supports("GL_ARB_gpu_shader5"); | |||
| g_Config.backend_info.bSupportsGeometryShaders = GLExtensions::Version() >= 320; | |||
| g_Config.backend_info.bSupportsPaletteConversion = GLExtensions::Supports("GL_ARB_texture_buffer_object"); | |||
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
|
I'm assuming @degasus will fix it on GLES before considering to get this merged. |
|
Android is using the default gameinis too? Those are made taking a compatibility approach and in regard to pc hardware, in android i think compatibility via settings is the least of the worries when most games are unplayable due to speed restrictions even with the fastest settings. |
degasus
force-pushed
the
ogl_paletted
branch
from
b0e483f
to
c7253be
Compare
|
This PR is ready imo, so reviews are welcome ;D |
degasus
force-pushed
the
ogl_paletted
branch
2 times, most recently
from
8d0b223
to
fdc3e17
Compare
|
👍 |
| @@ -140,7 +140,7 @@ static void InitBackendInfo() | |||
| g_Config.backend_info.bSupportsGeometryShaders = true; | |||
| g_Config.backend_info.bSupports3DVision = false; | |||
| g_Config.backend_info.bSupportsPostProcessing = true; | |||
| g_Config.backend_info.bSupportsPaletteConversion = false; | |||
| //g_Config.backend_info.bSupportsPaletteConversion = true; // is GPU dependent and must be set in renderer | |||
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
|
HdkR> I'm fine with it |
degasus
force-pushed
the
ogl_paletted
branch
from
fdc3e17
to
ac71029
Compare
OGL: support palette texture decoding
|
What a win for Simpsons and Twilight Princess. |
|
FifoCI detected that this change impacts graphical rendering. Here are the behavior differences detected by the system:
automated-fifoci-reporter |
No description provided.