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

Fix compilation error when _CLK_BREAKDOWN is defined in cagra. #202

Merged
merged 2 commits into from
Jul 2, 2024

Conversation

jiangyinzuo
Copy link
Contributor

Port from rapidsai/raft#2350

@jiangyinzuo jiangyinzuo requested a review from a team as a code owner June 27, 2024 06:03
Copy link

copy-pr-bot bot commented Jun 27, 2024

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@github-actions github-actions bot added the cpp label Jun 27, 2024
rapids-bot bot pushed a commit to rapidsai/raft that referenced this pull request Jun 27, 2024
PR #1740 forgot to rename `BLOCK_SIZE` in `#ifdef _CLK_BREAKDOWN` blocks.

The use of `RAFT_LOG_DEBUG` in kernel function results in compilation
errors, replace it with `printf`.

Also remove an unused function in search_single_cta_kernel-inl.cuh

After merging:

- [x] port to cuVS rapidsai/cuvs#202

Authors:
  - Yinzuo Jiang (https://github.com/jiangyinzuo)
  - Tamas Bela Feher (https://github.com/tfeher)
  - Corey J. Nolet (https://github.com/cjnolet)

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

URL: #2350
@cjnolet cjnolet added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Jun 27, 2024
@cjnolet
Copy link
Member

cjnolet commented Jun 27, 2024

/ok to test

@cjnolet
Copy link
Member

cjnolet commented Jul 2, 2024

/merge

@rapids-bot rapids-bot bot merged commit 8dfef2b into rapidsai:branch-24.08 Jul 2, 2024
55 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cpp improvement Improves an existing functionality non-breaking Introduces a non-breaking change
Projects
Development

Successfully merging this pull request may close these issues.

2 participants