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

macOS: Use clock_gettime_nsec_np() for omrtime_hires_clock() #6573

Merged
merged 1 commit into from
Jun 16, 2022

Conversation

knn-k
Copy link
Contributor

@knn-k knn-k commented Jun 15, 2022

This commit changes the implementation of omrtime_hires_clock() for
macOS.

Signed-off-by: KONNO Kazuhiro konno@jp.ibm.com

@knn-k
Copy link
Contributor Author

knn-k commented Jun 15, 2022

This is a revisit of #6568.
I also changed OMRTIME_HIRES_CLOCK_FREQUENCY to "nanoseconds / second" on macOS.

This commit changes the implementation of omrtime_hires_clock() for
macOS.

Signed-off-by: KONNO Kazuhiro <konno@jp.ibm.com>
@knn-k
Copy link
Contributor Author

knn-k commented Jun 16, 2022

I ran some OpenJ9 test sets with this PR on internal test servers. They finished successfully.

  • sanity.functional: job/Test_openjdk11_j9_sanity.functional_aarch64_mac_Personal/22/
  • sanity.openjdk: job/Grinder/25009

@0xdaryl
Copy link
Contributor

0xdaryl commented Jun 16, 2022

Jenkins build all

@0xdaryl 0xdaryl merged commit 6672d56 into eclipse-omr:master Jun 16, 2022
@knn-k knn-k deleted the macos_hires_clock branch June 16, 2022 02:18
knn-k added a commit to knn-k/openj9-omr that referenced this pull request Jun 23, 2022
This commit changes the implementation of omrtime_hires_clock() for
macOS.

Original PR: eclipse-omr/omr#6573

Signed-off-by: KONNO Kazuhiro <konno@jp.ibm.com>
@knn-k
Copy link
Contributor Author

knn-k commented Jun 22, 2023

Related PR for Linux: #7041

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants