[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