Skip to content

Commit

Permalink
Fix some typos in comments: evalute -> evaluate (NFC) (#65906)
Browse files Browse the repository at this point in the history
  • Loading branch information
SuperSodaSea committed Sep 10, 2023
1 parent 9f77fac commit 057564f
Show file tree
Hide file tree
Showing 7 changed files with 11 additions and 11 deletions.
2 changes: 1 addition & 1 deletion clang/include/clang/AST/Expr.h
Original file line number Diff line number Diff line change
Expand Up @@ -566,7 +566,7 @@ class Expr : public ValueStmt {
SmallVectorImpl<
PartialDiagnosticAt> &Diags);

/// isPotentialConstantExprUnevaluted - Return true if this expression might
/// isPotentialConstantExprUnevaluated - Return true if this expression might
/// be usable in a constant expression in C++11 in an unevaluated context, if
/// it were in function FD marked constexpr. Return false if the function can
/// never produce a constant expression, along with diagnostics describing
Expand Down
2 changes: 1 addition & 1 deletion clang/lib/Serialization/ASTWriter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6081,7 +6081,7 @@ void ASTRecordWriter::AddVarDeclInit(const VarDecl *VD) {
Val |= (ES->HasConstantInitialization ? 2 : 0);
Val |= (ES->HasConstantDestruction ? 4 : 0);
APValue *Evaluated = VD->getEvaluatedValue();
// If the evaluted result is constant, emit it.
// If the evaluated result is constant, emit it.
if (Evaluated && (Evaluated->isInt() || Evaluated->isFloat()))
Val |= 8;
}
Expand Down
2 changes: 1 addition & 1 deletion clang/unittests/Analysis/FlowSensitive/TransferTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5855,7 +5855,7 @@ TEST(TransferTest, AnonymousStructWithReferenceField) {

TEST(TransferTest, EvaluateBlockWithUnreachablePreds) {
// This is a crash repro.
// `false` block may not have been processed when we try to evalute the `||`
// `false` block may not have been processed when we try to evaluate the `||`
// after visiting `true`, because it is not necessary (and therefore the edge
// is marked unreachable). Trying to get the analysis state via
// `getEnvironment` for the subexpression still should not crash.
Expand Down
2 changes: 1 addition & 1 deletion flang/include/flang/Lower/Support/Utils.h
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ namespace Fortran::lower {
// Fortran::evaluate::Expr are functional values organized like an AST. A
// Fortran::evaluate::Expr is meant to be moved and cloned. Using the front end
// tools can often cause copies and extra wrapper classes to be added to any
// Fortran::evalute::Expr. These values should not be assumed or relied upon to
// Fortran::evaluate::Expr. These values should not be assumed or relied upon to
// have an *object* identity. They are deeply recursive, irregular structures
// built from a large number of classes which do not use inheritance and
// necessitate a large volume of boilerplate code as a result.
Expand Down
2 changes: 1 addition & 1 deletion flang/lib/Evaluate/shape.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1086,7 +1086,7 @@ void GetShapeHelper::AccumulateExtent(
result = std::move(result) + std::move(n);
if (context_) {
// Fold during expression creation to avoid creating an expression so
// large we can't evalute it without overflowing the stack.
// large we can't evaluate it without overflowing the stack.
result = Fold(*context_, std::move(result));
}
}
Expand Down
10 changes: 5 additions & 5 deletions mlir/include/mlir/Dialect/Linalg/IR/LinalgInterfaces.td
Original file line number Diff line number Diff line change
Expand Up @@ -324,7 +324,7 @@ def LinalgStructuredInterface : OpInterface<"LinalgOp"> {
// MLIR currently does not support dependent interfaces or interface
// inheritance. By construction all ops with StructuredOpInterface must
// implement DestinationStyleOpInterface.
// TODO: reevalute the need for a cast when a better mechanism exists.
// TODO: reevaluate the need for a cast when a better mechanism exists.
return getBlock()->getArguments().take_front(
cast<DestinationStyleOpInterface>(*this->getOperation())
.getNumDpsInputs());
Expand All @@ -342,7 +342,7 @@ def LinalgStructuredInterface : OpInterface<"LinalgOp"> {
// MLIR currently does not support dependent interfaces or interface
// inheritance. By construction all ops with StructuredOpInterface must
// implement DestinationStyleOpInterface.
// TODO: reevalute the need for a cast when a better mechanism exists.
// TODO: reevaluate the need for a cast when a better mechanism exists.
return getBlock()->getArguments().take_back(
cast<DestinationStyleOpInterface>(*this->getOperation())
.getNumDpsInits());
Expand Down Expand Up @@ -421,7 +421,7 @@ def LinalgStructuredInterface : OpInterface<"LinalgOp"> {
// MLIR currently does not support dependent interfaces or interface
// inheritance. By construction all ops with StructuredOpInterface must
// implement DestinationStyleOpInterface.
// TODO: reevalute the need for a cast when a better mechanism exists.
// TODO: reevaluate the need for a cast when a better mechanism exists.
return *(indexingMaps.begin() +
cast<DestinationStyleOpInterface>(*this->getOperation())
.getNumDpsInputs() +
Expand All @@ -442,7 +442,7 @@ def LinalgStructuredInterface : OpInterface<"LinalgOp"> {
// MLIR currently does not support dependent interfaces or interface
// inheritance. By construction all ops with StructuredOpInterface must
// implement DestinationStyleOpInterface.
// TODO: reevalute the need for a cast when a better mechanism exists.
// TODO: reevaluate the need for a cast when a better mechanism exists.
int64_t resultIndex =
opOperand->getOperandNumber() -
cast<DestinationStyleOpInterface>(*this->getOperation())
Expand Down Expand Up @@ -807,7 +807,7 @@ def LinalgStructuredInterface : OpInterface<"LinalgOp"> {
// MLIR currently does not support dependent interfaces or interface
// inheritance. By construction all ops with StructuredOpInterface must
// implement DestinationStyleOpInterface.
// TODO: reevalute the need for a cast when a better mechanism exists.
// TODO: reevaluate the need for a cast when a better mechanism exists.
//========================================================================//

int64_t getNumDpsInputs() {
Expand Down
2 changes: 1 addition & 1 deletion openmp/libomptarget/test/lit.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ def append_dynamic_library_path(name, value, sep):
else:
config.environment[name] = value

# Evalute the environment variable which is a string boolean value.
# Evaluate the environment variable which is a string boolean value.
def evaluate_bool_env(env):
env = env.lower()
possible_true_values = ["on", "true", "1"]
Expand Down

0 comments on commit 057564f

Please sign in to comment.