Skip to content

Commit

Permalink
Merge pull request #18013 from hrydgard/remove-bad-heuristic
Browse files Browse the repository at this point in the history
Replace a too-simple heuristic with a compat flag, fixing Castlevania flicker.
  • Loading branch information
hrydgard committed Aug 31, 2023
2 parents 7392fd1 + 131163b commit 92e600a
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 8 deletions.
8 changes: 0 additions & 8 deletions GPU/Common/FramebufferManagerCommon.cpp
Expand Up @@ -426,14 +426,6 @@ VirtualFramebuffer *FramebufferManagerCommon::DoSetRenderFrameBuffer(Framebuffer
u32 v_fb_end_ptr = v->fb_address + v->fb_stride * v->height * bpp;

if (params.fb_address > v->fb_address && params.fb_address < v_fb_first_line_end_ptr) {
// If the framebuffer we can join to is currently bound as a texture, we likely have
// a situation like in #9324 and don't want to do this.
u32 curTextureAddress = gstate.getTextureAddress(0);
if (v->fb_address == curTextureAddress) {
// Don't try these joining shenanigans.
continue;
}

const int x_offset = (params.fb_address - v->fb_address) / bpp;
if (x_offset < params.fb_stride && v->height >= drawing_height) {
// Pretty certainly a pure render-to-X-offset.
Expand Down
5 changes: 5 additions & 0 deletions assets/compat.ini
Expand Up @@ -1512,6 +1512,11 @@ ULES00227 = true
ULKS46057 = true
ULUS10050 = true

# Rainbow Six: Vegas: See #9324
# Replaces a heuristic we added in #16971, which broke Castlevania - Symphony of the Night.
ULES00584 = true
ULUS10206 = true

[RockmanDash2SoundFix]
# Rockman Dash 2, see #11442
ULJM05037 = true
Expand Down

0 comments on commit 92e600a

Please sign in to comment.