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

Create Tutorials “introduction-to-cilk_for” #141

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

Conversation

behoppe
Copy link
Member

@behoppe behoppe commented Aug 15, 2022

Automatically generated by Netlify CMS

@netlify
Copy link

netlify bot commented Aug 15, 2022

Deploy Preview for sage-licorice-6da44d ready!

Name Link
🔨 Latest commit f5353af
🔍 Latest deploy log https://app.netlify.com/sites/sage-licorice-6da44d/deploys/62fc0aca65c685000a3b8835
😎 Deploy Preview https://deploy-preview-141--sage-licorice-6da44d.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@behoppe behoppe requested review from ailiop and removed request for ailiop August 15, 2022 22:25
@behoppe behoppe added this to the Version 1.0 milestone Aug 15, 2022
@behoppe
Copy link
Member Author

behoppe commented Aug 15, 2022

@ailiop I created this PR to hold cilk_for work in progress. Please let me know if you're already working on a cilk_for page, or have ideas about how to pull this together. I am thinking I'll bring together an introduction to cilk_for based on existing materials, and then we can decide how to integrate with other parts of the website.

1 similar comment
@behoppe
Copy link
Member Author

behoppe commented Aug 15, 2022

@ailiop I created this PR to hold cilk_for work in progress. Please let me know if you're already working on a cilk_for page, or have ideas about how to pull this together. I am thinking I'll bring together an introduction to cilk_for based on existing materials, and then we can decide how to integrate with other parts of the website.

@behoppe
Copy link
Member Author

behoppe commented Aug 16, 2022

@dcurtisatmit, based on feedback last week from Charles, have started an introduction to cilk_for. It draws quite a bit of material from the Cilk Plus Programmer's Guide. Have you been updating that part of the Guide (pp 39-44)? Please let me know, and I would love to help merge your edits with this introduction to cilk_for. Thank you.

@ailiop
Copy link
Contributor

ailiop commented Aug 17, 2022

@behoppe Thank you for starting this! I will work on testing and editing the current page.

@wheatman
Copy link
Contributor

The supported example under "The loop control variable must not be modified in the loop body." currently has a race.

unsigned int j;
cilk_for (unsigned int i = 1; i < 16; ++i) j = f();

This is probably fine since it is showing what the compiler supports, but maybe this should be pointed out since its not a great example to follow.

@behoppe
Copy link
Member Author

behoppe commented Feb 7, 2024

@neboat I wonder if you recommend assigning anyone to this issue as a way to kick-start our documentation of cilk_for?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants