Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

[SYCL][ESIMD] Add tests for dpas API #866

Merged
merged 1 commit into from
Feb 25, 2022
Merged

Conversation

sndmitriev
Copy link

Signed-off-by: Sergey Dmitriev [email protected]

@sndmitriev sndmitriev requested a review from a team as a code owner February 24, 2022 15:29
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// REQUIRES: gpu-intel-dg2

Choose a reason for hiding this comment

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

As we talked, there are no DG2 or PVC machines in CI, so this will disable even compilation. Are you going to address this in a separate commit?

Copy link
Author

Choose a reason for hiding this comment

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

No, per discussion with CI folks, compilation tests are supposed to be checked in to compiler tests, while llvm-test-suite contains executable tests. So, it looks like CI would need to be extended to have these machines.

Choose a reason for hiding this comment

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

I see, thanks

@kbobrovs kbobrovs merged commit 6e7b3ce into intel Feb 25, 2022
@sndmitriev sndmitriev deleted the sndmitriev/esimd-dpas-tests branch February 25, 2022 05:53
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Apr 12, 2022
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Jun 17, 2022
myler added a commit to myler/llvm-test-suite that referenced this pull request Jun 17, 2022
CMPLRTST-16338: add new GPU name for REQUIRES: in test driver
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants