[llvm-commits] [llvm] r116240 - /llvm/trunk/lib/Target/ARM/ARMFastISel.cpp

[Chris Lattner]

On Oct 11, 2010, at 2:37 PM, Eric Christopher wrote:

> Author: echristo
> Date: Mon Oct 11 16:37:35 2010
> New Revision: 116240
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=116240&view=rev
> Log:
> We're not going to handle dynamic allocas anywhere else.

>     case Instruction::Alloca: {
> -      assert(false && "Alloca should have been handled earlier!");
> +      // Don't handle dynamic allocas.
> +      if (FuncInfo.StaticAllocaMap.count(cast<AllocaInst>(Obj)))
> +        assert(false && "Alloca should have been handled earlier!");

How about:

 assert(!FuncInfo.StaticAllocaMap.count(cast<AllocaInst>(Obj)) &&
        "Alloca should have been handled earlier!");

-Chris