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

CMakeLists.txt broken on MSYS2/MINGW64/AMD64 (Windows) #196

Open
Xunie opened this issue Dec 8, 2020 · 3 comments
Open

CMakeLists.txt broken on MSYS2/MINGW64/AMD64 (Windows) #196

Xunie opened this issue Dec 8, 2020 · 3 comments

Comments

@Xunie
Copy link

Xunie commented Dec 8, 2020

Configuring NNPACK I get a fatal error:

CMake Error at CMakeLists.txt:55 (MESSAGE):
  Unrecognized CMAKE_SYSTEM_PROCESSOR = AMD64

Platform: MSYS2 MINGW64 on a 64-bit Windows machine.

NNPACK/CMakeLists.txt

Lines 49 to 56 in 57616b9

# ---[ Build flags
IF(NOT CMAKE_SYSTEM_PROCESSOR)
IF(NOT IOS)
MESSAGE(WARNING "CMAKE_SYSTEM_PROCESSOR is not defined, automatic configuration may choose suboptimal options")
ENDIF()
ELSEIF(NOT CMAKE_SYSTEM_PROCESSOR MATCHES "^(i686|x86_64|armv5te|armv7-a|armv7l|armv7|armv7s|aarch64|arm64|arm64e)$")
MESSAGE(FATAL_ERROR "Unrecognized CMAKE_SYSTEM_PROCESSOR = ${CMAKE_SYSTEM_PROCESSOR}")
ENDIF()

Cause:

This all happens because CMAKE_SYSTEM_PROCESSOR is really just CMAKE_HOST_SYSTEM_PROCESSOR.

On Windows, CMAKE_HOST_SYSTEM_PROCESSOR is set to the value of the environment variable PROCESSOR_ARCHITECTURE.

And PROCESSOR_ARCHITECTURE may be either x86, IA64 or AMD64 See here.

@Maratyszcza
Copy link
Owner

I don't think NNPACK works on Windows even if you fix the CMakeLists

@Maratyszcza
Copy link
Owner

Generally, I suggest you take a look at XNNPACK library, which is a successor to NNPACK, and supports Windows well.

@Xunie
Copy link
Author

Xunie commented Dec 16, 2020

I don't think NNPACK works on Windows even if you fix the CMakeLists

If that's the case, there needs to be a case that deals with that and prints out a MESSAGE(FATAL_ERROR ...). I'm totally new to NNPACK and was merely trying to use tiny-dnn...

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