[PATCH] D90171: [SVE] Fix TypeSize warning in RuntimePointerChecking::insert
Joe Ellis via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 3 06:44:01 PST 2020
joechrisellis added inline comments.
================
Comment at: llvm/lib/Analysis/LoopAccessAnalysis.cpp:233
+ const SCEV *EltSizeSCEV =
+ SE->getSizeOfExpr(IntIdxTy, Ptr->getType()->getPointerElementType());
ScEnd = SE->getAddExpr(ScEnd, EltSizeSCEV);
----------------
fhahn wrote:
> I think this slightly changes the behavior if store and alloc size are different, is that intentional? (because getSizeOfExpr uses alloc size).
Excellent spot -- I have fixed this.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D90171/new/
https://reviews.llvm.org/D90171
More information about the llvm-commits
mailing list