[llvm] r238094 - Fix an unused variable warning in release builds.

[Craig Topper]

On Sat, May 23, 2015 at 10:17 PM, Justin Bogner <mail at justinbogner.com>
wrote:

> Craig Topper <craig.topper at gmail.com> writes:
> > Author: ctopper
> > Date: Sat May 23 03:20:33 2015
> > New Revision: 238094
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=238094&view=rev
> > Log:
> > Fix an unused variable warning in release builds.
> >
> > Modified:
> >     llvm/trunk/lib/Transforms/Scalar/LoopStrengthReduce.cpp
> >
> > Modified: llvm/trunk/lib/Transforms/Scalar/LoopStrengthReduce.cpp
> > URL:
> >
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/LoopStrengthReduce.cpp?rev=238094&r1=238093&r2=238094&view=diff
> >
> ==============================================================================
> > --- llvm/trunk/lib/Transforms/Scalar/LoopStrengthReduce.cpp (original)
> > +++ llvm/trunk/lib/Transforms/Scalar/LoopStrengthReduce.cpp Sat May 23
> 03:20:33 2015
> > @@ -4847,6 +4847,7 @@ LSRInstance::LSRInstance(Loop *L, Pass *
> >    unsigned NumUsers = 0;
> >    for (const IVStrideUse &U : IU) {
> >      if (++NumUsers > MaxIVUsers) {
> > +      (void)U;
>
> This warning seems like a bit of a false positive - we're iterating for
> effect here. Is this clang or other compilers warning? Should we file a
> bug that it isn't useful to warn on this?
>

Here's the buildbot failure that I got.
http://lab.llvm.org:8011/builders/lld-x86_64-darwin13/builds/10986


>
> >        DEBUG(dbgs() << "LSR skipping loop, too many IV Users in " << U
> << "\n");
> >        return;
> >      }
> >
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>



-- 
~Craig
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150523/299dfcd1/attachment.html>