-
Notifications
You must be signed in to change notification settings - Fork 130
[SYCL][ESIMD]Add test to validate correct split #1352
Conversation
Complementary compiler PR: intel/llvm#7191 |
SYCL/InvokeSimd/invoke_simd_conv.cpp
Outdated
// RUN: %clangxx -O0 -fsycl -fno-sycl-device-code-split-esimd -Xclang -fsycl-allow-func-ptr %s -o %t.out | ||
// RUN: env IGC_VCSaveStackCallLinkage=1 IGC_VCDirectCallsOnly=1 %GPU_RUN_PLACEHOLDER %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.
non-blocking comment: it may be a bit more convenient if a different name is used:
// RUN: %clangxx -O0 -fsycl -fno-sycl-device-code-split-esimd -Xclang -fsycl-allow-func-ptr %s -o %t.out | |
// RUN: env IGC_VCSaveStackCallLinkage=1 IGC_VCDirectCallsOnly=1 %GPU_RUN_PLACEHOLDER %t.out | |
// RUN: %clangxx -O0 -fsycl -fno-sycl-device-code-split-esimd -Xclang -fsycl-allow-func-ptr %s -o %t0.out | |
// RUN: env IGC_VCSaveStackCallLinkage=1 IGC_VCDirectCallsOnly=1 %GPU_RUN_PLACEHOLDER %t0.out |
/verify with intel/llvm#7191 |
Please provide explanation for the test falures |
Linux Test failures for InvokeSimd/invoke_simd_conv.cpp are expected. |
The PR was updated long ago last time. I rebased it to restart CI and be on safer side. |
Linux test failures: are not related to the test |
Windows test failures: |
@fineg74 - Thank you for the fix. It is merged. The test still has this line added Aug 16: "// TODO: enable on Windows once driver is ready". |
Still failing (#1451) Probably needs different command line, but I am not sure how to do it platform dependent. |
[SYCL] Change library loading and fix esimd selector function
* Add test to validate correct split * Disable execution of a test generated with -O0 option until crash issue is resolved
…e#1352) * Add test to validate correct split * Disable execution of a test generated with -O0 option until crash issue is resolved
No description provided.