Skip to content

Conversation

@dannypsnl
Copy link
Member

@dannypsnl dannypsnl commented Mar 31, 2021

result:

X
Y

and we can update type cache without suffering by infinite loop

@mewmew
Copy link
Member

mewmew commented Apr 19, 2021

Hi @dannypsnl,

For the same reason as outlined in #171 (comment), I would like to maintain the property that every value really is the underlying value (e.g. the result of an add instruction; *ir.AddInst) instead of being an identifier that has no underlying value (e.g. a *LocalIdent`).

While using LocalIdent solves the issue with having to use type caches for cyclic values, it also limits the expressiveness and power of the IR representation of the llir/llvm API.

@dannypsnl
Copy link
Member Author

Close #116 , the expressiveness is the main reason we give up cache in Ident.

@dannypsnl dannypsnl closed this Apr 19, 2021
@dannypsnl dannypsnl deleted the cache-v2 branch April 19, 2021 12:31
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.

3 participants