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)
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.)
Reporter | ||
Updated•4 years ago
|
Assignee: nobody → ydelendik
Status: NEW → ASSIGNED
Reporter | ||
Updated•4 years ago
|
Priority: P2 → P1
Assignee | ||
Comment 1•4 years ago
|
||
Pushed by ydelendik@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/df62bf6e4981
Better SIMD code for iNxM.compare_u. r=lth
Comment 3•4 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox89:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•