Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

add initialization for d_a#1622

Closed
ChengChen002 wants to merge 1 commit intooneapi-src:masteroneapi-src/oneAPI-samples:masterfrom
ChengChen002:masterCopy head branch name to clipboard
Closed

add initialization for d_a#1622
ChengChen002 wants to merge 1 commit intooneapi-src:masteroneapi-src/oneAPI-samples:masterfrom
ChengChen002:masterCopy head branch name to clipboard

Conversation

@ChengChen002
Copy link
Contributor

@ChengChen002 ChengChen002 commented May 5, 2023

The variables "d_a" are not initialized, so the result during reduction is uncertain and may overflow to a negative number, causing the test to fail.

Copy link

@wanyaman wanyaman left a comment

Choose a reason for hiding this comment

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

LGTM

@ChengChen002
Copy link
Contributor Author

@ManjulaChalla Could you help to review and merge this PR when free, thanks.

@ChengChen002 ChengChen002 marked this pull request as draft May 9, 2023 01:21
@ChengChen002 ChengChen002 marked this pull request as ready for review May 9, 2023 01:21
Copy link
Contributor

@IgorOchocki IgorOchocki left a comment

Choose a reason for hiding this comment

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

Is there any reason this is fix is supposed to be merged into the master branch instead of development?

@ChengChen002
Copy link
Contributor Author

Is there any reason this is fix is supposed to be merged into the master branch instead of development?

Sorry, I'm not familiar with the commit rules for this repository, should I submit a PR to the development branch?

@ChengChen002 ChengChen002 closed this by deleting the head repository May 10, 2023
@ChengChen002
Copy link
Contributor Author

Changed branch which I want to merge the PR into from master to development, see the PR. 

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.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.