Skip to content

Commit

Permalink
Vulkan: When deriving stencilref from color, don't forget to update t…
Browse files Browse the repository at this point in the history
…he value.

Plus minor cleanups.
  • Loading branch information
hrydgard committed Nov 12, 2017
1 parent ac78b90 commit 232e30f
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 2 additions & 0 deletions GPU/Vulkan/DrawEngineVulkan.cpp
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -997,6 +997,8 @@ void DrawEngineVulkan::DoFlush() {
} }
lastPipeline_ = pipeline; lastPipeline_ = pipeline;
} }
if (result.setStencil)
dynState_.stencilRef = result.stencilValue;

This comment has been minimized.

Copy link
@unknownbrackets

unknownbrackets Nov 12, 2017

Collaborator

Hmm, not 98f692c?

This comment has been minimized.

Copy link
@hrydgard

hrydgard Nov 12, 2017

Author Owner

Oh, duh. That's better. Please override my commit :)

ApplyDrawStateLate(renderManager, false, 0, pipeline->useBlendConstant); ApplyDrawStateLate(renderManager, false, 0, pipeline->useBlendConstant);
gstate_c.Clean(DIRTY_BLEND_STATE | DIRTY_DEPTHSTENCIL_STATE | DIRTY_RASTER_STATE | DIRTY_VIEWPORTSCISSOR_STATE); gstate_c.Clean(DIRTY_BLEND_STATE | DIRTY_DEPTHSTENCIL_STATE | DIRTY_RASTER_STATE | DIRTY_VIEWPORTSCISSOR_STATE);
lastPipeline_ = pipeline; lastPipeline_ = pipeline;
Expand Down
1 change: 0 additions & 1 deletion GPU/Vulkan/FramebufferVulkan.cpp
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -362,7 +362,6 @@ void FramebufferManagerVulkan::BindPostShader(const PostShaderUniforms &uniforms
} }


postShaderUniforms_ = uniforms; postShaderUniforms_ = uniforms;
// VulkanRenderManager *renderManager = (VulkanRenderManager *)draw_->GetNativeObject(Draw::NativeObject::RENDER_MANAGER);
cur2DPipeline_ = pipelinePostShader_; cur2DPipeline_ = pipelinePostShader_;


gstate_c.Dirty(DIRTY_VERTEXSHADER_STATE); gstate_c.Dirty(DIRTY_VERTEXSHADER_STATE);
Expand Down
3 changes: 2 additions & 1 deletion ext/native/base/logging.h
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -88,7 +88,8 @@ const char *GetFn(const char *fn);
OutputDebugStringUTF8(temp); \ OutputDebugStringUTF8(temp); \
} \ } \
} while (false) } while (false)
#define DUMPLOG(x) OutputDebugStringUTF8(x);
#define DUMPLOG(x) OutputDebugStringUTF8(x)


#ifdef _DEBUG #ifdef _DEBUG
#define DLOG(...) XLOG_IMPL("D", __VA_ARGS__) #define DLOG(...) XLOG_IMPL("D", __VA_ARGS__)
Expand Down
2 changes: 0 additions & 2 deletions ext/native/thin3d/VulkanQueueRunner.cpp
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@


// Debug help: adb logcat -s DEBUG PPSSPPNativeActivity PPSSPP // Debug help: adb logcat -s DEBUG PPSSPPNativeActivity PPSSPP


// TODO: This is only enough for 4x render resolution / 4x texture upscale for debugger.
// Maybe we should use a dynamically allocated one for larger?
void VulkanQueueRunner::CreateDeviceObjects() { void VulkanQueueRunner::CreateDeviceObjects() {
ILOG("VulkanQueueRunner::CreateDeviceObjects"); ILOG("VulkanQueueRunner::CreateDeviceObjects");
InitBackbufferRenderPass(); InitBackbufferRenderPass();
Expand Down

0 comments on commit 232e30f

Please sign in to comment.