From bfd31c6f12f1645a186ff1d521575b92a08d1231 Mon Sep 17 00:00:00 2001 From: luxufan Date: Mon, 19 Sep 2022 02:24:10 +0000 Subject: [PATCH] [MemorySSA][NFC] Use const whenever possible Differential Revision: https://reviews.llvm.org/D134162 --- llvm/include/llvm/Analysis/MemorySSA.h | 6 +++--- llvm/lib/Analysis/MemorySSA.cpp | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/llvm/include/llvm/Analysis/MemorySSA.h b/llvm/include/llvm/Analysis/MemorySSA.h index ac1daf8b0580..4a5eeb5a4020 100644 --- a/llvm/include/llvm/Analysis/MemorySSA.h +++ b/llvm/include/llvm/Analysis/MemorySSA.h @@ -1241,7 +1241,7 @@ private: /// Returns true if \p Ptr is guaranteed to be loop invariant for any possible /// loop. In particular, this guarantees that it only references a single /// MemoryLocation during execution of the containing function. - bool IsGuaranteedLoopInvariant(Value *Ptr) const; + bool IsGuaranteedLoopInvariant(const Value *Ptr) const; void fillInCurrentPair() { CurrentPair.first = *DefIterator; @@ -1252,7 +1252,7 @@ private: // to unknown guarantees that any memory accesses that access locations // after the pointer are considered as clobbers, which is important to // catch loop carried dependences. - if (!IsGuaranteedLoopInvariant(const_cast(Location.Ptr))) + if (!IsGuaranteedLoopInvariant(Location.Ptr)) CurrentPair.second = Location.getWithNewSize(LocationSize::beforeOrAfterPointer()); PHITransAddr Translator( @@ -1267,7 +1267,7 @@ private: CurrentPair.second = CurrentPair.second.getWithNewPtr(TransAddr); if (TransAddr && - !IsGuaranteedLoopInvariant(const_cast(TransAddr))) + !IsGuaranteedLoopInvariant(TransAddr)) CurrentPair.second = CurrentPair.second.getWithNewSize( LocationSize::beforeOrAfterPointer()); diff --git a/llvm/lib/Analysis/MemorySSA.cpp b/llvm/lib/Analysis/MemorySSA.cpp index 30e85dce77b0..ece45e528d10 100644 --- a/llvm/lib/Analysis/MemorySSA.cpp +++ b/llvm/lib/Analysis/MemorySSA.cpp @@ -2638,8 +2638,8 @@ void MemoryUse::deleteMe(DerivedUser *Self) { delete static_cast(Self); } -bool upward_defs_iterator::IsGuaranteedLoopInvariant(Value *Ptr) const { - auto IsGuaranteedLoopInvariantBase = [](Value *Ptr) { +bool upward_defs_iterator::IsGuaranteedLoopInvariant(const Value *Ptr) const { + auto IsGuaranteedLoopInvariantBase = [](const Value *Ptr) { Ptr = Ptr->stripPointerCasts(); if (!isa(Ptr)) return true;