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

AMDGPU SALU pseudos should not have side effects #128685

Open
jayfoad opened this issue Feb 25, 2025 · 1 comment
Open

AMDGPU SALU pseudos should not have side effects #128685

jayfoad opened this issue Feb 25, 2025 · 1 comment

Comments

@jayfoad
Copy link
Contributor

jayfoad commented Feb 25, 2025

The following pseudos do not have an explicit hasSideEffects = 0 in the *.td files, so the MCInstrs are marked as MCID::UnmodeledSideEffects:

S_ADD_CO_PSEUDO
S_MUL_I64_I32_PSEUDO
S_MUL_U64_U32_PSEUDO
S_SUB_CO_PSEUDO
S_UADDO_PSEUDO
S_USUBO_PSEUDO

These are all regular ALU operations without side effects.

@llvmbot
Copy link
Member

llvmbot commented Feb 25, 2025

@llvm/issue-subscribers-backend-amdgpu

Author: Jay Foad (jayfoad)

The following pseudos do not have an explicit `hasSideEffects = 0` in the *.td files, so the MCInstrs are marked as `MCID::UnmodeledSideEffects`: ``` S_ADD_CO_PSEUDO S_MUL_I64_I32_PSEUDO S_MUL_U64_U32_PSEUDO S_SUB_CO_PSEUDO S_UADDO_PSEUDO S_USUBO_PSEUDO ``` These are all regular ALU operations without side effects.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants