[llvm-commits] [llvm] r161256 - /llvm/trunk/lib/CodeGen/StrongPHIElimination.cpp

[David Blaikie]

On Fri, Aug 3, 2012 at 1:54 PM, Matt Beaumont-Gay <matthewbg at google.com> wrote:
> Author: matthewbg
> Date: Fri Aug  3 15:54:11 2012
> New Revision: 161256
>
> URL: http://llvm.org/viewvc/llvm-project?rev=161256&view=rev
> Log:
> Silence unused variable warning in -asserts build
>
> Modified:
>     llvm/trunk/lib/CodeGen/StrongPHIElimination.cpp
>
> Modified: llvm/trunk/lib/CodeGen/StrongPHIElimination.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/StrongPHIElimination.cpp?rev=161256&r1=161255&r2=161256&view=diff
> ==============================================================================
> --- llvm/trunk/lib/CodeGen/StrongPHIElimination.cpp (original)
> +++ llvm/trunk/lib/CodeGen/StrongPHIElimination.cpp Fri Aug  3 15:54:11 2012
> @@ -672,6 +672,7 @@
>        LiveInterval &SrcInterval = LI->getInterval(SrcReg);
>        SlotIndex PredIndex = LI->getMBBEndIdx(PredBB);
>        VNInfo *SrcVNI = SrcInterval.getVNInfoBefore(PredIndex);
> +      (void)SrcVNI;
>        assert(SrcVNI);

Looks like this could/should just be rolled into the assert so we
don't even call the function if it's a release build:

assert(SrcInterval.getVNInfoBefore(PredIndex));

(& to whoever actually wrote this originally - could use some text to
go with it)

>        continue;
>      }
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits