-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[package] cryptopp/8.9.0: Building with MSVC for ARM attempts to compile the intel only file rdrand.asm #26205
Comments
Hi @alanb-sony thanks a lot for taking the time to report this issue. I can confirm that I'm able to reproduce this locally, but unfortunately, even after patching it, I face a different error:
check_compile_link_option(
"-DCRYPTOPP_ARM_ACLE_HEADER=1 -march=armv8-a" CRYPTOPP_ARM_ACLE_HEADER
"${TEST_PROG_DIR}/test_arm_acle_header.cpp"
)
if(CRYPTOPP_ARM_ACLE_HEADER)
list(APPEND CRYPTOPP_COMPILE_DEFINITIONS "CRYPTOPP_ARM_ACLE_HEADER=1")
endif()
#if !defined(CRYPTOPP_ARM_ACLE_HEADER)
# if defined(__aarch32__) || defined(__aarch64__) || (__ARM_ARCH >= 8) || defined(__ARM_ACLE)
# define CRYPTOPP_ARM_ACLE_HEADER 1
# endif
#endif which, by having a totally different logic, means
Is this something you also see on your end? Nevertheless, if I force this to Thanks again for taking the time to report the issue! |
ah, yes, I forgot to raise that one with them, I'll do that now. My fix was to define |
The fix for the ACLE header has been committed uptream abdes/cryptopp-cmake@00a151f |
Added to #26208, thanks! |
Description
Building for crypto++ for MSVC ARM64 doesn't work when building with ninja without this fix abdes/cryptopp-cmake@1940cc0
Package and Environment Details
Conan profile
Profile host:
[settings]
arch=armv8
build_type=Debug
compiler=msvc
compiler.cppstd=14
compiler.runtime=dynamic
compiler.runtime_type=Debug
compiler.version=194
os=Windows
[conf]
tools.cmake.cmaketoolchain:generator=Ninja
tools.microsoft.msbuild:vs_version=17
Profile build:
[settings]
arch=x86_64
build_type=Release
compiler=msvc
compiler.cppstd=14
compiler.runtime=dynamic
compiler.runtime_type=Release
compiler.version=194
os=Windows
Steps to reproduce
conan install -r conancenter --requires cryptopp/8.9.0 --settings=arch=armv8 --build=missing --conf=tools.cmake.cmaketoolchain:generator=Ninja
Logs
Click to expand log
The text was updated successfully, but these errors were encountered: