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

[SYCL] Fix result check in joint matrix tests #1432

Merged
merged 1 commit into from
Dec 2, 2022

Conversation

steffenlarsen
Copy link

Some tests in SYCL/Matrix verify their results but only use the results to print "passed" or "failed". A subset of these tests have checks for "passed" in the output, but does not use FileCheck to govern the check. This commit changes these tests to return a non-zero value in the main function if the result verification failed. This allows the test system to correctly identify tests that fail this verification.

Some tests in SYCL/Matrix verify their results but only use the results
to print "passed" or "failed". A subset of these tests have checks for
"passed" in the output, but does not use FileCheck to govern the check.
This commit changes these tests to return a non-zero value in the main
function if the result verification failed. This allows the test system
to correctly identify tests that fail this verification.

Signed-off-by: Larsen, Steffen <[email protected]>
Copy link

@dkhaldi dkhaldi left a comment

Choose a reason for hiding this comment

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

LGTM

@againull againull merged commit 45daa9b into intel:intel Dec 2, 2022
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Mar 22, 2023
Some tests in SYCL/Matrix verify their results but only use the results
to print "passed" or "failed". A subset of these tests have checks for
"passed" in the output, but does not use FileCheck to govern the check.
This commit changes these tests to return a non-zero value in the main
function if the result verification failed. This allows the test system
to correctly identify tests that fail this verification.

Signed-off-by: Larsen, Steffen <[email protected]>
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…1432)

Some tests in SYCL/Matrix verify their results but only use the results
to print "passed" or "failed". A subset of these tests have checks for
"passed" in the output, but does not use FileCheck to govern the check.
This commit changes these tests to return a non-zero value in the main
function if the result verification failed. This allows the test system
to correctly identify tests that fail this verification.

Signed-off-by: Larsen, Steffen <[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.

3 participants