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

[SYCL] test for barrier batching #750

Merged
merged 3 commits into from
Jan 18, 2022
Merged

Conversation

smaslov-intel
Copy link

Compiler changes are in intel/llvm#5328
Signed-off-by: Sergey V Maslov [email protected]

Signed-off-by: Sergey V Maslov <[email protected]>
@smaslov-intel smaslov-intel requested a review from a team as a code owner January 17, 2022 20:23
@smaslov-intel
Copy link
Author

/verify with intel/llvm#5328

Signed-off-by: Sergey V Maslov <[email protected]>
Signed-off-by: Sergey V Maslov <[email protected]>
@smaslov-intel
Copy link
Author

/verify with intel/llvm#5328

@smaslov-intel
Copy link
Author

/verify with intel/llvm#5328

@vladimirlaz : is this the right syntax? How to verify that the changes from that PR were actually applied for testing? The way the tests fail (hang) look like they are with an unmodified compiler.

@vladimirlaz
Copy link

According to http://icl-jenkins2.sc.intel.com:8080/blue/organizations/jenkins/SYCL_CI%2Fintel%2FLin%2FLLVM_Test_Suite/detail/LLVM_Test_Suite/184/pipeline/79 the correct hash is used: 82b2f3fa20b071fe6aa972bdf63e5ea76b401b60

@smaslov-intel
Copy link
Author

According to http://icl-jenkins2.sc.intel.com:8080/blue/organizations/jenkins/SYCL_CI%2Fintel%2FLin%2FLLVM_Test_Suite/detail/LLVM_Test_Suite/184/pipeline/79 the correct hash is used: 82b2f3fa20b071fe6aa972bdf63e5ea76b401b60

Thanks! I will study it more then.

@smaslov-intel
Copy link
Author

According to http://icl-jenkins2.sc.intel.com:8080/blue/organizations/jenkins/SYCL_CI%2Fintel%2FLin%2FLLVM_Test_Suite/detail/LLVM_Test_Suite/184/pipeline/79 the correct hash is used: 82b2f3fa20b071fe6aa972bdf63e5ea76b401b60

Thanks! I will study it more then.

You link is for "Jenkins/llvm-test-suite" check and the failure there kind of makes sense.
But the fail in "Jenkins/pre-ci-linux" is not expected, and I don't see the right hash being used: http://icl-jenkins.sc.intel.com:8080/blue/organizations/jenkins/LLVM-Test-Suite-CI-TMP%2FLLVM-Test-Suite-CI-Linux/detail/LLVM-Test-Suite-CI-Linux/1536/pipeline/120

Are we only doing "Jenkins/llvm-test-suite" with an llvm PR?

@vladimirlaz
Copy link

/verify with affects only Jenkins/llvm-test-suite

@smaslov-intel
Copy link
Author

/verify with affects only Jenkins/llvm-test-suite

That explains it, thanks!

@smaslov-intel
Copy link
Author

This has expectedly passed with intel/llvm#5328.
The failures without the compiler changes are also as expected.

@againull againull merged commit 4508a3f into intel:intel Jan 18, 2022
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Apr 12, 2022
myler added a commit to myler/llvm-test-suite that referenced this pull request Apr 12, 2022
…xtra

CMPLRTST-15939: disable 'double' tests on unsupported GPUs.
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
@smaslov-intel smaslov-intel deleted the barrier branch October 31, 2022 17:36
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.

3 participants