[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


================
@@ -373,7 +373,8 @@ static bool canSkipClobberingStore(const StoreInst *SI,
     return false;
   if (MemoryLocation::get(SI).Size != MemLoc.Size)
     return false;
-  if (std::min(MemLocAlign, SI->getAlign()).value() < MemLoc.Size.getValue())
+  if (std::min(MemLocAlign, SI->getAlign()).value() <
+      MemLoc.Size.getUIntValue())
----------------
nikic wrote:

I believe this needs to exclude scalable vectors. We don't know whether the alignment is larger than the access size or not.

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


More information about the llvm-commits mailing list