-
Notifications
You must be signed in to change notification settings - Fork 339
OpenSSL: Consider asm build on ARM #22
Comments
I would like to know if there is someone who has ever built OpenSSL-1.0.2 on the current Windows ARM. Another issue is that the current no-asm configuration of openssl in node is made based on Linux. We have to check if there are some additional sources or defines needed to work on Windows. |
We never built it on Windows ARM. We use Visual Studio to build and there is no Visual Studio on ARM. Instead we cross-build ARM on x86/x64 hosts.
Do you have specific concerns for this? Are the functionalities covered by unit tests? We have been using the current no-asm configuration for Windows ARM and it builds/runs. |
@shigeki Could you please elaborate on the original |
@jianchun Yes, WIN_CE is very old platform and OpenSSL has not been maintaining it for a long time. I just commented about WIN_CE because https://github.com/nodejs/node-chakracore/blob/chakracore-master/deps/openssl/openssl/INSTALL.WCE indicates how to build openssl on Windows ARM and I'd like to know if it still work on the current Windows ARM with generating asm files. The current Node-Chakra is built on openssl with no asm support so that high performance of tls and crypto module is not expected.
The current Node only use a part of functionalities of openssl and tests does not cover all inputs parameters so there is no surprise to pass all unit tests. For define parameters, you can see the list of https://github.com/nodejs/node-chakracore/blob/chakracore-master/deps/openssl/doc/openssl_define_list.pdf and found that there are some differences between Linux and Win. For sources of no-asm build, it is defined in https://github.com/nodejs/node-chakracore/blob/chakracore-master/deps/openssl/openssl.gypi#L638-L653. I made its list from Linux build but I do not know if they are the same ones as Windows build with no-asm. |
Windows CE uses ARM, but WoA uses Thumb mode |
An cross-compiled Clang runs relatively well, and works fine(even with MSBuild) |
Windows CE support is not on our roadmap. |
As commented by @shigeki in nodejs/node#4765 , we should consider adding new config for WIN_CE.
The text was updated successfully, but these errors were encountered: