llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Timm Baeder (tbaederr) <details> <summary>Changes</summary> --- Full diff: https://github.com/llvm/llvm-project/pull/215078.diff 1 Files Affected: - (modified) clang/lib/AST/ByteCode/Compiler.cpp (+2-2) ``````````diff diff --git a/clang/lib/AST/ByteCode/Compiler.cpp b/clang/lib/AST/ByteCode/Compiler.cpp index 01cfdeb3ef0fb..316b2a4f092f9 100644 --- a/clang/lib/AST/ByteCode/Compiler.cpp +++ b/clang/lib/AST/ByteCode/Compiler.cpp @@ -37,8 +37,8 @@ static std::optional<bool> getBoolValue(const Expr *E) { return std::nullopt; } -/// Check if \c E has side-effects. This is used to avoid some tempoarary -/// variables and is supposed to be a quick check, not exhausite. That's why +/// Check if \c E has side-effects. This is used to avoid some temporary +/// variables and is supposed to be a quick check, not exhaustive. That's why /// we're not using Expr::HasSideEffects(). static bool isSideEffectFree(const Expr *E) { if (isa<IntegerLiteral, FloatingLiteral, CharacterLiteral, `````````` </details> https://github.com/llvm/llvm-project/pull/215078 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
