Wojciech Matyjewicz
2007-Sep-03 12:08 UTC
[LLVMdev] Small patch for BasicAliasAnalysis.cpp
Hi! I think there is a break missing in switch statement of AddressMightEscape function. It causes redundant recurrent call. Wojtek -------------- next part -------------- An embedded and charset-unspecified text was scrubbed... Name: BasicAliasAnalysis.patch URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20070903/481046a5/attachment.ksh>
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
Reasonably Related Threads
- [LLVMdev] BasicAliasAnalysis and out-of-bound GEP indices
- [LLVMdev] BasicAliasAnalysis and out-of-bound GEP indices
- [LLVMdev] BasicAliasAnalysis and out-of-bound GEP indices
- [LLVMdev] Data Structure Analysis
- [LLVMdev] BasicAliasAnalysis and out-of-bound GEP indices