Skip to content

Allow compilation when OpenMP is disabled#1346

Merged
rapids-bot[bot] merged 5 commits intorapidsai:branch-25.12from
robertmaynard:bug/allow_compilation_with_openmp_disabled
Oct 1, 2025
Merged

Allow compilation when OpenMP is disabled#1346
rapids-bot[bot] merged 5 commits intorapidsai:branch-25.12from
robertmaynard:bug/allow_compilation_with_openmp_disabled

Conversation

@robertmaynard
Copy link
Copy Markdown
Contributor

Since we compile CUVS with warnings as errors, we need to disable the unknown pragma warning generated by our OpenMP pragmas so we can still compile.

Since we compile CUVS with warnings as errors, we need
to disable the unknown pragma warning generated by our
OpenMP pragmas so we can still compile.
@robertmaynard robertmaynard requested review from a team as code owners September 19, 2025 15:16
@robertmaynard robertmaynard added the bug Something isn't working label Sep 19, 2025
@robertmaynard robertmaynard added the non-breaking Introduces a non-breaking change label Sep 19, 2025
Copy link
Copy Markdown
Contributor

@tfeher tfeher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @robertmaynard, LGTM.

@robertmaynard robertmaynard changed the base branch from branch-25.10 to branch-25.12 September 25, 2025 13:31
@robertmaynard
Copy link
Copy Markdown
Contributor Author

/merge

@rapids-bot rapids-bot Bot merged commit bd94586 into rapidsai:branch-25.12 Oct 1, 2025
84 checks passed
@robertmaynard robertmaynard deleted the bug/allow_compilation_with_openmp_disabled branch October 1, 2025 18:46
rmaschal pushed a commit to rmaschal/cuvs that referenced this pull request Oct 6, 2025
Since we compile CUVS with warnings as errors, we need to disable the unknown pragma warning generated by our OpenMP pragmas so we can still compile.

Authors:
  - Robert Maynard (https://github.com/robertmaynard)
  - Corey J. Nolet (https://github.com/cjnolet)

Approvers:
  - Tamas Bela Feher (https://github.com/tfeher)

URL: rapidsai#1346
enp1s0 pushed a commit to enp1s0/cuvs that referenced this pull request Oct 22, 2025
Since we compile CUVS with warnings as errors, we need to disable the unknown pragma warning generated by our OpenMP pragmas so we can still compile.

Authors:
  - Robert Maynard (https://github.com/robertmaynard)
  - Corey J. Nolet (https://github.com/cjnolet)

Approvers:
  - Tamas Bela Feher (https://github.com/tfeher)

URL: rapidsai#1346
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Introduces a non-breaking change

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants