[MemorySSA][NFC] Use const whenever possible

Differential Revision: https://reviews.llvm.org/D134162
This commit is contained in:
luxufan 2022-09-19 02:24:10 +00:00
parent eff5c3e6c6
commit bfd31c6f12
2 changed files with 5 additions and 5 deletions

View File

@ -1241,7 +1241,7 @@ private:
/// Returns true if \p Ptr is guaranteed to be loop invariant for any possible /// 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 /// loop. In particular, this guarantees that it only references a single
/// MemoryLocation during execution of the containing function. /// MemoryLocation during execution of the containing function.
bool IsGuaranteedLoopInvariant(Value *Ptr) const; bool IsGuaranteedLoopInvariant(const Value *Ptr) const;
void fillInCurrentPair() { void fillInCurrentPair() {
CurrentPair.first = *DefIterator; CurrentPair.first = *DefIterator;
@ -1252,7 +1252,7 @@ private:
// to unknown guarantees that any memory accesses that access locations // to unknown guarantees that any memory accesses that access locations
// after the pointer are considered as clobbers, which is important to // after the pointer are considered as clobbers, which is important to
// catch loop carried dependences. // catch loop carried dependences.
if (!IsGuaranteedLoopInvariant(const_cast<Value *>(Location.Ptr))) if (!IsGuaranteedLoopInvariant(Location.Ptr))
CurrentPair.second = CurrentPair.second =
Location.getWithNewSize(LocationSize::beforeOrAfterPointer()); Location.getWithNewSize(LocationSize::beforeOrAfterPointer());
PHITransAddr Translator( PHITransAddr Translator(
@ -1267,7 +1267,7 @@ private:
CurrentPair.second = CurrentPair.second.getWithNewPtr(TransAddr); CurrentPair.second = CurrentPair.second.getWithNewPtr(TransAddr);
if (TransAddr && if (TransAddr &&
!IsGuaranteedLoopInvariant(const_cast<Value *>(TransAddr))) !IsGuaranteedLoopInvariant(TransAddr))
CurrentPair.second = CurrentPair.second.getWithNewSize( CurrentPair.second = CurrentPair.second.getWithNewSize(
LocationSize::beforeOrAfterPointer()); LocationSize::beforeOrAfterPointer());

View File

@ -2638,8 +2638,8 @@ void MemoryUse::deleteMe(DerivedUser *Self) {
delete static_cast<MemoryUse *>(Self); delete static_cast<MemoryUse *>(Self);
} }
bool upward_defs_iterator::IsGuaranteedLoopInvariant(Value *Ptr) const { bool upward_defs_iterator::IsGuaranteedLoopInvariant(const Value *Ptr) const {
auto IsGuaranteedLoopInvariantBase = [](Value *Ptr) { auto IsGuaranteedLoopInvariantBase = [](const Value *Ptr) {
Ptr = Ptr->stripPointerCasts(); Ptr = Ptr->stripPointerCasts();
if (!isa<Instruction>(Ptr)) if (!isa<Instruction>(Ptr))
return true; return true;