[llvm-commits] [llvm] r116790 - /llvm/trunk/lib/CodeGen/MachineLICM.cpp

[Evan Cheng]

Oops. Thanks for fixing this!

Evan

On Oct 18, 2010, at 7:50 PM, Andrew Trick wrote:

> Author: atrick
> Date: Mon Oct 18 21:50:50 2010
> New Revision: 116790
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=116790&view=rev
> Log:
> Fix for machine licm assert: RCCost <= RegPressure[RCId]
> in MultiSource/Benchmarks/VersaBench/beamformer/beamformer.
> SmallSet.insert returns true if the element is inserted.
> 
> Modified:
>    llvm/trunk/lib/CodeGen/MachineLICM.cpp
> 
> Modified: llvm/trunk/lib/CodeGen/MachineLICM.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/MachineLICM.cpp?rev=116790&r1=116789&r2=116790&view=diff
> ==============================================================================
> --- llvm/trunk/lib/CodeGen/MachineLICM.cpp (original)
> +++ llvm/trunk/lib/CodeGen/MachineLICM.cpp Mon Oct 18 21:50:50 2010
> @@ -604,7 +604,7 @@
>       if (!Reg || TargetRegisterInfo::isPhysicalRegister(Reg))
>         continue;
> 
> -      bool isNew = !RegSeen.insert(Reg);
> +      bool isNew = RegSeen.insert(Reg);
>       const TargetRegisterClass *RC = MRI->getRegClass(Reg);
>       EVT VT = *RC->vt_begin();
>       unsigned RCId = TLI->getRepRegClassFor(VT)->getID();
> @@ -634,7 +634,7 @@
>     if (!Reg || TargetRegisterInfo::isPhysicalRegister(Reg))
>       continue;
> 
> -    bool isNew = !RegSeen.insert(Reg);
> +    bool isNew = RegSeen.insert(Reg);
>     if (NoImpact)
>       continue;
> 
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits