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

recompiler: fixed fragment shader built-in attribute access #1676

Merged
merged 3 commits into from
Dec 6, 2024

Conversation

roamic
Copy link
Collaborator

@roamic roamic commented Dec 6, 2024

Registers contain enable flags for built-in attributes that should be checked when recompiling in order to access the correct built-in.

Fixes inverted normals issue in BB that caused some artifacts on grass and mirrored objects (like dog fur).

@GHU7924
Copy link

GHU7924 commented Dec 6, 2024

shadPS4 v0 4 1 WIP fix-fs-inputs 81a5062 _ CUSA03173 - Bloodborne™ 01 09 Пт, 6 декабря 2024 г  22_38_23
shadPS4 v0 4 1 WIP fix-fs-inputs 81a5062 _ CUSA03173 - Bloodborne™ 01 09 Пт, 6 декабря 2024 г  22_39_59

When I first started the game, it crashed without errors, immediately after the character appeared on the location, but then it loaded normally, I managed to run a little. Although the performance of the game seemed to deteriorate (I did not immediately turn on the fps counter, but later I saw 25-28 frames in places).
Your fix seems to fix the problems. (Dark spots are a problem #1594, so don't pay attention to it).

@ZuwasieMan
Copy link

ZuwasieMan commented Dec 6, 2024

It is a subtle difference, but it is there.
It looks less pixelated and more natural, softer and smoother in this PR.
The older implementation (main) was very dark, probably incorrect.

i.e. lets have a look at some grass on the ground.
(Btw it is better to compare the images at full screen)

main:
bb_otroej_hierba

PR:
bb_otroej_hierba_despues png

A closer look at the same grass:

main:
bb_hierba

PR:
bb_hierba_despues

I was not able to see any performance changes.

Very good job

@raphaelthegreat raphaelthegreat merged commit 8eacb88 into shadps4-emu:main Dec 6, 2024
10 checks passed
diegolix29 pushed a commit to diegolix29/shadPS4 that referenced this pull request Dec 7, 2024
…emu#1676)

* recompiler: fixed fragment shader built-in attribute access

* handle en/addr separately

* handle other registers as well
diegolix29 pushed a commit to diegolix29/shadPS4 that referenced this pull request Dec 7, 2024
…emu#1676)

* recompiler: fixed fragment shader built-in attribute access

* handle en/addr separately

* handle other registers as well
diegolix29 pushed a commit to diegolix29/shadPS4 that referenced this pull request Dec 8, 2024
…emu#1676)

* recompiler: fixed fragment shader built-in attribute access

* handle en/addr separately

* handle other registers as well
diegolix29 pushed a commit to diegolix29/shadPS4 that referenced this pull request Dec 9, 2024
…emu#1676)

* recompiler: fixed fragment shader built-in attribute access

* handle en/addr separately

* handle other registers as well
@roamic roamic deleted the fix-fs-inputs branch December 9, 2024 15:17
lucashartmann added a commit to lucashartmann/shadPS4 that referenced this pull request Dec 9, 2024
Xcedf pushed a commit to Xcedf/shadPS4 that referenced this pull request Dec 10, 2024
…emu#1676)

* recompiler: fixed fragment shader built-in attribute access

* handle en/addr separately

* handle other registers as well
Xcedf pushed a commit to Xcedf/shadPS4 that referenced this pull request Dec 10, 2024
…emu#1676)

* recompiler: fixed fragment shader built-in attribute access

* handle en/addr separately

* handle other registers as well
Xcedf pushed a commit to Xcedf/shadPS4 that referenced this pull request Dec 28, 2024
…emu#1676)

* recompiler: fixed fragment shader built-in attribute access

* handle en/addr separately

* handle other registers as well
Xcedf pushed a commit to Xcedf/shadPS4 that referenced this pull request Dec 28, 2024
…emu#1676)

* recompiler: fixed fragment shader built-in attribute access

* handle en/addr separately

* handle other registers as well
Xcedf added a commit to Xcedf/shadPS4 that referenced this pull request Dec 31, 2024
…hadps4-emu#1676)"

This reverts commit 8eacb88.

Revert "semaphore: Add GCD semaphore implementation. (shadps4-emu#1677)"

This reverts commit e1ecfb8.

Revert "hot-fix: Silence depth macrotiled warning"

This reverts commit 357b782.

Revert "buffer_cache: Bump usable address space to 40bits"

This reverts commit 6acfdd5.
Xcedf added a commit to Xcedf/shadPS4 that referenced this pull request Dec 31, 2024
Xcedf added a commit to Xcedf/shadPS4 that referenced this pull request Dec 31, 2024
…hadps4-emu#1676)"

This reverts commit 8eacb88.

Revert "semaphore: Add GCD semaphore implementation. (shadps4-emu#1677)"

This reverts commit e1ecfb8.
Xcedf added a commit to Xcedf/shadPS4 that referenced this pull request Dec 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants