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] Add SNode to offline-cache key #4716

Merged
merged 4 commits into from
Apr 18, 2022

Conversation

PGZXB
Copy link
Contributor

@PGZXB PGZXB commented Apr 5, 2022

Related issue = #4401

Temporary:

  1. Reusing ExpressionHumanFriendlyPrinter to impl ExpressionOfflineCacheKeyGenerator
  2. get_hashed_offline_cache_key_of_snode and get_hashed_offline_cache_key are declared/defined in llvm_offline_cache.*. I will move them to offline_cache_util.*

@netlify
Copy link

netlify bot commented Apr 5, 2022

Deploy Preview for docsite-preview ready!

Name Link
🔨 Latest commit c404205
🔍 Latest deploy log https://app.netlify.com/sites/docsite-preview/deploys/62524c76d359ed0008194df7
😎 Deploy Preview https://deploy-preview-4716--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.

@strongoier
Copy link
Contributor

I doubt whether this is the correct way to hash SNodes. We need to make sure that:

  • All SNodeTrees are the same. This can be done by using your hash for the root of a SNodeTree.
  • In an expression/statement, two SNodes can be considered same if and only if they are in the same SNodeTree and with same snode id.

@PGZXB PGZXB requested review from k-ye and strongoier and removed request for k-ye April 12, 2022 06:38
Copy link
Contributor

@strongoier strongoier left a comment

Choose a reason for hiding this comment

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

LGTM!

@strongoier strongoier merged commit 7262bb0 into taichi-dev:master Apr 18, 2022
k-ye pushed a commit to k-ye/taichi that referenced this pull request May 5, 2022
* Add SNode to offline-cache key

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Fix solution of hashing snode

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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

3 participants