diff options
author | Florian Hahn <flo@fhahn.com> | 2025-02-15 16:17:42 +0100 |
---|---|---|
committer | Florian Hahn <flo@fhahn.com> | 2025-02-15 16:17:42 +0100 |
commit | 4664a4c66b816af53f596935c3aaa2eca143ae9c (patch) | |
tree | 4b51df8f2f89d18976e2d9127bece4af245dd924 /llvm/lib/Analysis/LoopAccessAnalysis.cpp | |
parent | 21e956df9b2b283c2f2ed710c542ebeebf8473ff (diff) | |
download | llvm-4664a4c66b816af53f596935c3aaa2eca143ae9c.zip llvm-4664a4c66b816af53f596935c3aaa2eca143ae9c.tar.gz llvm-4664a4c66b816af53f596935c3aaa2eca143ae9c.tar.bz2 |
[LAA] Use getPointer/setPointer in createCheckForAccess (NFC).
Use getPointer/setPointer to clarify we are accessing/modifying the
rurrent value.
Diffstat (limited to 'llvm/lib/Analysis/LoopAccessAnalysis.cpp')
-rw-r--r-- | llvm/lib/Analysis/LoopAccessAnalysis.cpp | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/llvm/lib/Analysis/LoopAccessAnalysis.cpp b/llvm/lib/Analysis/LoopAccessAnalysis.cpp index 4bdcccd..e5b87d2 100644 --- a/llvm/lib/Analysis/LoopAccessAnalysis.cpp +++ b/llvm/lib/Analysis/LoopAccessAnalysis.cpp @@ -1143,9 +1143,8 @@ bool AccessAnalysis::createCheckForAccess(RuntimePointerChecking &RtCheck, SmallVector<PointerIntPair<const SCEV *, 1, bool>> TranslatedPtrs = findForkedPointer(PSE, StridesMap, Ptr, TheLoop); - for (const auto &P : TranslatedPtrs) { - const SCEV *PtrExpr = get<0>(P); - if (!hasComputableBounds(PSE, Ptr, PtrExpr, TheLoop, Assume)) + for (auto &P : TranslatedPtrs) { + if (!hasComputableBounds(PSE, Ptr, P.getPointer(), TheLoop, Assume)) return false; // When we run after a failing dependency check we have to make sure @@ -1161,8 +1160,7 @@ bool AccessAnalysis::createCheckForAccess(RuntimePointerChecking &RtCheck, // If there's only one option for Ptr, look it up after bounds and wrap // checking, because assumptions might have been added to PSE. if (TranslatedPtrs.size() == 1) - TranslatedPtrs[0] = {replaceSymbolicStrideSCEV(PSE, StridesMap, Ptr), - false}; + P.setPointer(replaceSymbolicStrideSCEV(PSE, StridesMap, Ptr)); } for (auto [PtrExpr, NeedsFreeze] : TranslatedPtrs) { |