[llvm] [Analysis] Add Scalable field in MemoryLocation.h (PR #69716)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 20 07:38:09 PDT 2023


================
@@ -1103,7 +1104,8 @@ bool MemoryDependenceResults::getNonLocalPointerDepFromBB(
         // be conservative.
         ThrowOutEverything =
             CacheInfo->Size.isPrecise() != Loc.Size.isPrecise() ||
-            CacheInfo->Size.getValue() < Loc.Size.getValue();
+            TypeSize::isKnownLT(CacheInfo->Size.getValue(),
----------------
nikic wrote:

```suggestion
            !TypeSize::isKnownGE(CacheInfo->Size.getValue(),
```
I believe you will miss some invalidations otherwise.

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


More information about the llvm-commits mailing list