Index: llvm/lib/Transforms/Scalar/LICM.cpp =================================================================== --- llvm/lib/Transforms/Scalar/LICM.cpp +++ llvm/lib/Transforms/Scalar/LICM.cpp @@ -558,7 +558,7 @@ for (BasicBlock::iterator II = BB->end(); II != BB->begin();) { Instruction &I = *--II; - // If the instruction is dead, we would try to sink it because it isn't + // If the instruction is dead, we wouldn't try to sink it because it isn't // used in the loop, instead, just delete it. if (isInstructionTriviallyDead(&I, TLI)) { LLVM_DEBUG(dbgs() << "LICM deleting dead inst: " << I << '\n'); @@ -1126,7 +1126,7 @@ isa(I) || isa(I)); } /// Return true if all of the alias sets within this AST are known not to -/// contain a Mod, or if MSSA knows thare are no MemoryDefs in the loop. +/// contain a Mod, or if MSSA knows there are no MemoryDefs in the loop. bool isReadOnly(AliasSetTracker *CurAST, const MemorySSAUpdater *MSSAU, const Loop *L) { if (CurAST) { @@ -1349,13 +1349,12 @@ return false; } else if (const auto *MD = dyn_cast(&MA)) { if (auto *LI = dyn_cast(MD->getMemoryInst())) { - (void)LI; // Silence warning. assert(!LI->isUnordered() && "Expected unordered load"); return false; } // Any call, while it may not be clobbering SI, it may be a use. if (auto *CI = dyn_cast(MD->getMemoryInst())) { - // Check if the call may read from the memory locattion written + // Check if the call may read from the memory location written // to by SI. Check CI's attributes and arguments; the number of // such checks performed is limited above by NoOfMemAccTooLarge. ModRefInfo MRI = AA->getModRefInfo(CI, MemoryLocation::get(SI)); @@ -2097,7 +2096,7 @@ // Note that proving a load safe to speculate requires proving // sufficient alignment at the target location. Proving it guaranteed // to execute does as well. Thus we can increase our guaranteed - // alignment as well. + // alignment as well. if (!DereferenceableInPH || (InstAlignment > Alignment)) if (isSafeToExecuteUnconditionally(*Load, DT, TLI, CurLoop, SafetyInfo, ORE,