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.
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
Add initial CoreCLR compilation support for Apple Silicon #40435
Add initial CoreCLR compilation support for Apple Silicon #40435
Changes from all commits
1c1083e
83a45b8
af1acc4
37d29f8
8d21b27
e72e45c
657a57b
7457f37
f3dcdf3
96d4787
ba2f9ae
6cb8ceb
27ebc7a
02423a1
b80e9c0
bca94f7
0e7b3c8
06eef47
d13d6bc
0a827d5
7d202fa
69d14ed
8f678c0
d8935a7
061ecc4
3c1a830
cdd5da3
59e9888
7ffe655
e588304
c6658cd
a051fe7
7404605
7ca8f3e
82ff768
757ee92
c38cd7c
8aa1381
26ff8a3
2564ebd
325d184
8b4fd1f
e48b63b
1a87f7d
79ee527
9d2d5dd
0a89117
ab869de
45788c5
5440278
8015a0c
389c9cb
af4db13
49eb289
af0d7eb
6677bce
bf2ae4f
6a884d0
29edb16
1747d55
012febf
b06e8b3
b725312
923ecbb
ad5eabc
7eced9d
d94eba5
05214ef
03af208
24532ac
c291613
60ac6e5
6ba565a
7651eb5
59682d3
80b9126
62575e2
68a08bf
1025180
44eeb38
ef69659
50c931c
7e525b3
93d8866
5c725ce
0e61103
ef57726
e641d6a
9667e8c
c5cee94
bcdc032
260b5dd
271d396
78d00a2
fcf3424
e186f01
6817c64
f9b9fff
d3e49f0
0d4eba8
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just curious - what was wrong with
mach_absolute_time
?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It doesn't exist on aarch64 and Apple doc for mach_absolute_time recommends to replace it with clock_gettime_nsec_np
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See https://developer.apple.com/documentation/kernel/1462446-mach_absolute_time
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have noticed that
HAVE_MACH_ABSOLUTE_TIME
is still used under libraries. Do we need to fix it there as well?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, the strange thing is that libraries compile for arm64 with it just fine. So maybe the failure due to which I've modified the usage of mach_absolute_time to clock_gettime_nsec_np was something temporary in the middle of the fixes I was making. Anyways, the change is good since the mach_absolute_time is deprecated, so we should change it in the libraries too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, opened #43320