-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[aot] Reorganized C-API headers #6199
Conversation
✅ Deploy Preview for docsite-preview canceled.
|
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!
b2659e7
to
b5f42b7
Compare
797884e
to
fa9840c
Compare
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.
lol, looks like this PR is a mixture of:
- Remove third-party dependencies (openGL, Vulkan, ...)
- C-api Versioning
- code clean up
- code formatting
I do think C-API Versioning worth a separate PR, can we at least split that out? The other parts LGTM
@jim19930609 sounds good. the entire thing breaks compilation heavily so many changes are made. i'll try chopping'm up |
a205972
to
a8d5dc6
Compare
[pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci Update taichi.h Update taichi.h Reorganized headers Fixed compilation [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci Fixed compilation Fixed compilation Fixed compilation Fixed compilation [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci
a8d5dc6
to
ff5c1ea
Compare
[pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci Update taichi.h Update taichi.h Reorganized headers Fixed compilation [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci Fixed compilation Fixed compilation Fixed compilation Fixed compilation [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci
…into capi-header1
As @jim19930609 mentioned before including vulkan headers might be contaminating the user namespaces so this PR removes the include and macro. The users should include
vulkan/vulkan.h
beforetaichi/taichi.h
.