Skip to content

Adding the examples of the target nowait article#2663

Merged
jimmytwei merged 1 commit intooneapi-src:developmentfrom
jadhbeika:development
May 28, 2025
Merged

Adding the examples of the target nowait article#2663
jimmytwei merged 1 commit intooneapi-src:developmentfrom
jadhbeika:development

Conversation

@jadhbeika
Copy link

Existing Sample Changes

Description

This PR both deletes old samples and Adds New Samples. The samples are related to the "nowait on target" article which was re-written for the following 2 reasons.
the previous article did not explain/investigate deeply the benefit of nowait and when it is beneficial to use it. In this version we design a benchmark and we show the benefit of nowait by comparing 3 sequential versions (cpu-only. gpu_only and hybrid_blocking) to an async verison using nowait (hybrid_non_blocking)

OpenMP 6.0 updated the syntax of Nowait which is now allowed to have an optional argument. the optional nowait argument allows runtime control over synchronization
The new specs can be found on page 481 of OpenMP-API-Specification-6-0

Type of change

  • [ X] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  • [x ] Command Line

Checklist

Administrative

  • Review sample design with the appropriate Domain Expert:
  • If you have any new dependencies/binaries, inform the oneAPI Code Samples Project Manager

Code Development

Security and Legal

  • OSPDT Approval (see Project Manager for assistance)
  • Compile using the following compiler flags and fix any warnings, the falgs are: "/Wall -Wformat-security -Werror=format-security"
  • Bandit Scans (Python only)
  • Virus scan

Review

  • Review DPC++ code with Paul Peterseon. (GitHub User: pmpeter1)
  • Review readme with Tom Lenth(@tomlenth) and/or Project Manager
  • Tested using Dev Cloud when applicable

@jimmytwei jimmytwei merged commit 8aa75ad into oneapi-src:development May 28, 2025
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.

2 participants