[turbofan] Fix algorithm comment in Int32Mod
We don't currently try the fast path if left side is negative. See {if_lhs_negative} branch. Change-Id: Ieea2977209902ad90c29ff3b305ca047fafbdd80 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3974511 Auto-Submit: Victor Gomes <victorgomes@chromium.org> Commit-Queue: Nico Hartmann <nicohartmann@chromium.org> Reviewed-by: Nico Hartmann <nicohartmann@chromium.org> Cr-Commit-Position: refs/heads/main@{#83873}
This commit is contained in:
parent
c23bc72c83
commit
defc4088cc
@ -2304,19 +2304,13 @@ Node* EffectControlLinearizer::LowerCheckedInt32Mod(Node* node,
|
||||
// if rhs <= 0 then
|
||||
// rhs = -rhs
|
||||
// deopt if rhs == 0
|
||||
// let msk = rhs - 1 in
|
||||
// if lhs < 0 then
|
||||
// let lhs_abs = -lsh in
|
||||
// let res = if rhs & msk == 0 then
|
||||
// lhs_abs & msk
|
||||
// else
|
||||
// lhs_abs % rhs in
|
||||
// if lhs < 0 then
|
||||
// deopt if res == 0
|
||||
// -res
|
||||
// else
|
||||
// res
|
||||
// let res = lhs_abs % rhs in
|
||||
// deopt if res == 0
|
||||
// -res
|
||||
// else
|
||||
// let msk = rhs - 1 in
|
||||
// if rhs & msk == 0 then
|
||||
// lhs & msk
|
||||
// else
|
||||
|
Loading…
Reference in New Issue
Block a user