On Thu, Oct 19, 2017 at 9:31 AM, Tom de Vries <tom_devr...@mentor.com> wrote: > On 10/09/2017 03:34 PM, Richard Biener wrote: >> >> On Thu, Oct 5, 2017 at 3:16 PM, Bin Cheng <bin.ch...@arm.com> wrote: >>> >>> Hi, >>> Function generate_loops_for_partition chooses arbitrary path when >>> removing exit >>> condition not in partition. This is fine for now because it's impossible >>> to have >>> loop exit condition in case of innermost distribution. After extending >>> to loop >>> nest distribution, we must choose exit edge/path for inner loop's exit >>> condition, >>> otherwise an infinite empty loop will be generated. Test case added. >>> >>> Bootstrap and test in patch set on x86_64 and AArch64, is it OK? >> >> >> Ok. >> >> Richard. >> >>> Thanks, >>> bin >>> 2017-10-04 Bin Cheng <bin.ch...@arm.com> >>> >>> * tree-loop-distribution.c (generate_loops_for_partition): >>> Remove >>> inner loop's exit stmt by making it always exit the loop, >>> otherwise >>> we would generate an infinite empty loop. >>> >>> gcc/testsuite/ChangeLog >>> 2017-10-04 Bin Cheng <bin.ch...@arm.com> >>> >>> * gcc.dg/tree-ssa/ldist-27.c: New test. > > > Hi, > > I've committed patch below to specify the stack size requirements of this > test-case (fixing the test failure for nvptx). Hi, Maybe we can simply make the structure a global variable?
Thanks, bin > > Does it make sense to trim down the test-case using #ifdef STACK_SIZE? > > Thanks, > - Tom