[llvm] [MemDepAnalysis] Don't reuse NonLocalPointerDeps cache if memory location size differs (PR #116936)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 20 01:11:50 PST 2024


https://github.com/nikic commented:

I think that in principle, not reusing cache entries from a greater size for a smaller one is correct. As you point out, it may be incorrect if the larger size is UB.

It should still be fine to reuse a cache entry without size for one with, and it would also be valid to use an upperBound() larger size instead of a precise() one. But possibly it's not worth bothering with it.

https://github.com/llvm/llvm-project/pull/116936


More information about the llvm-commits mailing list