Skip to content

Conversation

@legrosbuffle
Copy link
Contributor

move_score_t is not trivially default-constructible, clang is unhappy with a shared array of move_score_t:

error: initialization is not supported for __shared__ variables

Use an aligned char array instead.

`move_score_t` is not trivially default-constructible, clang is unhappy
with a shared array of `move_score_t`:

`error: initialization is not supported for __shared__ variables`

Use an aligned char array instead.
@legrosbuffle legrosbuffle requested a review from a team as a code owner June 23, 2025 14:42
@legrosbuffle legrosbuffle requested review from Kh4ster and rg20 June 23, 2025 14:42
@copy-pr-bot
Copy link

copy-pr-bot bot commented Jun 23, 2025

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.

@rgsl888prabhu rgsl888prabhu added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Jun 24, 2025
@rgsl888prabhu rgsl888prabhu added this to the 25.08 milestone Jun 24, 2025
@rgsl888prabhu
Copy link
Collaborator

/ok to test e909725

@rgsl888prabhu
Copy link
Collaborator

/ok to test dd92f4b

@rgsl888prabhu
Copy link
Collaborator

Thank you @legrosbuffle for the PR

Copy link
Contributor

@aliceb-nv aliceb-nv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the PR :)

@aliceb-nv
Copy link
Contributor

/ok to test 94950b5

@aliceb-nv
Copy link
Contributor

/ok to test a0878e0

@aliceb-nv
Copy link
Contributor

/merge

@rapids-bot rapids-bot bot merged commit a4d26d8 into NVIDIA:branch-25.08 Jun 25, 2025
141 of 142 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants