Closed Bug 1690478 Opened 4 years ago Closed 4 years ago

SIMD optimization x64/x86: Better code for iNxM.compare_u

Categories

(Core :: JavaScript: WebAssembly, defect, P1)

x86_64
All
defect

Tracking

()

RESOLVED FIXED
89 Branch
Tracking Status
firefox89 --- fixed

People

(Reporter: lth, Assigned: yury)

References

(Blocks 2 open bugs)

Details

Attachments

(1 file)

The code we generate for unsigned integer comparisons is truly embarrassing. I don't remember if this came from SIMD.js or it is something I put together in my first heady days of SIMD programming but it looks like we can do much better, see https://github.com/WebAssembly/simd/issues/188.

(The code is so bad that I'm giving this a P2 and marking it as a defect, we might try to fix this before we ship SIMD.)

Assignee: nobody → ydelendik
Status: NEW → ASSIGNED
Blocks: 1695585
Priority: P2 → P1
Pushed by ydelendik@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/df62bf6e4981 Better SIMD code for iNxM.compare_u. r=lth
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: