[PATCH] D90171: [SVE] Fix TypeSize warning in RuntimePointerChecking::insert
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 3 06:52:13 PST 2020
fhahn 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);
----------------
joechrisellis wrote:
> 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.
Not sure how hard it would be to come up with a test, but it would probably be good to have one
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