[llvm-commits] CVS: llvm/include/llvm/Constants.h
Reid Spencer
rspencer at reidspencer.com
Thu Jan 11 17:02:10 PST 2007
On Thu, 2007-01-11 at 16:54 -0800, Chris Lattner wrote:
> > Index: llvm/include/llvm/Constants.h
> > diff -u llvm/include/llvm/Constants.h:1.114 llvm/include/llvm/
> > Constants.h:1.115
> > --- llvm/include/llvm/Constants.h:1.114 Thu Jan 11 06:24:13 2007
> > +++ llvm/include/llvm/Constants.h Thu Jan 11 08:31:10 2007
> > @@ -96,13 +96,13 @@
> > /// @brief Get a ConstantInt for a specific value.
> > static ConstantInt *get(const Type *Ty, int64_t V);
> >
> > - /// Returns the opposite value of this ConstantInt value if it's
> > a boolean
> > - /// constant.
> > + /// Returns the opposite value of this ConstantInt.
> > /// @brief Get inverse value.
> > inline ConstantInt *inverted() const {
> > static ConstantInt *CI = 0;
> > if (CI) return CI;
> > - return CI = new ConstantInt(getType(), Val ^ (-1));
> > + return CI = new ConstantInt(getType(),
> > + Val ^ (getType() == Type::BoolTy ?
> > 1 : -1));
> > }
>
> This method is incorrect for all values. Please remove it.
Can't remove it, its used. But, I think we can just simplify it to its
intended usage: trade true for false and false for true. Should I name
it getInvertedBool ?
>
> -Chris
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
More information about the llvm-commits
mailing list