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

[SYCL] Fix warnings in reduction tests; make them more stable and verbose #394

Merged
merged 2 commits into from
Aug 10, 2021

Conversation

v-klochkov
Copy link

@v-klochkov v-klochkov commented Aug 4, 2021

The main motivation for this patch was the difficulties with reading
the diagnostics printed by reduction tests in CI testing.
The patch should fix that and also help reproducing the CI issues easier.

Signed-off-by: Vyacheslav N Klochkov [email protected]

@v-klochkov v-klochkov force-pushed the reduction_range_lit branch from c5554a7 to 1ac2fe0 Compare August 5, 2021 00:55
@v-klochkov v-klochkov changed the title [SYCL] Fix warnings in reduction tests and make them more robust [SYCL] Fix warnings in reduction tests; make them more stable and verbose Aug 5, 2021
@v-klochkov v-klochkov force-pushed the reduction_range_lit branch 4 times, most recently from b27e9b4 to 9e8469a Compare August 9, 2021 19:16
…bose

The main motivation for this patch was the difficulties with reading
the diagnostics printed by reduction tests in CI testing.
The patch should fix that and also help reproducing the CI issues easier.

Signed-off-by: Vyacheslav N Klochkov <[email protected]>
@v-klochkov v-klochkov force-pushed the reduction_range_lit branch from 9e8469a to 2dfd030 Compare August 9, 2021 19:49
The test was supposed to be disabled by intel#389, but a wrong syntax was
chosen there to do that. Fixed it here.

Signed-off-by: Vyacheslav N Klochkov <[email protected]>
@v-klochkov v-klochkov merged commit 14622ae into intel:intel Aug 10, 2021
@v-klochkov v-klochkov deleted the reduction_range_lit branch August 10, 2021 17:15
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Apr 12, 2022
[ESIMD] Enable aot_mixed.cpp on Linux due to driver update.
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…bose (intel/llvm-test-suite#394)

* [SYCL] Fix warnings in reduction tests; make them more stable and verbose

1) The main motivation for this patch was the difficulties with reading
the diagnostics printed by reduction tests in CI testing.
The patch should fix that and also help reproducing the CI issues easier.

2) Disable the test reduction_nd_N_vars.cpp
The test was supposed to be disabled by intel#389, but a wrong syntax was
chosen there to do that. Fixed it here.

Signed-off-by: Vyacheslav N Klochkov <[email protected]>
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