-
Notifications
You must be signed in to change notification settings - Fork 245
Forgot to add inline
in is_address_from
#5349
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
Forgot to add inline
in is_address_from
#5349
Conversation
I have one more idea - is it too late to chenge the order of the arguments? I was strongly for this order before, but I somehow don't like it anymore, I think it would make more sense to have the |
this is an edge case. In general, this is considered a breaking change and not allowed. The functionality has been added in 3.0, but we documented it very recently. https://nvidia.github.io/cccl/libcudacxx/extended_api/memory/is_address_from.html |
I know. It was in 3.1 actually, so I am not sure whether it's too late for the change |
🟩 CI finished in 1h 08m: Pass: 100%/209 | Total: 1d 12h | Avg: 10m 24s | Max: 34m 30s | Hits: 97%/319261
|
Project | |
---|---|
CCCL Infrastructure | |
CCCL Packaging | |
+/- | libcu++ |
CUB | |
Thrust | |
CUDA Experimental | |
stdpar | |
python | |
CCCL C Parallel Library | |
Catch2Helper |
Modifications in project or dependencies?
Project | |
---|---|
CCCL Infrastructure | |
+/- | CCCL Packaging |
+/- | libcu++ |
+/- | CUB |
+/- | Thrust |
+/- | CUDA Experimental |
+/- | stdpar |
+/- | python |
+/- | CCCL C Parallel Library |
+/- | Catch2Helper |
🏃 Runner counts (total jobs: 209)
# | Runner |
---|---|
128 | linux-amd64-cpu16 |
23 | windows-amd64-cpu16 |
16 | linux-amd64-gpu-l4-latest-1 |
12 | linux-arm64-cpu16 |
11 | linux-amd64-gpu-rtx2080-latest-1 |
10 | linux-amd64-gpu-h100-latest-1 |
6 | linux-amd64-gpu-rtxa6000-latest-1 |
3 | linux-amd64-gpu-rtx4090-latest-1 |
I think backporting to 3.1 is still fine, but let's also ask the grand integrator @wmaxey. Regarding the order of arguments, I don't have a strong opinion. |
I'm in favor of the change proposed by @davebayer. |
And I've just found another problem in the docs: template <typename T> // < remove this
[[nodiscard]] __device__ inline
bool is_address_from(address_space space, const void* ptr) Can you fix it, please? |
sure! |
actually, would be also nice to also have cuda::device::is_address_from(global_var, address_space::global); // no &global_var What do you think? |
I don't think the function name describes this overload very well. I think it the user should just do I would expect to have something like |
Actually that makes no sense, because the spaces can overlap. Okay, so what about is_object_from? |
Backport failed for Please cherry-pick the changes locally and resolve any conflicts. git fetch origin branch/3.1.x
git worktree add -d .worktree/backport-5349-to-branch/3.1.x origin/branch/3.1.x
cd .worktree/backport-5349-to-branch/3.1.x
git switch --create backport-5349-to-branch/3.1.x
git cherry-pick -x 5dbfd89ccc2e5d6b7f95889b5daa792225d5b26f |
|
No description provided.