NVIDIA / NVIDIA/raft

Rename headers and function qualifiers to enable CPU-only compilation

Open
#1,663 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

feature request
Dominant language
Cuda
Stars
1k
Forks
251
Avg merge
1d 8h
Merged PRs (30d)
31

Description

cpp/include/raft/util/pow2_utils.cuh can be renamed to cpp/include/raft/util/pow2_utils.hpp and the function qualifiers within this file can be changed to _RAFT_HOST_DEVICE, which will enable this to be used in CPU-only compilation. There are more examples where this change can be made.
cpp/include/raft/util/fast_int_div.cuh is another such example.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with cpp/include/raft/util/pow2_utils.cuh and cpp/include/raft/util/fast_int_div.cuh, comparing their headers and function qualifiers. Identify the additional examples covered by the issue, then verify that the renamed headers and qualifier changes allow CPU-only compilation without breaking existing CUDA use.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
build-system, compilers
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.