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

[Bruno Cardoso Lopes via llvm-commits]

Thanks Tobias!

On Mon, Mar 6, 2017 at 10:21 AM, Tobias Grosser <tobias at grosser.es> wrote:
> 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



-- 
Bruno Cardoso Lopes
http://www.brunocardoso.cc