[llvm] 0e0dea8 - Add missing override modifiers for FileCheckExpressionAST::eval() overrides.

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 11 10:56:01 PST 2019


Author: Simon Pilgrim
Date: 2019-11-11T18:51:46Z
New Revision: 0e0dea82681a86a09ceed0afd321367a35bfa051

URL: https://github.com/llvm/llvm-project/commit/0e0dea82681a86a09ceed0afd321367a35bfa051
DIFF: https://github.com/llvm/llvm-project/commit/0e0dea82681a86a09ceed0afd321367a35bfa051.diff

LOG: Add missing override modifiers for FileCheckExpressionAST::eval() overrides.

Added: 
    

Modified: 
    llvm/lib/Support/FileCheckImpl.h

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Support/FileCheckImpl.h b/llvm/lib/Support/FileCheckImpl.h
index 0fd8523bef8f..eee653a3b32a 100644
--- a/llvm/lib/Support/FileCheckImpl.h
+++ b/llvm/lib/Support/FileCheckImpl.h
@@ -51,7 +51,7 @@ class FileCheckExpressionLiteral : public FileCheckExpressionAST {
   FileCheckExpressionLiteral(uint64_t Val) : Value(Val) {}
 
   /// \returns the literal's value.
-  Expected<uint64_t> eval() const { return Value; }
+  Expected<uint64_t> eval() const override { return Value; }
 };
 
 /// Class to represent an undefined variable error, which quotes that
@@ -133,7 +133,7 @@ class FileCheckNumericVariableUse : public FileCheckExpressionAST {
       : Name(Name), NumericVariable(NumericVariable) {}
 
   /// \returns the value of the variable referenced by this instance.
-  Expected<uint64_t> eval() const;
+  Expected<uint64_t> eval() const override;
 };
 
 /// Type of functions evaluating a given binary operation.
@@ -164,7 +164,7 @@ class FileCheckASTBinop : public FileCheckExpressionAST {
   /// using EvalBinop on the result of recursively evaluating the operands.
   /// \returns the expression value or an error if an undefined numeric
   /// variable is used in one of the operands.
-  Expected<uint64_t> eval() const;
+  Expected<uint64_t> eval() const override;
 };
 
 class FileCheckPatternContext;


        


More information about the llvm-commits mailing list