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

[misc] Lazy load spirv code from disk during offline cache #6000

Merged
merged 6 commits into from
Sep 13, 2022

Conversation

PGZXB
Copy link
Contributor

@PGZXB PGZXB commented Sep 7, 2022

Related issue = #4401

ps: The above testing was run in an environment with 100MB+ cache files.
(100MB is the default value of offline_cache_max_size_of_files

@PGZXB
Copy link
Contributor Author

PGZXB commented Sep 7, 2022

/rebase

@taichi-gardener taichi-gardener force-pushed the dev-gfx_CacheManager-lazy-loading branch from 87afe7b to d6776e8 Compare September 7, 2022 13:04
@PGZXB
Copy link
Contributor Author

PGZXB commented Sep 8, 2022

/rebase

@taichi-gardener taichi-gardener force-pushed the dev-gfx_CacheManager-lazy-loading branch from d6776e8 to 54a6909 Compare September 8, 2022 08:12
@netlify
Copy link

netlify bot commented Sep 8, 2022

Deploy Preview for docsite-preview ready!

Name Link
🔨 Latest commit e715cd8
🔍 Latest deploy log https://app.netlify.com/sites/docsite-preview/deploys/631fd09c82fe210009a7b651
😎 Deploy Preview https://deploy-preview-6000--docsite-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Copy link
Contributor

@ailzhang ailzhang left a comment

Choose a reason for hiding this comment

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

LGTM!

taichi/runtime/gfx/aot_module_loader_impl.cpp Outdated Show resolved Hide resolved
@PGZXB PGZXB merged commit c6a88ba into taichi-dev:master Sep 13, 2022
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