On Fri, Dec 11, 2009 at 5:27 PM, Mike Stump <[email protected]> wrote:
> +            if (IdxSigned) {
> +              Builder.CreateCondBr(Builder.CreateICmpSGE(Idx,
> +                                     llvm::ConstantInt::get(Idx->getType(), 
> 0)),
> +                                   Cont, getAbortBB());
> +              EmitBlock(Cont);
> +              Cont = createBasicBlock("cont");
> +              Builder.CreateCondBr(Builder.CreateICmpSLT(Idx,
> +                                  llvm::ConstantInt::get(Idx->getType(), 
> Size)),
> +                                   Cont, getAbortBB());
> +              EmitBlock(Cont);
> +            } else {
> +              llvm::BasicBlock *Cont = createBasicBlock("cont");
> +              Builder.CreateCondBr(Builder.CreateICmpULT(Idx,
> +                                  llvm::ConstantInt::get(Idx->getType(), 
> Size)),
> +                                   Cont, getAbortBB());
> +              EmitBlock(Cont);
> +            }

Both sides of this if statment are functionally identical.

-Eli

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to