Skip to content

Commit

Permalink
Auto merge of rust-lang#103299 - nikic:usub-overflow, r=wesleywiser
Browse files Browse the repository at this point in the history
Don't use usub.with.overflow intrinsic

The canonical form of a usub.with.overflow check in LLVM are separate sub + icmp instructions, rather than a usub.with.overflow intrinsic. Using usub.with.overflow will generally result in worse optimization potential.

The backend will attempt to form usub.with.overflow when it comes to actual instruction selection. This is not fully reliable, but I believe this is a better tradeoff than using the intrinsic in IR.

Fixes rust-lang#103285.
  • Loading branch information
bors committed Oct 30, 2022
2 parents a90279b + 24a3810 commit 6b61b21
Showing 0 changed files with 0 additions and 0 deletions.

0 comments on commit 6b61b21

Please sign in to comment.