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

[SYCL] [NATIVECPU] Integrate OneAPI Construction Kit vectorizer #12659

Merged
merged 59 commits into from
Feb 28, 2024

Conversation

PietroGhg
Copy link
Contributor

This PR enables Whole Function Vectorization for the Native CPU backend by integrating the OneAPI Construction Kit's vectorizer.

Copy link
Contributor

github-actions bot commented Feb 8, 2024

✅ With the latest revision this PR passed the C/C++ code formatter.

Copy link
Contributor

@elizabethandrews elizabethandrews left a comment

Choose a reason for hiding this comment

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

FE changes LGTM

@PietroGhg
Copy link
Contributor Author

PietroGhg commented Feb 12, 2024

Hi @intel/dpcpp-tools-reviewers, can I get some feedback on this PR?

Copy link
Contributor

@smanna12 smanna12 left a comment

Choose a reason for hiding this comment

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

FE changes LGTM

@PietroGhg
Copy link
Contributor Author

@intel/dpcpp-tools-reviewers friendly ping :)

@PietroGhg
Copy link
Contributor Author

Hi @intel/llvm-gatekeepers, this looks ready to be merged, thank you

@sommerlukas sommerlukas merged commit 330ac57 into intel:sycl Feb 28, 2024
13 checks passed
sommerlukas added a commit that referenced this pull request Feb 28, 2024
@sommerlukas
Copy link
Contributor

Compilation of the project failed in post-commit. I've put up #12852 to temporarily revert this PR while the reason for the failure is investigated.

@PietroGhg
Copy link
Contributor Author

The post commit failure should be fixed by #12854

aelovikov-intel pushed a commit that referenced this pull request Feb 28, 2024
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.

6 participants