tesseract icon indicating copy to clipboard operation
tesseract copied to clipboard

Add framework for implementation of support for AVX512VNNI

Open stweil opened this issue 1 year ago • 9 comments

Using AVX512VNNI instructions is still missing.

stweil avatar Aug 06 '22 17:08 stweil

Ready for testing.

I suggest to backup the home directory and any other important stuff, in case this patch will trigger a hard crash (reboot).

amitdo avatar Aug 09 '22 12:08 amitdo

I suggest to backup the home directory and any other important stuff, in case this patch will trigger a hard crash (reboot).

Will it? I cannot test it before August 21 because I currently don't have access to the required hardware.

stweil avatar Aug 09 '22 12:08 stweil

Will it?

I don't think so...

amitdo avatar Aug 09 '22 12:08 amitdo

Stefan, will you able to test the code today or in the upcoming days?

amitdo avatar Aug 21 '22 15:08 amitdo

I just finished a first run of intsimdmatrix_test with the new code. It works, but I was surprised because the performance is identical to the AVX2 implementation.

stweil avatar Aug 21 '22 18:08 stweil

Support for sw build is still missing. Therefore CI currently fails.

stweil avatar Aug 21 '22 18:08 stweil

I was surprised because the performance is identical to the AVX2 implementation.

This is disappointing.

We can also remove the 'ones' vector. The vnni version does not need it.

amitdo avatar Aug 21 '22 18:08 amitdo

Did you fetch and merge my commit locally?

git switch avx512-vnni
git log -1

Do you see my commit?

amitdo avatar Aug 21 '22 19:08 amitdo

We can also remove the 'ones' vector. The vnni version does not need it.

I removed it locally. But that did not change the generated code. Obviously the compiler also removed it.

stweil avatar Aug 22 '22 07:08 stweil