From 6f884e07c57809d8a3af4f6374797fbfaf019382 Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Mon, 9 Mar 2009 05:50:45 +0000 Subject: [PATCH] make GlobalValue::removeDeadConstantUsers() const. llvm-svn: 66403 --- llvm/include/llvm/GlobalValue.h | 2 +- llvm/lib/VMCore/Globals.cpp | 14 +++++++------- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/llvm/include/llvm/GlobalValue.h b/llvm/include/llvm/GlobalValue.h index 33f50c907b87..45528b2713f2 100644 --- a/llvm/include/llvm/GlobalValue.h +++ b/llvm/include/llvm/GlobalValue.h @@ -205,7 +205,7 @@ public: /// off of this global value, remove them. This method is useful for clients /// that want to check to see if a global is unused, but don't want to deal /// with potentially dead constants hanging off of the globals. - void removeDeadConstantUsers(); + void removeDeadConstantUsers() const; // Methods for support type inquiry through isa, cast, and dyn_cast: static inline bool classof(const GlobalValue *) { return true; } diff --git a/llvm/lib/VMCore/Globals.cpp b/llvm/lib/VMCore/Globals.cpp index de1b247150ea..5abe1f9ac40d 100644 --- a/llvm/lib/VMCore/Globals.cpp +++ b/llvm/lib/VMCore/Globals.cpp @@ -28,17 +28,17 @@ using namespace llvm; /// removeDeadUsersOfConstant - If the specified constantexpr is dead, remove /// it. This involves recursively eliminating any dead users of the /// constantexpr. -static bool removeDeadUsersOfConstant(Constant *C) { +static bool removeDeadUsersOfConstant(const Constant *C) { if (isa(C)) return false; // Cannot remove this while (!C->use_empty()) { - Constant *User = dyn_cast(C->use_back()); + const Constant *User = dyn_cast(C->use_back()); if (!User) return false; // Non-constant usage; if (!removeDeadUsersOfConstant(User)) return false; // Constant wasn't dead } - C->destroyConstant(); + const_cast(C)->destroyConstant(); return true; } @@ -46,11 +46,11 @@ static bool removeDeadUsersOfConstant(Constant *C) { /// off of this global value, remove them. This method is useful for clients /// that want to check to see if a global is unused, but don't want to deal /// with potentially dead constants hanging off of the globals. -void GlobalValue::removeDeadConstantUsers() { - Value::use_iterator I = use_begin(), E = use_end(); - Value::use_iterator LastNonDeadUser = E; +void GlobalValue::removeDeadConstantUsers() const { + Value::use_const_iterator I = use_begin(), E = use_end(); + Value::use_const_iterator LastNonDeadUser = E; while (I != E) { - if (Constant *User = dyn_cast(*I)) { + if (const Constant *User = dyn_cast(*I)) { if (!removeDeadUsersOfConstant(User)) { // If the constant wasn't dead, remember that this was the last live use // and move on to the next constant.