-
Notifications
You must be signed in to change notification settings - Fork 9.7k
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] remove global variables #7710
Conversation
Thanks for your refactoring! I'm aware this is a draft PR test-backend-ops on ARC A770 GPU
llama-bench
|
@AidanBeltonS Thank you for your reminder! I am aware current interaction between SYCL and common is not perfect, you can review the rough design. |
b49f1c0
to
8dfc5a7
Compare
cc8c48b
to
f32f17a
Compare
@AidanBeltonS @NeoZhangJianyu now |
Yes. |
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.
Thanks for the changes! This is a big but necessary update. I have some minor comments
remove global variables and pack into context
f32f17a
to
d342abc
Compare
Still crash in multiple GPUs: |
@AidanBeltonS @NeoZhangJianyu This PR fixed SYCL broken since #7640 (comment) and I believe it solves #7777 and related, please have a try. Known issues: multi-card support still broken |
I have tested on the A100 GPU and can confirm this fixes #7777 |
Following #7566
Remaining