This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][debug] Fix pre_dec handling in vartrack
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Rainer Orth <ro at CeBiTec dot Uni-Bielefeld dot DE>
- Cc: Tom de Vries <tdevries at suse dot de>, gcc-patches at gcc dot gnu dot org, Jim Wilson <wilson at tuliptree dot org>, Jason Merrill <jason at redhat dot com>, Cary Coutant <ccoutant at gmail dot com>
- Date: Tue, 17 Jul 2018 11:22:30 +0200
- Subject: Re: [PATCH][debug] Fix pre_dec handling in vartrack
- References: <20180715212156.z2fjckukdui5ackf@delia> <20180716072410.GY7166@tucnak> <20180716090206.GA7166@tucnak> <yddefg2p6fm.fsf@CeBiTec.Uni-Bielefeld.DE>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Tue, Jul 17, 2018 at 11:17:33AM +0200, Rainer Orth wrote:
> it turns out the test FAILs on i386-pc-soalris2.11 with -m64: the dump
> doesn't even contain argp: However, adding -fomit-frame-pointer makes
> it work, and still PASSes on x86_64-pc-linux-gnu.
>
> Ok for for mainline?
Ok, thanks.
> 2018-07-17 Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE>
>
> * gcc.target/i386/vartrack-1.c (dg-options): Add
> -fomit-frame-pointer.
>
> diff --git a/gcc/testsuite/gcc.target/i386/vartrack-1.c b/gcc/testsuite/gcc.target/i386/vartrack-1.c
> --- a/gcc/testsuite/gcc.target/i386/vartrack-1.c
> +++ b/gcc/testsuite/gcc.target/i386/vartrack-1.c
> @@ -1,5 +1,5 @@
> /* { dg-require-effective-target lp64 } */
> -/* { dg-options "-O1 -g -fdump-rtl-vartrack-details-slim" } */
> +/* { dg-options "-O1 -g -fomit-frame-pointer -fdump-rtl-vartrack-details-slim" } */
>
> static volatile int vv = 1;
>
Jakub