On Mar 1, 2007, at 1:50 PM, Reid Spencer wrote:
> Index: llvm/include/llvm/Constants.h
> diff -u llvm/include/llvm/Constants.h:1.135 llvm/include/llvm/ 
> Constants.h:1.136
> --- llvm/include/llvm/Constants.h:1.135       Thu Mar  1 13:30:07 2007
> +++ llvm/include/llvm/Constants.h     Thu Mar  1 15:50:41 2007
> @@ -127,6 +127,13 @@
> +  /// This is just a convenience method to make client code  
> smaller for a
> +  /// common case.
> +  /// @brief Determine if the value is one.
> +  virtual bool isUnitValue() const {
> +    return Val == 1;
> +  }

Why is this virtual?  How about "isOne()" or something?

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

Reply via email to