Giters

wx257osn2 / qoixx

Single Header Quite Fast QOI(Quite OK Image Format) Implementation written in C++20

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Add support for 32bit arm

igor-microblink opened this issue · comments

commented

At the moment 32bit android arm is not supported, so the fallback without SIMD is required.

commented

@igor-microblink Does this change fix this? If so, I will merge this into master . (I believe that this is one of the solution, but I don't have 32bit ARM environment, so can't check it.)

commented

Thanks. Proposed change is fix for compilation error on armebi v7a.

__ARM_FEATURE_SVE is not defined for armeabi-v7a. Is it possible to add impl which utilizes arm neon for 32bit ARMs? AFAIK ARM NEON is supported from armeabi-v7.

commented

__ARM_FEATURE_SVE is not defined for armeabi-v7a

Yes, cause Arm SVE is an extension for armv8 and one of basic instructions in armv9.

Is it possible to add impl which utilizes arm neon for 32bit ARMs? AFAIK ARM NEON is supported from armeabi-v7.

Hmm, currently qoixx doesn't support armv7 as SIMD-available environment, because I don't have any development environment of armv7. It is possible technically cause NEON is an extension for armv7, but qoixx basically assumes 64-bit architecture (e.g. fast hash calculation using 64bit integer arithmetic; this might be not fast on 32bit environment), and currently I don't think I'd support it aggressively. Of course PR for supporting armv7 with NEON would be welcome, I will review it.