[llvm-commits] CVS: llvm/include/llvm/Function.h GlobalValue.h GlobalVariable.h Module.h Value.h ValueSymbolTable.h

[Chris Lattner]

> @@ -121,6 +121,16 @@
>    /// value is outside of the current translation unit...
>    virtual bool isDeclaration() const = 0;
>
> +  /// removeFromParent - This method unlinks 'this' from the  
> containing module,
> +  /// but does not delete it.
> +  ///
> +  virtual void removeFromParent() = 0;
> +
> +  /// eraseFromParent - This method unlinks 'this' from the  
> containing module
> +  /// and deletes it.
> +  ///
> +  virtual void eraseFromParent() = 0;

Why do these all need to be virtual and accessible from GlobalValue?

Otherwise, looks good!

-Chris