Skip to content

Conversation

@Junru-Sheng
Copy link
Contributor

Implement conversion patterns for AtenSubOp, AtenMulOp, AtenDivOp:
Use unified template patterns ConvertAtenBinaryScalarOp to handle AtenAddOp, AtenSubOp, AtenMulOp;
Use unified template patterns ConvertAtenDivOp to handle AtenDivOp, AtenDivIntOp, AtenDivFloatOp.

@Junru-Sheng Junru-Sheng marked this pull request as ready for review December 9, 2025 15:26
Copy link
Contributor

@CoTinker CoTinker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks.

…lOp`, `AtenDivOp`.

Implement conversion patterns for `AtenSubOp`, `AtenMulOp`, `AtenDivOp`:
Use unified template patterns `ConvertAtenBinaryScalarOp` to handle `AtenAddOp`, `AtenSubOp`, `AtenMulOp`;
Use unified template patterns `ConvertAtenDivOp` to handle `AtenDivOp`, `AtenDivIntOp`, `AtenDivFloatOp`.
@zjgarvey
Copy link
Collaborator

Looks good, let me know if you want to merge.

@CoTinker CoTinker merged commit e3200d9 into llvm:main Dec 11, 2025
3 checks passed
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.

3 participants