[llvm-commits] [llvm] r84327 - /llvm/trunk/lib/CodeGen/CodePlacementOpt.cpp

[Chris Lattner]

On Oct 17, 2009, at 2:33 AM, Daniel Dunbar wrote:

> Author: ddunbar
> Date: Sat Oct 17 04:33:00 2009
> New Revision: 84327
>
> URL: http://llvm.org/viewvc/llvm-project?rev=84327&view=rev
> Log:
> Suppress -Asserts warning.

Despite the name, I think that AnalyzeBranch does mutate the code in  
some cases on some targets.  Is this true?

-Chris

>
> Modified:
>    llvm/trunk/lib/CodeGen/CodePlacementOpt.cpp
>
> Modified: llvm/trunk/lib/CodeGen/CodePlacementOpt.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/CodePlacementOpt.cpp?rev=84327&r1=84326&r2=84327&view=diff
>
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> ======================================================================
> --- llvm/trunk/lib/CodeGen/CodePlacementOpt.cpp (original)
> +++ llvm/trunk/lib/CodeGen/CodePlacementOpt.cpp Sat Oct 17 04:33:00  
> 2009
> @@ -153,8 +153,8 @@
>
>   MachineBasicBlock *TBB = 0, *FBB = 0;
>   SmallVector<MachineOperand, 4> Cond;
> -  bool B = TII->AnalyzeBranch(*MBB, TBB, FBB, Cond);
> -  assert(!B && "UpdateTerminators requires analyzable  
> predecessors!");
> +  assert(!TII->AnalyzeBranch(*MBB, TBB, FBB, Cond) &&
> +         "UpdateTerminators requires analyzable predecessors!");
>   if (Cond.empty()) {
>     if (TBB) {
>       // The block has an unconditional branch. If its successor is  
> now
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits