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

[d3d9] Adjust SM2 VS/PS caps #4647

Merged
merged 1 commit into from
Jan 26, 2025
Merged

Conversation

WinterSnowfall
Copy link
Contributor

@WinterSnowfall WinterSnowfall commented Jan 26, 2025

These were ultimately (largely) correct, but they do have D3D constants defined.

I still have to check how all of these show up on a SM1 capable card, so WIP for now. Update: I've checked, and they're all 0 for SM1 (and fixed-function ofc).

Doubt it matters much beyond esthetics, but let's unleash it in experimental for a while.

@WinterSnowfall WinterSnowfall marked this pull request as ready for review January 26, 2025 20:29
@doitsujin
Copy link
Owner

Has this been compared against native, or do the actuial values not change and we just report 0 if we disable sm2 support?

@WinterSnowfall
Copy link
Contributor Author

WinterSnowfall commented Jan 26, 2025

Has this been compared against native, or do the actuial values not change and we just report 0 if we disable sm2 support?

For now I've only compared it with WineD3D, which seems to follow the constants, so I don't have much concerns there. And for SM1, I checked on Windows XP / a GeForce 4 Ti and they are all 0.

P.S.: The constants are identical in value to what @misyltoad had set initially, btw.

@doitsujin doitsujin merged commit 5b21286 into doitsujin:master Jan 26, 2025
4 checks passed
@WinterSnowfall WinterSnowfall deleted the d3d9-sm2caps branch January 26, 2025 21:05
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.

2 participants