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

[BUG]: Shadow warning in thrust #335

Closed
1 task done
hageboeck opened this issue Aug 15, 2023 · 1 comment · Fixed by #334
Closed
1 task done

[BUG]: Shadow warning in thrust #335

hageboeck opened this issue Aug 15, 2023 · 1 comment · Fixed by #334
Assignees
Labels
bug Something isn't working right. thrust For all items related to Thrust.

Comments

@hageboeck
Copy link
Contributor

Is this a duplicate?

Type of Bug

Something else

Component

Thrust

Describe the bug

Using cuda12, I observe a shadow warning in thrust.

The relevant code is here:
https://github.com/NVIDIA/thrust/blob/24486a169a62a58ef8f824d3dc9613c006b6f5a7/thrust/detail/execute_with_dependencies.h#L60-L64

Line 60 is shadowed by line 64. I guess a simple renaming would help.

How to Reproduce

Compile thrust with all warnings enabled.

Expected behavior

No warnings of course

Reproduction link

No response

Operating System

Alma8

nvidia-smi output

No response

NVCC version

nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2023 NVIDIA Corporation
Built on Fri_Jan__6_16:45:21_PST_2023
Cuda compilation tools, release 12.0, V12.0.140
Build cuda_12.0.r12.0/compiler.32267302_0

@github-actions github-actions bot added the needs triage Issues that require the team's attention label Aug 15, 2023
@github-actions
Copy link
Contributor

Hi @hageboeck!

Thanks for submitting this issue - the CCCL team has been notified and we'll get back to you as soon as we can!
In the mean time, feel free to add any relevant information to this issue.

@miscco miscco added thrust For all items related to Thrust. bug Something isn't working right. and removed needs triage Issues that require the team's attention labels Aug 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working right. thrust For all items related to Thrust.
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants