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

[SYCL] Add optimized/non-optimized CHECKs for SYCL/XPTI/kernel/* #1000

Merged
merged 2 commits into from
Apr 21, 2022

Conversation

aelovikov-intel
Copy link

If the test-suite was configured to pass -O0 the tests were changing their
behavior because no dead kernel arguments optimization was performed, resulting
in failing CHECKs.

Be explicit in the "RUN:" commands and expected "CHECK:" lines.

If the test-suite was configured to pass -O0 the tests were changing their
behavior because no dead kernel arguments optimization was performed, resulting
in failing CHECKs.

Be explicit in the "RUN:" commands and expected "CHECK:" lines.
@aelovikov-intel aelovikov-intel requested a review from a team as a code owner April 19, 2022 21:21
againull
againull previously approved these changes Apr 19, 2022
The latter isn't the spelling of the option on Windows.
@aelovikov-intel
Copy link
Author

All the failures are unrelated to the tests modified here. Patch is ready to be merged in. @againull , could you please help with this?

@againull againull merged commit 3bd46fd into intel:intel Apr 21, 2022
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Jun 17, 2022
…el#1000)

If the test-suite was configured to pass -O0 the tests were changing their
behavior because no dead kernel arguments optimization was performed, resulting
in failing CHECKs.

Be explicit in the "RUN:" commands and expected "CHECK:" lines.
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Jun 17, 2022
@aelovikov-intel aelovikov-intel deleted the explicit_xpti_checks branch August 15, 2022 20:46
aelovikov-intel added a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…el/llvm-test-suite#1000)

If the test-suite was configured to pass -O0 the tests were changing their
behavior because no dead kernel arguments optimization was performed, resulting
in failing CHECKs.

Be explicit in the "RUN:" commands and expected "CHECK:" lines.
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