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

Dev222 #2

Closed
wants to merge 12 commits into from
Closed

Dev222 #2

wants to merge 12 commits into from

Conversation

suvorovrain
Copy link

No pull request can be accepted unless you first sign the Contributor License Agreement (see the PDF agreement in the Contributor_License folder). Sign the PDF and email it to me (digital signature OK). Submit all PRs to the dev2 branch only.

DrTimothyAldenDavis and others added 12 commits December 19, 2024 19:17
The complexity of the build matrix for the Ubuntu runners grew to a level
which made it very difficult to understand which set of runners will be
started with which configuration.

Overhaul the build matrix settings by moving most of its complexity to the
build steps.
That should make it easier to potentially adjust the settings for one of
the runners in the matrix in the future.
It looks like the stack is smashed at some point in GraphBLAS when using
Clang as the compiler on Ubuntu with optimization level `-O3`.

It's currently unclear to me why that happens. But changing the build
type to "Debug" (i.e., optimization level `-O0`) works around that issue.
Building CHOLMOD_CUDA requires OpenMP. Instead of failing later on
during compilation, emit an error during configuration that point to the
underlying issue.
CI: Overhaul build matrix for Ubuntu runners and work around issue with Clang
vkutuev pushed a commit to vkutuev/SuiteSparse that referenced this pull request Apr 2, 2025
Так как файл был скомпилирован под таргет систему, то и загрузчик там
вшит типичный для таргет системы, но нам то нужно сослаться на свой
загрузчик. Поэтому, мы попробуем обмануть elf и подсунуть ему свой
загрузчик под именем загрузчика таргет ситсемы
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants