Skip to content

[NFC][SYCL] Update tests which were not properly working for Windows #2804

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

Merged
merged 2 commits into from
Dec 3, 2020

Conversation

mdtoguchi
Copy link
Contributor

A few of the Driver sycl* tests were not configured properly to work
in a windows environment and were disabled. Make the needed adjustments
to the tests and break out any specific testing that needs Linux and
Windows specificity.

A few of the Driver sycl* tests were not configured properly to work
in a windows environment and were disabled.  Make the needed adjustments
to the tests and break out any specific testing that needs Linux and
Windows specificity.
@mdtoguchi
Copy link
Contributor Author

This work is to address #888

@bader bader linked an issue Nov 21, 2020 that may be closed by this pull request
@mdtoguchi mdtoguchi requested a review from AGindinson November 30, 2020 22:31
Copy link
Contributor

@AGindinson AGindinson left a comment

Choose a reason for hiding this comment

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

LGTM! A minor suggestion would be to add descriptive headers to sycl-intelfpga-aoco*.cpp files.

@bader bader merged commit f2a91b3 into intel:sycl Dec 3, 2020
jsji pushed a commit that referenced this pull request Dec 31, 2024
Add a new form of load/store operations for cooperative matrices that accepts two separate arguments: the row index and the column index. Unlike the original approach requiring a pointer to the matrix base, this new form of load/store operations is expected to yield better optimized code on 2dblock read/write instructions on PVC.

CapabilityCooperativeMatrixOffsetInstructionsINTEL = 6238
OpCooperativeMatrixLoadOffsetINTEL = 6239
OpCooperativeMatrixStoreOffsetINTEL = 6240

Spec: #12497

Original commit:
KhronosGroup/SPIRV-LLVM-Translator@193661c352de3ca
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.

[SYCL][Driver] Enable full clang driver unit tests on Windows
3 participants