Skip to content

Remove OpenMP dependencies from CMake#2443

Merged
rapids-bot[bot] merged 1 commit intorapidsai:branch-22.08from
seunghwak:enh_cmake_cleanup
Jul 26, 2022
Merged

Remove OpenMP dependencies from CMake#2443
rapids-bot[bot] merged 1 commit intorapidsai:branch-22.08from
seunghwak:enh_cmake_cleanup

Conversation

@seunghwak
Copy link
Contributor

Close #2442

We no longer use OpenMP in cuGraph. Remove code segments searching for OpenMP.

@seunghwak seunghwak added 3 - Ready for Review improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jul 25, 2022
@seunghwak seunghwak added this to the 22.08 milestone Jul 25, 2022
@seunghwak seunghwak requested a review from a team as a code owner July 25, 2022 20:02
@seunghwak seunghwak self-assigned this Jul 25, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jul 25, 2022

Codecov Report

❗ No coverage uploaded for pull request base (branch-22.08@d6b4cc5). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff               @@
##             branch-22.08    #2443   +/-   ##
===============================================
  Coverage                ?   61.30%           
===============================================
  Files                   ?      106           
  Lines                   ?     5398           
  Branches                ?        0           
===============================================
  Hits                    ?     3309           
  Misses                  ?     2089           
  Partials                ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d6b4cc5...48af18d. Read the comment docs.

@ChuckHastings
Copy link
Collaborator

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 2c50989 into rapidsai:branch-22.08 Jul 26, 2022
@seunghwak seunghwak deleted the enh_cmake_cleanup branch July 26, 2022 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove OpenMP dependencies from the cuGraph CMakeLists.txt files

3 participants