[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