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

RESOLVED WONTFIX

Status

()

defect
P5
normal
RESOLVED WONTFIX
3 years ago
3 years ago

People

(Reporter: lth, Unassigned, Mentored)

Tracking

(Blocks 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

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.
Mentor: lhansen
Priority: P3 → P5
These operations are asm.js operations and are being removed from the baseline compiler, see bug 1333770.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.