Skip to content
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 op dispatch #93

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

zhaoguochun1995
Copy link
Contributor

No description provided.

@zhaoguochun1995 zhaoguochun1995 force-pushed the zgc/deeplinx_ext_support_op_dispatch branch from 4bae507 to d0c203a Compare April 15, 2024 09:12
@@ -86,3 +87,16 @@ def get_ext():
cmdclass={"build_ext": BuildExtensionWithCompdb},
install_requires=["einops"],
)


setup(
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

请教下 这个位置引入cppextension 是为了使用”TORCH_LIBRARY“吗 我记得BuildExtension和CppExtension 是在cuda 编译才用到, 如果在其他设备上用这种方式编译会不会有问题?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

setup里面的只是个草稿,还有待完善。cppextension不光光可以编译cuda kernel,也可以只编译cpp文件的。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants