Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rsx: Remove incorrect hack #13158

Merged
merged 1 commit into from
Jan 2, 2023
Merged

rsx: Remove incorrect hack #13158

merged 1 commit into from
Jan 2, 2023

Conversation

kd-11
Copy link
Contributor

@kd-11 kd-11 commented Jan 2, 2023

The commit was originally added to restore previously working functionality that was still not understood at the time. It solved the regression but I never went back to figure out if the "fix" was correct after making all the texture encoding discoveries using realhw. Turns out this hack actually breaks tons of games and the original bug was fixed properly years ago.
See notes in the guilty commit: f559c08

Fixes #8718 and probably a multitude of other alpha-related glitches.

@Megamouse
Copy link
Contributor

image

@kd-11 kd-11 merged commit 7423abb into RPCS3:master Jan 2, 2023
@kd-11 kd-11 deleted the rsx-b8-fix branch January 2, 2023 20:03
@ppamorim
Copy link

ppamorim commented Jan 3, 2023

@kd-11 Is this related to #12158?

@Rex000
Copy link

Rex000 commented Jan 3, 2023

The emulator won't update from a previous build to this new build. Tried updating from 0.0.25-14516, but it only updates to 0.0.25-14517

@ppamorim
Copy link

ppamorim commented Jan 3, 2023

@Rex000 Are you on macOS?

@TigiHof
Copy link

TigiHof commented Jan 3, 2023

The emulator won't update from a previous build to this new build. Tried updating from 0.0.25-14516, but it only updates to 0.0.25-14517

Can confirm this for Windows 10, no automatic update! I downloaded the update with this fix manually and got the "experimental build" warning.

@Asinin3
Copy link
Contributor

Asinin3 commented Jan 3, 2023

Its not related to kd-11's change. It's due to Cirrus dropping x86 macos containers after new years. We have been aware since before it happened, and it was being looked into for awhile now.

@Rex000
Copy link

Rex000 commented Jan 3, 2023

@Rex000 Are you on macOS?

Windows 10

@kd-11
Copy link
Contributor Author

kd-11 commented Jan 3, 2023

@kd-11 Is this related to #12158?

No, that is a moltenvk bug. I'll try some workarounds once macos builds are working again.

@RPCS3 RPCS3 deleted a comment from 5Soac5 Jan 18, 2023
@RPCS3 RPCS3 deleted a comment from Robatoshi Jan 18, 2023
@AniLeo
Copy link
Member

AniLeo commented Jan 18, 2023

Locking due to spam

@RPCS3 RPCS3 locked and limited conversation to collaborators Jan 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

COD Black Ops 2 [NPUB31054] - Text not visible
7 participants