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.

Sign up for GitHub

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

Jump to bottom

CI: Add clang test for loongarch64 #4642

Merged
merged 2 commits into from
Apr 23, 2024
Merged

CI: Add clang test for loongarch64 #4642

merged 2 commits into from
Apr 23, 2024

Conversation

XiWeiGu
Copy link
Contributor

@XiWeiGu XiWeiGu commented Apr 15, 2024

No description provided.

@XiWeiGu XiWeiGu force-pushed the loongarch64_clang branch from fcf94d0 to 875b7fe Compare April 15, 2024 07:54
@martin-frbg
Copy link
Collaborator

AFAICT, github-hosted runners do not provide Debian 10 (or any version of Debian), I guess you could try if runs-on: ubuntu-20.04 is close enough to support installing those packages

@XiWeiGu
Copy link
Contributor Author

XiWeiGu commented Apr 17, 2024

AFAICT, github-hosted runners do not provide Debian 10 (or any version of Debian), I guess you could try if runs-on: ubuntu-20.04 is close enough to support installing those packages

Thank you for the reminder, I will revise promptly.

@XiWeiGu XiWeiGu force-pushed the loongarch64_clang branch from 875b7fe to 1cdad09 Compare April 23, 2024 11:30
@XiWeiGu
Copy link
Contributor Author

XiWeiGu commented Apr 23, 2024

By the way, I also wanted to ask how to enable large page memory allocation. Currently, I haven't found any available options, so I had to manually modify the code to achieve it.

@martin-frbg
Copy link
Collaborator

There is an (almost) undocumented compile parameter named HUGETLBFILE_ALLOCATION that is to be set to the mount point of the hugetlbfs (for creation of a backing pseudofile named gotoblas.), or you can define HUGETLB_ALLOCATION=1 for accesses through shared memory instructions. Both are not used by default on any architecture, so there is a small risk that they may land you in code whose performance has not been tested since the early days of GotoBLAS

@martin-frbg martin-frbg added this to the 0.3.28 milestone Apr 23, 2024
@martin-frbg martin-frbg merged commit ae695d4 into OpenMathLib:develop Apr 23, 2024
70 of 74 checks passed
@martin-frbg
Copy link
Collaborator

(that said, a build with make HUGETLB_ALLOCATION=1 completes and tests without any issues on x86_64)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.3.28
Development

Successfully merging this pull request may close these issues.
2 participants
@XiWeiGu @martin-frbg