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

Conversation

@rbegam
Copy link

@rbegam rbegam commented Apr 30, 2021

Signed-off-by: rbegam [email protected]

@rbegam rbegam requested a review from a team as a code owner April 30, 2021 19:52
@rbegam
Copy link
Author

rbegam commented Apr 30, 2021

This test depends on PR intel/llvm#3593 and vise versa.

@vladimirlaz
Copy link

This test depends on PR intel/llvm#3593 and vise versa.

it looks like we need to submit intel/llvm change first (CI will pass as test is skipped and the tests should be tested locally). Once next nightly is available the CI for intel/llvm-test-suite change can be restarted and change submitted

@rbegam
Copy link
Author

rbegam commented May 11, 2021

@vladimirlaz I agree that this test needs to be merged after the compiler change intel/llvm#3593. But that PR is actually failing in pre-commit checks s the test is unexpectedly passing:
http://icl-jenkins.sc.intel.com:8080/blue/organizations/jenkins/SYCL_CI%2Fintel%2FLin%2FLLVM_Test_Suite/detail/LLVM_Test_Suite/4259/pipeline

Unexpectedly Passed Tests (1):
SYCL :: Basic/parallel_for_indexers.cpp

@vladimirlaz
Copy link

The test failures are expected and need intel/llvm#3593 to be fixed

@vladimirlaz vladimirlaz merged commit 8af7707 into intel:intel May 12, 2021
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