[llvm] r297014 - New Test-Case for Region Analysis

[Tobias Grosser via llvm-commits]

Hi Bruno,

should have already been fixed in 297020. I assumed the patch is too
simple and I missed a minor mis-spelling.

Best,
Tobias

On Mon, Mar 6, 2017, at 07:18 PM, Bruno Cardoso Lopes via llvm-commits
wrote:
> Hi Tobias,
> 
> Seems like this test is now failing:
> http://green.lab.llvm.org/green/job/Verify-Machineinstrs_AArch64/4982/consoleFull#-90502495649ba4694-19c4-4d7e-bec5-911270d8a58c
> 
> Can you take a look?
> 
> Thanks,
> 
> On Mon, Mar 6, 2017 at 7:20 AM, Tobias Grosser via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
> > Author: grosser
> > Date: Mon Mar  6 09:20:38 2017
> > New Revision: 297014
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=297014&view=rev
> > Log:
> > New Test-Case for Region Analysis
> >
> > While working on improvements to region info analysis, this test case caused an
> > incorrect region bb2 => bb3 to be detected.
> >
> > Reviewers: grosser
> >
> > Contributed-by: Nandini Singhal <cs15mtech01004 at iith.ac.in>
> >
> > Subscribers: llvm-commits
> >
> > Differential Revision: https://reviews.llvm.org/D30652
> >
> > Added:
> >     llvm/trunk/test/Analysis/RegionInfo/outgoing_edge_1.ll
> >
> > Added: llvm/trunk/test/Analysis/RegionInfo/outgoing_edge_1.ll
> > URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/RegionInfo/outgoing_edge_1.ll?rev=297014&view=auto
> > ==============================================================================
> > --- llvm/trunk/test/Analysis/RegionInfo/outgoing_edge_1.ll (added)
> > +++ llvm/trunk/test/Analysis/RegionInfo/outgoing_edge_1.ll Mon Mar  6 09:20:38 2017
> > @@ -0,0 +1,39 @@
> > +; REQUIRES: asserts
> > +; RUN: opt -regions -analyze < %s | FileCheck %s
> > +; RUN: opt < %s -passes='print<regions>' 2>&1 | FileCheck %s
> > +
> > +; While working on improvements to region info analysis, this test
> > +; case caused an incorrect region bb2 => bb3 to be detected.
> > +
> > +define internal i8 @main_read() nounwind {
> > +bb:
> > +   br label %bb1
> > +
> > +bb1:
> > +   br i1 true, label %bb2, label %bb7
> > +
> > +bb2:
> > +  br i1 true, label %bb4, label %bb3
> > +
> > +bb3:
> > +  br i1 true, label %bb4, label %bb8
> > +
> > +bb4:
> > +   br label %bb5
> > +
> > +bb5:
> > +   br label %bb6
> > +
> > +bb6:
> > +   br label %bb1
> > +
> > +bb7:
> > +   br label %bb5
> > +
> > +bb8:
> > +   ret i8 1
> > +}
> > +
> > +; CHECK:    [0] bb => <Function Return>
> > +; CHECK-NEXT: [1] bb1 => bb8
> > +; CHECK-NEXT: End Region tree
> >
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at lists.llvm.org
> > http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
> 
> 
> 
> -- 
> Bruno Cardoso Lopes
> http://www.brunocardoso.cc
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits