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

feat(tree): add static_assert shim macro #556

Merged
merged 1 commit into from
Sep 9, 2024
Merged

Conversation

include/nccl_ofi.h Show resolved Hide resolved
@aws-nslick aws-nslick changed the base branch from aws-nslick/stack/2 to master September 4, 2024 19:07
@aws-nslick aws-nslick requested a review from rauteric as a code owner September 4, 2024 19:07
@aws-nslick aws-nslick changed the base branch from master to aws-nslick/stack/2 September 4, 2024 19:07
@aws-nslick aws-nslick changed the base branch from aws-nslick/stack/2 to master September 4, 2024 19:16
@aws-nslick aws-nslick changed the base branch from master to aws-nslick/stack/2 September 4, 2024 19:17
@aws-nslick aws-nslick changed the base branch from aws-nslick/stack/2 to master September 4, 2024 20:25
@aws-nslick aws-nslick changed the title tree: add static_assert shim macro feat(tree): add static_assert shim macro Sep 4, 2024
@aws-nslick aws-nslick changed the base branch from master to aws-nslick/stack/2 September 4, 2024 20:25
@aws-nslick aws-nslick changed the base branch from master to aws-nslick/stack/2 September 5, 2024 01:56
@aws-nslick aws-nslick changed the base branch from aws-nslick/stack/2 to master September 5, 2024 02:02
@aws-nslick aws-nslick changed the base branch from master to aws-nslick/stack/2 September 5, 2024 02:03
@aws-nslick aws-nslick changed the base branch from aws-nslick/stack/2 to master September 5, 2024 02:07
@aws-nslick aws-nslick changed the base branch from master to aws-nslick/stack/2 September 5, 2024 02:08
@aws-nslick aws-nslick changed the base branch from aws-nslick/stack/2 to master September 5, 2024 02:22
@aws-nslick aws-nslick requested a review from rauteric September 5, 2024 17:29
rauteric
rauteric previously approved these changes Sep 5, 2024
@a-szegel
Copy link
Contributor

a-szegel commented Sep 5, 2024

bot:aws:retest

@aws-nslick aws-nslick changed the base branch from master to aws-nslick/stack/2 September 6, 2024 20:11
@aws-nslick aws-nslick changed the base branch from aws-nslick/stack/2 to master September 6, 2024 20:31
@aws-nslick aws-nslick dismissed rauteric’s stale review September 6, 2024 20:31

The base branch was changed.

@aws-nslick aws-nslick changed the base branch from master to aws-nslick/stack/2 September 6, 2024 20:32
rauteric
rauteric previously approved these changes Sep 6, 2024
@aws-nslick
Copy link
Contributor Author

bot:aws:retest

static_assert is a c++ keyword, so when not using c++, create a macro
under the same name that aliases _Static_assert to static_assert. This
means static_assert can be used in both cases.

stack-info: PR: #556, branch: aws-nslick/stack/3
Signed-off-by: Nicholas Sielicki <[email protected]>
@aws-nslick aws-nslick changed the base branch from aws-nslick/stack/2 to master September 8, 2024 18:59
@aws-nslick aws-nslick dismissed rauteric’s stale review September 8, 2024 18:59

The base branch was changed.

@aws-nslick aws-nslick requested a review from rauteric September 8, 2024 20:21
@aws-nslick
Copy link
Contributor Author

bot:aws:retest

@aws-nslick aws-nslick merged commit fdebc77 into master Sep 9, 2024
62 of 63 checks passed
@aws-nslick
Copy link
Contributor Author

Commit previously passed without modifications.

@aws-nslick aws-nslick deleted the aws-nslick/stack/3 branch September 27, 2024 00:56
aws-ofiwg-bot pushed a commit to aws-ofiwg-bot/aws-ofi-nccl that referenced this pull request Oct 4, 2024
static_assert is a c++ keyword, so when not using c++, create a macro
under the same name that aliases _Static_assert to static_assert. This
means static_assert can be used in both cases.
Signed-off-by: Nicholas Sielicki <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants