-
Notifications
You must be signed in to change notification settings - Fork 787
[SYCL] Release specialization constant buffer #5167
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
Conversation
/summary:run |
/summary:run |
|
Should be fixed by intel/llvm-test-suite#655. Please, re-run the job. |
I re-ran jobs. |
@bader What should I do if |
It looks like tested compiler doesn't include 9abe201. I assume it's because re-run took the old version of the sycl branch. I suggest merging tip of the sycl branch to your private branch. This should trigger new pre-commit, which will test updated compiler. @alexbatashev, FYI. |
Ok, I'll merge 9abe201 and try again |
I suggest you merge HEAD of the |
Got it. |
/summary:run |
/verify with intel/llvm-test-suite#643 |
Failure in |
I see a lot of failures in the logs. Most of them are already reported and handled, but I see a few new issues: Linux_LLVM-Test-Suite_CUDA_FAILURE.log [2021-12-22T16:18:05.327Z] ******************** Windows_LLVM-Test-Suite_FAILURE.log [2021-12-22T17:35:48.479Z] Failed Tests (5): Please, make sure these fails are reported. |
@vladimirlaz, please, merge when you think it's ready. |
XPTI test fails, because test fix also pulled in one line, that is only reported by #5172, which is not in this branch. I'll think of ways of merging tip of SYCL into current branch to workaround this issue. |
For re-runs? |
Exactly |
@alexbatashev Do I need merge last commits, merged into sycl, and re-run tests? |
/summary:run |
This patch releases spec const buffer implementation.
Tests is placed to llvm-test-suite.
link to PR in llvm-test-suite: intel/llvm-test-suite#643