-
Notifications
You must be signed in to change notification settings - Fork 802
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
Support bufferizable in okl #9787
Conversation
…cuda-graph-in-okl
TensorManager和TmpBufferManager这两个类型包括其内置类的的写法我感觉有点混乱,我建议把内置类都独立出来,然后把声明都写到同一个头文件里面(或者把这两个类放一个头文件,内置类们放一个头文件,没必要把每个类都放一个头文件) 这几个类型也是有交叉的,有些操作不需要写两遍,建议抽出基类使用继承或者使用Mixin。 实现统一放到cpp里,尽量别在头文件里搞。 |
|
Code got formatted by CI. Please request CI again if you still want to have this PR merged. If the PR is from a forked repo, please download the patch files from the GitHub Actions web page and apply them locally. |
CI failed when running job: Build cpu. PR label automerge has been removed |
Speed stats:
|
View latest API docs preview at: https://staging.oneflow.info/docs/Oneflow-Inc/oneflow/pr/9787/ |
Speed stats:
|
View latest API docs preview at: https://staging.oneflow.info/docs/Oneflow-Inc/oneflow/pr/9787/ |
No description provided.