[LLVMdev] Small patch for BasicAliasAnalysis.cpp

Evan Cheng evan.cheng at apple.com
Wed Sep 5 14:36:24 PDT 2007


Applied thanks!

Evan

On Sep 3, 2007, at 5:08 AM, Wojciech Matyjewicz wrote:

> Hi!
>
> I think there is a break missing in switch statement of
> AddressMightEscape function. It causes redundant recurrent call.
>
> Wojtek
>
>
> Index: lib/Analysis/BasicAliasAnalysis.cpp
> ===================================================================
> --- lib/Analysis/BasicAliasAnalysis.cpp	(revision 41682)
> +++ lib/Analysis/BasicAliasAnalysis.cpp	(working copy)
> @@ -220,6 +220,7 @@
>     case Instruction::GetElementPtr:
>       if (AddressMightEscape(I))
>         return true;
> +      break; // next use.
>     case Instruction::BitCast:
>       if (!isa<PointerType>(I->getType()))
>         return true;
>
>
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev




More information about the llvm-dev mailing list