search for: exitbr

Attached 2012/2/8 Marcello Maggioni <hayarms at gmail.com>: > Mmm, sorry, the patch I posted crashes if ExitBr is null (which it may > be ...) , this one should be ok (and passess all the ScalarEvolution > tests in LLVM): > > diff --git a/lib/Analysis/ScalarEvolution.cpp b/lib/Analysis/ScalarEvolution.cpp > index daf7742..b10fab2 100644 > --- a/lib/Analysis/ScalarEvolution.cpp > +++ b/l...

...42..fcbaffe 100644 --- a/lib/Analysis/ScalarEvolution.cpp +++ b/lib/Analysis/ScalarEvolution.cpp @@ -4293,6 +4293,11 @@ ScalarEvolution::ComputeExitLimit(const Loop *L, BasicBlock *ExitingBlock) { // // FIXME: we should be able to handle switch instructions (with a single exit) BranchInst *ExitBr = dyn_cast<BranchInst>(ExitingBlock->getTerminator()); + BranchInst* BrFirstSucc = dyn_cast<BranchInst>(ExitBr-> + getSuccessor(0)->getTerminator()); + BranchInst* BrSecondSucc = dyn_cast<BranchInst>(ExitBr-> +...

Mmm, sorry, the patch I posted crashes if ExitBr is null (which it may be ...) , this one should be ok (and passess all the ScalarEvolution tests in LLVM): diff --git a/lib/Analysis/ScalarEvolution.cpp b/lib/Analysis/ScalarEvolution.cpp index daf7742..b10fab2 100644 --- a/lib/Analysis/ScalarEvolution.cpp +++ b/lib/Analysis/ScalarEvolution.cpp @@...

...es isn't indented > properly. > > Nick > > On 8 February 2012 12:05, Marcello Maggioni <hayarms at gmail.com> wrote: >> >> Attached >> >> 2012/2/8 Marcello Maggioni <hayarms at gmail.com>: >> > Mmm, sorry, the patch I posted crashes if ExitBr is null (which it may >> > be ...) , this one should be ok (and passess all the ScalarEvolution >> > tests in LLVM): >> > >> > diff --git a/lib/Analysis/ScalarEvolution.cpp >> > b/lib/Analysis/ScalarEvolution.cpp >> > index daf7742..b10fab2 1006...

...the // in your comments. One of the comment lines isn't indented properly. Nick On 8 February 2012 12:05, Marcello Maggioni <hayarms at gmail.com> wrote: > Attached > > 2012/2/8 Marcello Maggioni <hayarms at gmail.com>: > > Mmm, sorry, the patch I posted crashes if ExitBr is null (which it may > > be ...) , this one should be ok (and passess all the ScalarEvolution > > tests in LLVM): > > > > diff --git a/lib/Analysis/ScalarEvolution.cpp > b/lib/Analysis/ScalarEvolution.cpp > > index daf7742..b10fab2 100644 > > --- a/lib/Analys...

...t; >> > On 8 February 2012 12:05, Marcello Maggioni <hayarms at gmail.com> wrote: >> >> >> >> Attached >> >> >> >> 2012/2/8 Marcello Maggioni <hayarms at gmail.com>: >> >> > Mmm, sorry, the patch I posted crashes if ExitBr is null (which it >> >> > may >> >> > be ...) , this one should be ok (and passess all the ScalarEvolution >> >> > tests in LLVM): >> >> > >> >> > diff --git a/lib/Analysis/ScalarEvolution.cpp >> >> > b/lib/An...

...; > > Nick > > > > On 8 February 2012 12:05, Marcello Maggioni <hayarms at gmail.com> wrote: > >> > >> Attached > >> > >> 2012/2/8 Marcello Maggioni <hayarms at gmail.com>: > >> > Mmm, sorry, the patch I posted crashes if ExitBr is null (which it may > >> > be ...) , this one should be ok (and passess all the ScalarEvolution > >> > tests in LLVM): > >> > > >> > diff --git a/lib/Analysis/ScalarEvolution.cpp > >> > b/lib/Analysis/ScalarEvolution.cpp > >> &g...

...8 February 2012 12:05, Marcello Maggioni <hayarms at gmail.com> wrote: >>> >> >>> >> Attached >>> >> >>> >> 2012/2/8 Marcello Maggioni <hayarms at gmail.com>: >>> >> > Mmm, sorry, the patch I posted crashes if ExitBr is null (which it >>> >> > may >>> >> > be ...) , this one should be ok (and passess all the ScalarEvolution >>> >> > tests in LLVM): >>> >> > >>> >> > diff --git a/lib/Analysis/ScalarEvolution.cpp >>&gt...

...5, Marcello Maggioni <hayarms at gmail.com> wrote: >>>> >> >>>> >> Attached >>>> >> >>>> >> 2012/2/8 Marcello Maggioni <hayarms at gmail.com>: >>>> >> > Mmm, sorry, the patch I posted crashes if ExitBr is null (which it >>>> >> > may >>>> >> > be ...) , this one should be ok (and passess all the ScalarEvolution >>>> >> > tests in LLVM): >>>> >> > >>>> >> > diff --git a/lib/Analysis/ScalarEvolu...