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

David Green via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 2 06:20:28 PDT 2023


================
@@ -150,16 +159,22 @@ class LocationSize {
       return beforeOrAfterPointer();
     if (Value == AfterPointer || Other.Value == AfterPointer)
       return afterPointer();
+    if (isScalable() != Other.isScalable())
+      return afterPointer();
 
     return upperBound(std::max(getValue(), Other.getValue()));
----------------
davemgreen wrote:

It looks like this max will implicitly convert to uint64_t. The != above for scalable vectors may need to be a ||.

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


More information about the llvm-commits mailing list