[llvm] r255706 - Suppress unused variable warning in the no-asserts build.

Richard Trieu via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 15 15:42:35 PST 2015


The one from the line above is from me.  Evgeniy and I both saw the unused
variable and fixed them slightly differently at about the same time.  Only
one cast is needed, so I'll just remove my change.

On Tue, Dec 15, 2015 at 3:35 PM, Peter Collingbourne via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> On Tue, Dec 15, 2015 at 11:30:29PM -0000, Evgeniy Stepanov via
> llvm-commits wrote:
> > Author: eugenis
> > Date: Tue Dec 15 17:30:29 2015
> > New Revision: 255706
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=255706&view=rev
> > Log:
> > Suppress unused variable warning in the no-asserts build.
> >
> > Modified:
> >     llvm/trunk/lib/Transforms/IPO/CrossDSOCFI.cpp
> >
> > Modified: llvm/trunk/lib/Transforms/IPO/CrossDSOCFI.cpp
> > URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/IPO/CrossDSOCFI.cpp?rev=255706&r1=255705&r2=255706&view=diff
> >
> ==============================================================================
> > --- llvm/trunk/lib/Transforms/IPO/CrossDSOCFI.cpp (original)
> > +++ llvm/trunk/lib/Transforms/IPO/CrossDSOCFI.cpp Tue Dec 15 17:30:29
> 2015
> > @@ -92,6 +92,7 @@ ConstantInt *CrossDSOCFI::extractBitSetT
> >      (void)F;
> >      // But can never be a function declaration.
> >      assert(!F || !F->isDeclaration());
> > +    (void)F; // Suppress unused variable warning in the no-asserts
> build.
>
> You already have this line above?
>
> --
> Peter
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151215/e0b4dc86/attachment.html>


More information about the llvm-commits mailing list