1815790 - Replace intgemm with a xsimd alternative

Status: RESOLVED FIXED

(Firefox :: Translations, enhancement, P3)

Description

[Marco Castelluccio [:marco]]

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

This would also allow us to avoid integrating armgemm.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

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

For more information, please visit auto_nag documentation.

Comment 2

[[:sergesanspaille]]

Attachment: Bug 1815790 - Replace intgemm by gemmology r=yury!

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.

Comment 3

[André Natal]

(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.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/19ca420febe8
Replace intgemm by gemmology r=yury

Comment 5

[Cosmin Sabou [:CosminS]]

Backed out for causing SM bustages on GemmologyEngineAVX2.cpp.

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&group_state=expanded&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=19ca420febe8d57f887b2cf932fffe22c47c5d9a&selectedTaskRun=P6d67PDYQRSuFyDqUN8qFA.0

Failure logs:

• https://treeherder.mozilla.org/logviewer?job_id=410400523&repo=autoland&lineNumber=100356
• GemmologyEngineAVX2.cpp
• FileToBugzillaMappingError: Missing Bugzilla component

Backout link: https://hg.mozilla.org/integration/autoland/rev/6b6e5626e9594a9c03529e0a1dd34edb5df61ca9

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a8fd4d476a47
Replace intgemm by gemmology r=yury

Comment 7

[Narcis Beleuzu [:NarcisB]]

Backed out for Linting opt failure on gemmology

Backout link: https://hg.mozilla.org/integration/autoland/rev/065fdd32bf3ec58c2755b0dc6d09293f97915c80
Log link: https://treeherder.mozilla.org/logviewer?job_id=410514299&repo=autoland&lineNumber=49

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6cc1c7fef689
Replace intgemm by gemmology r=yury

Comment 9

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/6cc1c7fef689