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

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 23 05:50:45 PDT 2023


================
@@ -373,7 +373,10 @@ 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 (MemoryLocation::get(SI).Size.isScalable() || MemLoc.Size.isScalable())
----------------
nikic wrote:

```suggestion
  if (MemLoc.Size.isScalable())
```
Sufficient, because both sizes are the same (checked in the previous line).

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


More information about the llvm-commits mailing list