Wasm baseline: Use conditional move to avoid branches for MinMaxI32, AbsI32 (investigate)

RESOLVED WONTFIX

Status

()

P5
normal
RESOLVED WONTFIX
2 years ago
2 years ago

People

(Reporter: lth, Unassigned, Mentored)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

2 years ago
What the summary says.  This needs to be backed by benchmarks on relevant platforms (plural).  Info on the net suggests that conditional move is not always a winner for performance.  It may be we can't discern a performance difference, in which case we should go for smaller code.
(Reporter)

Updated

2 years ago
Mentor: lhansen
(Reporter)

Updated

2 years ago
Priority: P3 → P5
(Reporter)

Comment 1

2 years ago
These operations are asm.js operations and are being removed from the baseline compiler, see bug 1333770.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.