Closed Bug 1815790 Opened 2 years ago Closed 2 years ago

Replace intgemm with a xsimd alternative

Categories

(Firefox :: Translation, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
113 Branch
Tracking Status
firefox113 --- fixed

People

(Reporter: marco, Assigned: sergesanspaille)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

An alternative solution based on xsimd would be more maintenable than intgemm.

This would also allow us to avoid integrating armgemm.

The severity field is not set for this bug.
:anatal, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(anatal)
Type: defect → enhancement
Flags: needinfo?(anatal)
Priority: -- → P3

gemmology is a rewrite of intgemm based on xsimd (that we already
vendor), with a focus on the API we actually use.

It also supports sse2 and has a decent implementation for arm32 and
aarch64.

(In reply to [:sergesanspaille] from comment #2)

Created attachment 9320348 [details]
WIP: Bug 1815790 - Replace intgemm by gemmology

gemmology is a rewrite of intgemm based on xsimd (that we already
vendor), with a focus on the API we actually use.

It also supports sse2 and has a decent implementation for arm32 and
aarch64.

That's awesome.

Summary: Evaluate replacing intgemm with a xsimd alternative → Replace intgemm with a xsimd alternative
Blocks: 1820240
Assignee: nobody → sguelton
Attachment #9320348 - Attachment description: WIP: Bug 1815790 - Replace intgemm by gemmology → Bug 1815790 - Replace intgemm by gemmology r=yury
Status: NEW → ASSIGNED
Blocks: 1755325
Attachment #9320348 - Attachment description: Bug 1815790 - Replace intgemm by gemmology r=yury → Bug 1815790 - Replace intgemm by gemmology r=yury!
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 113 Branch
Flags: needinfo?(sguelton)
Regressions: 1825285
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: