Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

neondot on armv7-32bit breaks the build #981

Closed
honglh opened this issue Aug 19, 2020 · 1 comment
Closed

neondot on armv7-32bit breaks the build #981

honglh opened this issue Aug 19, 2020 · 1 comment

Comments

@honglh
Copy link

honglh commented Aug 19, 2020

Hi, a recent change breaks the build for ARMv7-32 bit (e.g. RPI2, or RPI3-32bit). This is what I set for cmake

set(CMAKE_SYSTEM_PROCESSOR armv7)
FAILED: ...  -c ../../src/qs8-gemm/gen/8x8c4-minmax-neondot.c
arm-linux-gnueabihf-gcc: error: unrecognized argument in option '-march=armv8.2-a+dotprod'

I think the XNNPACK_NEONDOT_MICROKERNEL_SRCS should be limited to if user specify dotprod modifier in the CMAKE_SYSTEM_PROCESSOR E.g.

set(CMAKE_SYSTEM_PROCESSOR armv8.2-a+dotprod)
@Maratyszcza
Copy link
Contributor

Fixed in 7359463

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants