-
Notifications
You must be signed in to change notification settings - Fork 730
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[SYCL][NFC] Drop Gen9 detection from E2E tests #14452
base: sycl
Are you sure you want to change the base?
[SYCL][NFC] Drop Gen9 detection from E2E tests #14452
Conversation
Gen9 HW is not officially supported anymore by our product, we don't have such machines in our CI and therefore it doesn't make sense to keep those legacy LIT features and their usage.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm submitting this as a draft for now to wait for #14451 to reduce amount of conflicts. However, I think that it worth to do an early review of removed files.
@intel/dpcpp-esimd-reviewers, could you please take a look at them? I simply removed them because they require gen9 and nothing else, but I'm not that familiar with ESIMD to say for sure that it was a correct decision and we shouldn't rewrite those tests instead.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
im gonna add new tests for gen12 soon, this test is dead so its okay to remove it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
esimd lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
im gonna add new tests for gen12 soon, this test is dead so its okay to remove it
@@ -13,16 +13,16 @@ | |||
// RUN: %clangxx -fsycl -fsycl-targets=spir64 %s -o %t.out | |||
// RUN: %{run} %t.out | |||
|
|||
// RUN: %clangxx -fsycl -fsycl-targets=spir64_gen -Xsycl-target-backend "-device gen9" %s -o %t.out | |||
// RUN: %clangxx -fsycl -fsycl-targets=spir64_gen -Xsycl-target-backend "-device gen12lp" %s -o %t.out | |||
// RUN: %{run} %t.out |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems to me like this is GPU AOT, so it should be limited to only run on GPU.
// RUN: %{run} %t.out | |
// RUN: %if gpu %{ %{run} %t.out %} |
Signed-off-by: Larsen, Steffen <[email protected]>
Signed-off-by: Larsen, Steffen <[email protected]>
Signed-off-by: Larsen, Steffen <[email protected]>
I cannot approve my own PR, but my comments about recent changes were addressed, so I'm fine with merging this. |
Gen9 HW is not officially supported anymore by our product, we don't have such machines in our CI and therefore it doesn't make sense to keep those legacy LIT features and their usage.