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

[openmp] using h/w clock tick to measure time instead of gettimeofday… #85474

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

devnexen
Copy link
Member

@devnexen devnexen commented Mar 15, 2024

… on Linux/arm64.

using counter timer frequency register for that purpose, available in user mode.

@llvmbot llvmbot added the openmp:libomp OpenMP host runtime label Mar 15, 2024
@brad0 brad0 requested a review from shiltian March 16, 2024 07:00
@devnexen devnexen force-pushed the omp_arm64_upd branch 2 times, most recently from e2932d2 to 5895bea Compare March 16, 2024 23:35
@brad0
Copy link
Contributor

brad0 commented Mar 17, 2024

Out of curiosity, where was this tested? and why was it disabled on macOS?

@devnexen
Copy link
Member Author

devnexen commented Mar 17, 2024

I ll be giving another round of tests on Linux after last changes. on macOs two tests no longer passed.

@brad0
Copy link
Contributor

brad0 commented Mar 17, 2024

I ll be giving another round of tests on Linux after last changes. on macOs two tests no longer passed.

What were the tests? details?

@devnexen
Copy link
Member Author

one of them was omp_lock so directly related to the changes. anyhow putting it back to draft for now.

@devnexen devnexen marked this pull request as draft March 17, 2024 08:05
@devnexen devnexen force-pushed the omp_arm64_upd branch 2 times, most recently from c4b7c57 to afd4d05 Compare March 23, 2024 20:02
@devnexen devnexen marked this pull request as ready for review March 23, 2024 20:03
Copy link

✅ With the latest revision this PR passed the Python code formatter.

Copy link

✅ With the latest revision this PR passed the C/C++ code formatter.

… on Linux/arm64.

using counter timer frequency register for that purpose, available in user mode.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
openmp:libomp OpenMP host runtime
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants