RE: [PATCH][GCC][AArch64] Cleanup the AArch64 testsuite when stack-clash is on [Patch (7/7)]

[Tamar Christina <Tamar dot Christina at arm dot com>]

Hi All,

I'm looking for permission to backport this patch to the GCC-8 branch
to fix PR86486.

OK for backport?

Thanks,
Tamar

> -----Original Message-----
> From: gcc-patches-owner@gcc.gnu.org <gcc-patches-owner@gcc.gnu.org>
> On Behalf Of Tamar Christina
> Sent: Friday, September 28, 2018 17:36
> To: Jeff Law <law@redhat.com>; gcc-patches@gcc.gnu.org
> Cc: nd <nd@arm.com>; James Greenhalgh <James.Greenhalgh@arm.com>;
> Richard Earnshaw <Richard.Earnshaw@arm.com>; Marcus Shawcroft
> <Marcus.Shawcroft@arm.com>
> Subject: RE: [PATCH][GCC][AArch64] Cleanup the AArch64 testsuite when
> stack-clash is on [Patch (7/7)]
> 
> Hi All,
> 
> This is a minor update to fix stack-check-12.c for ilp32.
> The datatype would be too small on ilp32 to require a probe.
> 
> gcc/testsuite/
> 2018-08-28  Tamar Christina  <tamar.christina@arm.com>
> 
> 	PR target/86486
> 	* gcc.dg/pr82788.c: Skip for AArch64.
> 	* gcc.dg/guality/vla-1.c: Turn off stack-clash.
> 	* gcc.target/aarch64/subsp.c: Likewise.
> 	* gcc.dg/params/blocksort-part.c: Skip stack-clash checks
> 	on AArch64.
> 	* gcc.dg/stack-check-10.c: Add AArch64 specific checks.
> 	* gcc.dg/stack-check-12.c: ILP32 fixup.
> 	* gcc.dg/stack-check-5.c: Add AArch64 specific checks.
> 	* gcc.dg/stack-check-6a.c: Skip on AArch64, we don't support this.
> 	* testsuite/lib/target-supports.exp
> 	(check_effective_target_frame_pointer_for_non_leaf): AArch64
> does not
> 	require frame pointer for non-leaf functions.
> 
> As before I assume the OK still applies as the change is trivial so will commit
> this with the rest once all approved.
> 
> Thanks,
> Tamar
> 
> > -----Original Message-----
> > From: Jeff Law <law@redhat.com>
> > Sent: Tuesday, August 28, 2018 19:13
> > To: Tamar Christina <Tamar.Christina@arm.com>; gcc-patches@gcc.gnu.org
> > Cc: nd <nd@arm.com>; James Greenhalgh <James.Greenhalgh@arm.com>;
> > Richard Earnshaw <Richard.Earnshaw@arm.com>; Marcus Shawcroft
> > <Marcus.Shawcroft@arm.com>
> > Subject: Re: [PATCH][GCC][AArch64] Cleanup the AArch64 testsuite when
> > stack-clash is on [Patch (7/7)]
> >
> > On 08/28/2018 06:18 AM, Tamar Christina wrote:
> > > Hi All,
> > >
> > > Since this patch series now contains SVE support I am removing the
> > > changes to the SVE tests in this patch series.  I assume the OK
> > > still stands as the only change here is undoing updates to three files.
> > >
> > > Thanks,
> > > Tamar
> > >
> > > gcc/testsuite/
> > > 2018-08-28  Tamar Christina  <tamar.christina@arm.com>
> > >
> > > 	PR target/86486
> > > 	* gcc.dg/pr82788.c: Skip for AArch64.
> > > 	* gcc.dg/guality/vla-1.c: Turn off stack-clash.
> > > 	* gcc.target/aarch64/subsp.c: Likewise.
> > > 	* gcc.dg/params/blocksort-part.c: Skip stack-clash checks
> > > 	on AArch64.
> > > 	* gcc.dg/stack-check-10.c: Add AArch64 specific checks.
> > > 	* gcc.dg/stack-check-5.c: Add AArch64 specific checks.
> > > 	* gcc.dg/stack-check-6a.c: Skip on AArch64, we don't support this.
> > > 	* testsuite/lib/target-supports.exp
> > > 	(check_effective_target_frame_pointer_for_non_leaf): AArch64
> > does not
> > > 	require frame pointer for non-leaf functions.
> > Yea, the OK still applies.
> > jeff