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.

Sign up for GitHub

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

Jump to bottom

Initial Execution Model documentation #3955

Open
wants to merge 61 commits into
base: main
Choose a base branch
from
Open

Initial Execution Model documentation #3955

wants to merge 61 commits into from
+296 −0

Conversation

gonzalobg
Copy link
Collaborator

@gonzalobg gonzalobg commented Feb 27, 2025 •
edited
Loading

Description

Initial documentation for the CUDA C++ Execution Model. We can expand this overtime but we need to start somewhere.

See review in #3873 .

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

gonzalobg and others added 18 commits February 21, 2025 12:06
@gonzalobg
Fix typo
0af7452
@gonzalobg
Fix typos
41f57bb
@gonzalobg
Clarify additions and modifications to C++ standard.
bd48ba2
@gonzalobg @mhoemmen
Fix typo
fb3b573 
Co-authored-by: Mark Hoemmen <[email protected]>
@gonzalobg @mhoemmen
Rephrase note
39fc26e 
Co-authored-by: Mark Hoemmen <[email protected]>
@gonzalobg @mhoemmen
Rephrase
3fbab9d 
Co-authored-by: Mark Hoemmen <[email protected]>
@pre-commit-ci
[pre-commit.ci] auto code formatting
ecae57d
@gonzalobg
Merge branch 'main' into execution_model
1f90b6c
@gonzalobg
Rephrase
bde9281
@gonzalobg
Rephrase
cb7906d
@gonzalobg
Merge branch 'main' into execution_model
b8e37a3
@pre-commit-ci
[pre-commit.ci] auto code formatting
ac61f7a
@gonzalobg
Fix typo
35fb6b7
@gonzalobg
Remove redundancy
7449b51
@gonzalobg
Fix bug
2334ebc 
The old wording may allow non steps to start executing (e.g. non atomic data accesses). Just saying that it does not start should suffice to prevent evaluating any expression.
@gonzalobg
Clarify semantics
5504adb
@gonzalobg
Merge branch 'main' into execution_model
9d4ac26
@gonzalobg
fixup
fe7ad79
@gonzalobg gonzalobg requested a review from a team as a code owner February 27, 2025 12:00
@gonzalobg gonzalobg requested a review from gonidelis February 27, 2025 12:00
@copy-pr-bot copy-pr-bot
Copy link

copy-pr-bot bot commented Feb 27, 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.

@gonzalobg gonzalobg changed the title Initial Execution Modeldocumentation Initial Execution Mode ldocumentation Feb 27, 2025
@gonzalobg
Copy link
Collaborator Author

@miscco can we run the test of this branch?
The last time it was merged without testing.

@gonzalobg
Copy link
Collaborator Author

pre-commit.ci autofix

@gonzalobg gonzalobg changed the title Initial Execution Mode ldocumentation Initial Execution Model documentation Feb 27, 2025
@elstehle
Copy link
Collaborator

/ok to test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gonidelis gonidelis Awaiting requested review from gonidelis gonidelis is a code owner automatically assigned from NVIDIA/cccl-codeowners

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
CCCL
Status: In Review
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gonzalobg @elstehle