[PATCH] D44082: Fixed some more spelling mistake in comments of LLVM Analysis passes
Reshabh Sharma via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 5 21:16:59 PST 2018
reshabh updated this revision to Diff 137126.
reshabh added a comment.
Removed a false correction and updated a wrong spelling .
https://reviews.llvm.org/D44082
Files:
lib/Analysis/BasicAliasAnalysis.cpp
lib/Analysis/GlobalsModRef.cpp
lib/Analysis/InstructionSimplify.cpp
lib/Analysis/LazyValueInfo.cpp
lib/Analysis/ScalarEvolution.cpp
Index: lib/Analysis/ScalarEvolution.cpp
===================================================================
--- lib/Analysis/ScalarEvolution.cpp
+++ lib/Analysis/ScalarEvolution.cpp
@@ -3776,7 +3776,7 @@
}
/// Check whether value has nuw/nsw/exact set but SCEV does not.
-/// TODO: In reality it is better to check the poison recursevely
+/// TODO: In reality it is better to check the poison recursively
/// but this is better than nothing.
static bool SCEVLostPoisonFlags(const SCEV *S, const Value *V) {
if (auto *I = dyn_cast<Instruction>(V)) {
@@ -5914,7 +5914,7 @@
// I first became poison as K):
//
// 1. In the set of iterations including and after K, the loop body executes
- // no side effects. In this case executing the backege an infinte number
+ // no side effects. In this case executing the backege an infinite number
// of times will yield undefined behavior.
//
// 2. In the set of iterations including and after K, the loop body executes
Index: lib/Analysis/LazyValueInfo.cpp
===================================================================
--- lib/Analysis/LazyValueInfo.cpp
+++ lib/Analysis/LazyValueInfo.cpp
@@ -620,7 +620,7 @@
// and the like to prove non-nullness, but it's not clear that's worth it
// compile time wise. The context-insensitive value walk done inside
// isKnownNonZero gets most of the profitable cases at much less expense.
- // This does mean that we have a sensativity to where the defining
+ // This does mean that we have a sensitivity to where the defining
// instruction is placed, even if it could legally be hoisted much higher.
// That is unfortunate.
PointerType *PT = dyn_cast<PointerType>(BBI->getType());
Index: lib/Analysis/InstructionSimplify.cpp
===================================================================
--- lib/Analysis/InstructionSimplify.cpp
+++ lib/Analysis/InstructionSimplify.cpp
@@ -2030,7 +2030,7 @@
default:
return nullptr;
- // Equality comaprisons are easy to fold.
+ // Equality comparisons are easy to fold.
case CmpInst::ICMP_EQ:
case CmpInst::ICMP_NE:
break;
Index: lib/Analysis/GlobalsModRef.cpp
===================================================================
--- lib/Analysis/GlobalsModRef.cpp
+++ lib/Analysis/GlobalsModRef.cpp
@@ -725,7 +725,7 @@
if (InputGV == GV)
return false;
- // Distinct GlobalVariables never alias, unless overriden or zero-sized.
+ // Distinct GlobalVariables never alias, unless overridden or zero-sized.
// FIXME: The condition can be refined, but be conservative for now.
auto *GVar = dyn_cast<GlobalVariable>(GV);
auto *InputGVar = dyn_cast<GlobalVariable>(InputGV);
Index: lib/Analysis/BasicAliasAnalysis.cpp
===================================================================
--- lib/Analysis/BasicAliasAnalysis.cpp
+++ lib/Analysis/BasicAliasAnalysis.cpp
@@ -961,7 +961,7 @@
// heap state at the point the guard is issued needs to be consistent in case
// the guard invokes the "deopt" continuation.
- // NB! This function is *not* commutative, so we specical case two
+ // NB! This function is *not* commutative, so we special case two
// possibilities for guard intrinsics.
if (isIntrinsicCall(CS1, Intrinsic::experimental_guard))
@@ -1128,7 +1128,7 @@
// If a we have (a) a GEP and (b) a pointer based on an alloca, and the
// beginning of the object the GEP points would have a negative offset with
-// repsect to the alloca, that means the GEP can not alias pointer (b).
+// respect to the alloca, that means the GEP can not alias pointer (b).
// Note that the pointer based on the alloca may not be a GEP. For
// example, it may be the alloca itself.
// The same applies if (b) is based on a GlobalVariable. Note that just being
@@ -1625,7 +1625,7 @@
return NoAlias;
// Function arguments can't alias with things that are known to be
- // unambigously identified at the function level.
+ // unambiguously identified at the function level.
if ((isa<Argument>(O1) && isIdentifiedFunctionLocal(O2)) ||
(isa<Argument>(O2) && isIdentifiedFunctionLocal(O1)))
return NoAlias;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D44082.137126.patch
Type: text/x-patch
Size: 4210 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180306/b5910820/attachment.bin>
More information about the llvm-commits
mailing list