[llvm] [SCEV] Remove EqCacheSCEV (PR #133186)
Arthur Eubanks via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 27 16:38:35 PDT 2025
aeubanks wrote:
I believe this is only a partial fix because if we limit ourselves to three characters we can still end up in a situation like
```
abc
abcd
abce
abcd
```
and we don't end up deduplicating the `abcd`s. However, removing the depth limit may have compile time consequences, whereas the usefulness of the cache I think is unknown (discussed in https://reviews.llvm.org/D26389). this PR is an improvement on its own
https://github.com/llvm/llvm-project/pull/133186
More information about the llvm-commits
mailing list