constify

Author: vgvassilev

clang/include/clang/Interpreter/Interpreter.h

@@ -197,7 +197,7 @@ class Interpreter {
   /// @name Value and pretty printing support
   /// @{
 
-  std::string ValueDataToString(const Value &V);
+  std::string ValueDataToString(const Value &V) const;
   std::string ValueTypeToString(const Value &V) const;
 
   llvm::Expected<Expr *> convertExprToValue(Expr *E);

clang/include/clang/Interpreter/Value.h

@@ -119,9 +119,9 @@ class REPL_EXTERNAL_VISIBILITY Value {
   ~Value();
 
   void printType(llvm::raw_ostream &Out) const;
-  void printData(llvm::raw_ostream &Out);
-  void print(llvm::raw_ostream &Out);
-  void dump();
+  void printData(llvm::raw_ostream &Out) const;
+  void print(llvm::raw_ostream &Out) const;
+  void dump() const;
   void clear();
 
   ASTContext &getASTContext();

clang/lib/Interpreter/InterpreterValuePrinter.cpp

@@ -180,14 +180,15 @@ static std::string CharPtrToString(const char *Ptr) {
 namespace clang {
 
 struct ValueRef : public Value {
-  ValueRef(Interpreter *In, void *Ty) : Value(In, Ty) {
+  ValueRef(const Interpreter *In, void *Ty)
+      : Value(const_cast<Interpreter *>(In), Ty) {
     // Tell the base class to not try to deallocate if it manages the value.
     IsManuallyAlloc = false;
   }
   void setData(long double D) { Data.m_LongDouble = D; }
 };
 
-std::string Interpreter::ValueDataToString(const Value &V) {
+std::string Interpreter::ValueDataToString(const Value &V) const {
   Sema &S = getCompilerInstance()->getSema();
   ASTContext &Ctx = S.getASTContext();
 

clang/lib/Interpreter/Value.cpp

@@ -254,18 +254,18 @@ const ASTContext &Value::getASTContext() const {
   return getInterpreter().getASTContext();
 }
 
-void Value::dump() { print(llvm::outs()); }
+void Value::dump() const { print(llvm::outs()); }
 
 void Value::printType(llvm::raw_ostream &Out) const {
   Out << Interp->ValueTypeToString(*this);
 }
 
-void Value::printData(llvm::raw_ostream &Out) {
+void Value::printData(llvm::raw_ostream &Out) const {
   Out << Interp->ValueDataToString(*this);
 }
 // FIXME: We do not support the multiple inheritance case where one of the base
 // classes has a pretty-printer and the other does not.
-void Value::print(llvm::raw_ostream &Out) {
+void Value::print(llvm::raw_ostream &Out) const {
   assert(OpaqueType != nullptr && "Can't print default Value");
 
   // Don't even try to print a void or an invalid type, it doesn't make sense.