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

[OpenGL] Add mem_offset_in_parent to serialized file in AOT. #3418

Merged
merged 1 commit into from
Nov 8, 2021

Conversation

ailzhang
Copy link
Contributor

@ailzhang ailzhang commented Nov 8, 2021

Related issue = #

@ailzhang ailzhang requested a review from k-ye November 8, 2021 01:25
@netlify
Copy link

netlify bot commented Nov 8, 2021

✔️ Deploy Preview for jovial-fermat-aa59dc ready!

🔨 Explore the source changes: 00c58ee

🔍 Inspect the deploy log: https://app.netlify.com/sites/jovial-fermat-aa59dc/deploys/6188e75394a113000714d2f0

😎 Browse the preview: https://deploy-preview-3418--jovial-fermat-aa59dc.netlify.app

@ailzhang ailzhang requested a review from bobcao3 November 8, 2021 01:25
taichi/backends/opengl/aot_data.h Outdated Show resolved Hide resolved
taichi/backends/opengl/aot_module_builder_impl.cpp Outdated Show resolved Hide resolved
@ailzhang ailzhang force-pushed the opengl_mem_offset branch 2 times, most recently from c2e3d53 to 78dba93 Compare November 8, 2021 06:35
@ailzhang ailzhang requested a review from k-ye November 8, 2021 07:25
Copy link
Member

@k-ye k-ye left a comment

Choose a reason for hiding this comment

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

LGTM!

@@ -31,5 +31,31 @@ void AotModuleBuilder::add_kernel_template(const std::string &identifier,
add_per_backend_tmpl(identifier, key, kernel);
}

bool AotModuleBuilder::all_fields_are_dense_in_container(
Copy link
Member

Choose a reason for hiding this comment

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

I feel like this can have an even wider applicability, but the base AOT builder seems good enough :-)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yup agreed! I'd just keep this here for now since we know it'll be useful for all aot backends for sure. Later we can move it to other places if needed :D

@@ -48,6 +48,10 @@ class AotModuleBuilder {
virtual void add_per_backend_tmpl(const std::string &identifier,
const std::string &key,
Kernel *kernel) = 0;

bool all_fields_are_dense_in_container(const SNode *container);
Copy link
Member

Choose a reason for hiding this comment

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

nit: static

Copy link
Contributor Author

Choose a reason for hiding this comment

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

nice catch!

@ailzhang ailzhang merged commit 3056cbc into taichi-dev:master Nov 8, 2021
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.

2 participants