> @@ -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

_______________________________________________
llvm-commits mailing list
llvm-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

Reply via email to