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

Fix "loop not vectorized" warning when running with newer icpx compiler #2818

Merged
merged 4 commits into from
Jun 24, 2024

Conversation

Vika-F
Copy link
Contributor

@Vika-F Vika-F commented Jun 13, 2024

The warning stared to appear after PR #2794 was merged.

SYCL parallel_for in is_sorted function was re-written to fix the warning.

@Vika-F Vika-F added the dpc++ Issue/PR related to DPC++ functionality label Jun 13, 2024
@Vika-F
Copy link
Contributor Author

Vika-F commented Jun 13, 2024

/intelci: run

@Vika-F
Copy link
Contributor Author

Vika-F commented Jun 17, 2024

CI with the newer icpx compiler: http://intel-ci.intel.com/ef2c8d40-f2f6-f105-a086-a4bf010d0e2e

@Vika-F
Copy link
Contributor Author

Vika-F commented Jun 17, 2024

/intelci: run

@Vika-F Vika-F marked this pull request as ready for review June 18, 2024 08:47
Copy link
Contributor

@avolkov-intel avolkov-intel left a comment

Choose a reason for hiding this comment

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

LGTM with minor comment about use of ternary operator

Co-authored-by: Anatoly Volkov <117643568+avolkov-intel@users.noreply.github.com>
@Vika-F
Copy link
Contributor Author

Vika-F commented Jun 19, 2024

/intelci: run

@Vika-F
Copy link
Contributor Author

Vika-F commented Jun 19, 2024

CI with the new compiler: http://intel-ci.intel.com/ef2ee00d-02e4-f186-a42f-a4bf010d0e2e

@Vika-F
Copy link
Contributor Author

Vika-F commented Jun 21, 2024

@Vika-F Vika-F merged commit 89a5667 into oneapi-src:main Jun 24, 2024
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dpc++ Issue/PR related to DPC++ functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants