Skip to content

[Mips] Do not emit instruction teq if divisor is non-zero immediate value in FastISel implementation #135768

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

Merged
merged 1 commit into from
Apr 25, 2025

Conversation

yingopq
Copy link
Contributor

@yingopq yingopq commented Apr 15, 2025

Add a check before emitting the teq instruction to check whether the divisor is a non-zero immediate value.

Fix #130629.

@yingopq yingopq force-pushed the Fix_bug_issue_130629 branch from 5bff92a to 84da4de Compare April 15, 2025 09:44
Copy link

github-actions bot commented Apr 15, 2025

✅ With the latest revision this PR passed the C/C++ code formatter.

@yingopq yingopq force-pushed the Fix_bug_issue_130629 branch from 84da4de to c4ae12d Compare April 15, 2025 09:49
@brad0
Copy link
Contributor

brad0 commented Apr 16, 2025

cc @wzssyqa

@yingopq
Copy link
Contributor Author

yingopq commented Apr 23, 2025

Ping

@yingopq yingopq requested a review from wzssyqa April 23, 2025 07:19
@yingopq yingopq force-pushed the Fix_bug_issue_130629 branch from c4ae12d to 8fa06ea Compare April 24, 2025 08:38
@yingopq yingopq force-pushed the Fix_bug_issue_130629 branch from 8fa06ea to 126d7ba Compare April 25, 2025 02:51
…alue in FastISel implementation

Add a check before emitting the teq instruction to check whether
the divisor is a non-zero immediate value.

Fix llvm#130629.
@yingopq yingopq force-pushed the Fix_bug_issue_130629 branch from 126d7ba to 099c8fc Compare April 25, 2025 08:24
@wzssyqa wzssyqa merged commit 194da37 into llvm:main Apr 25, 2025
11 checks passed
qethu pushed a commit to zkMIPS/llvm-project that referenced this pull request Apr 25, 2025
… FastISel implementation (llvm#135768)

Add a check before emitting the teq instruction to check whether the
divisor is a non-zero immediate value.

Fix llvm#130629.
IanWood1 pushed a commit to IanWood1/llvm-project that referenced this pull request May 6, 2025
…alue in FastISel implementation (llvm#135768)

Add a check before emitting the teq instruction to check whether the
divisor is a non-zero immediate value.

Fix llvm#130629.
IanWood1 pushed a commit to IanWood1/llvm-project that referenced this pull request May 6, 2025
…alue in FastISel implementation (llvm#135768)

Add a check before emitting the teq instruction to check whether the
divisor is a non-zero immediate value.

Fix llvm#130629.
IanWood1 pushed a commit to IanWood1/llvm-project that referenced this pull request May 6, 2025
…alue in FastISel implementation (llvm#135768)

Add a check before emitting the teq instruction to check whether the
divisor is a non-zero immediate value.

Fix llvm#130629.
Ankur-0429 pushed a commit to Ankur-0429/llvm-project that referenced this pull request May 9, 2025
…alue in FastISel implementation (llvm#135768)

Add a check before emitting the teq instruction to check whether the
divisor is a non-zero immediate value.

Fix llvm#130629.
qethu pushed a commit to zkMIPS/llvm-project that referenced this pull request May 10, 2025
… FastISel implementation (llvm#135768)

Add a check before emitting the teq instruction to check whether the
divisor is a non-zero immediate value.

Fix llvm#130629.
qethu pushed a commit to zkMIPS/llvm-project that referenced this pull request May 22, 2025
… FastISel implementation (llvm#135768)

Add a check before emitting the teq instruction to check whether the
divisor is a non-zero immediate value.

Fix llvm#130629.
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.

MIPS: trap is not needed if divisor is not zero
3 participants