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

Implement flags for RDNA instructions #728

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

OFFTKP
Copy link
Contributor

@OFFTKP OFFTKP commented Sep 2, 2024

No description provided.

@OFFTKP OFFTKP changed the title Implement overflow flag for S_ADD_I32 Implement overflow flag for scalar instructions Sep 2, 2024
@OFFTKP OFFTKP changed the title Implement overflow flag for scalar instructions Implement flags for scalar instructions Sep 2, 2024
@OFFTKP OFFTKP changed the title Implement flags for scalar instructions Implement flags for RDNA instructions Sep 2, 2024
@OFFTKP
Copy link
Contributor Author

OFFTKP commented Sep 2, 2024

I didn't really test, just went off the manual

@Marocco2
Copy link

Marocco2 commented Sep 2, 2024

I didn't really test, just went off the manual

I have a RDNA2 GPU. How can I test this PR?

@OFFTKP
Copy link
Contributor Author

OFFTKP commented Sep 2, 2024

I didn't really test, just went off the manual

I have a RDNA2 GPU. How can I test this PR?

It's not about an RDNA2 GPU, it's mostly about finding a shader that uses these instructions and depends on these carry values which is not easy

I think it just needs someone to give it a read to make sure everything looks fine and it should be good.

@Hermiten
Copy link
Collaborator

Hermiten commented Oct 4, 2024

Open for 1 month, could a developer give his opinion?

@polybiusproxy
Copy link
Collaborator

I assume this is in PR limbo due to lack of testing

@OFFTKP
Copy link
Contributor Author

OFFTKP commented Oct 5, 2024

I assume this is in PR limbo due to lack of testing

https://discord.com/channels/1080089157554155590/1258386683691274361/1280292310499856435

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