Add initial support for Darwin arm64#34238
Merged
Uh oh!
There was an error while loading. Please reload this page.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
nodejs-github-bot added build 
richardlau added arm 
evanlucas commented Jul 7, 2020
Uh oh!
There was an error while loading. Please reload this page.
evanlucasforce-pushed the arm64 branch 2 times, most recently from 4169c57 to 3e476e2Compare
addaleax approved these changes Jul 7, 2020
ContributorAuthor
evanlucas commented Jul 21, 2020
We found a way to get os.cpus() working properly on both ARM and intel Macs. Once libuv/libuv#2914 lands, I'll get it pulled in. With some V8 patches, I was able to get wasi working properly. The inspector still has some issues that I am digging in to. Perhaps I should back out the libuv patch and we go ahead and land this and do it more incrementally? Thoughts? |
ContributorAuthor
evanlucas commented Aug 4, 2020
The bump to V8 8.5 (#34337) should fix the inspector and wasi issues. |
evanlucasforce-pushed the arm64 branch 2 times, most recently from a5e0038 to ad29cb6Compare
Collaborator
nodejs-github-bot commented Aug 25, 2020
cjihrig approved these changes Aug 25, 2020
Member
AshCripps commented Aug 25, 2020
Also started a build on our DTKs as they are not part of the main runs yet - https://ci.nodejs.org/job/node-test-commit-osx-arm/4/ |
AshCripps approved these changes Aug 25, 2020
ContributorAuthor
evanlucas commented Aug 25, 2020
@AshCripps thanks! The inspector and wasi tests are still going to fail until V8 8.5 is landed. I have a patch somewhere that works for those without the V8 upgrade. I'll try to find that and submit another PR after this one lands |
This adds the required files for supporting openssl on arm64. This uses the same configuration that can be found in openssl/openssl#12369 to generate the required files. PR-URL: nodejs#34238 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Ash Cripps <[email protected]>
Ref: nodejs/TSC#886 Ref: nodejs#34043 PR-URL: nodejs#34238 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Ash Cripps <[email protected]>
ContributorAuthor
evanlucas commented Aug 25, 2020
Landed in 4b6036a...e3b79e3 |
richardlau pushed a commit that referenced this pull request Sep 1, 2020
This adds the required files for supporting openssl on arm64. This uses the same configuration that can be found in openssl/openssl#12369 to generate the required files. PR-URL: #34238 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Ash Cripps <[email protected]>
richardlau pushed a commit that referenced this pull request Sep 1, 2020
Ref: nodejs/TSC#886 Ref: #34043 PR-URL: #34238 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Ash Cripps <[email protected]>
5 tasks
addaleax pushed a commit that referenced this pull request Sep 22, 2020
This adds the required files for supporting openssl on arm64. This uses the same configuration that can be found in openssl/openssl#12369 to generate the required files. PR-URL: #34238 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Ash Cripps <[email protected]>
addaleax pushed a commit that referenced this pull request Sep 22, 2020
Ref: nodejs/TSC#886 Ref: #34043 PR-URL: #34238 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Ash Cripps <[email protected]>
addaleax pushed a commit that referenced this pull request Sep 22, 2020
This adds the required files for supporting openssl on arm64. This uses the same configuration that can be found in openssl/openssl#12369 to generate the required files. PR-URL: #34238 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Ash Cripps <[email protected]>
addaleax pushed a commit that referenced this pull request Sep 22, 2020
Ref: nodejs/TSC#886 Ref: #34043 PR-URL: #34238 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Ash Cripps <[email protected]>
Merged
5 tasks
7 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
build: add support for build on arm64
deps: add openssl support for arm64
This adds the required files for supporting openssl
on arm64.
This uses the same configuration that can be found in
openssl/openssl#12369 to generate the required
files.
TODO
ld: warning: -no_pie ignored for arm64message during buildChecklist
make -j4 test(UNIX), orvcbuild test(Windows) passes