This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Improve debugging info (PR debug/53948 P1 regression)
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Jeff Law <law at redhat dot com>
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 8 Feb 2013 18:55:08 +0100
- Subject: Re: [PATCH] Improve debugging info (PR debug/53948 P1 regression)
- References: <51152DBB.2090007@redhat.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Fri, Feb 08, 2013 at 09:54:19AM -0700, Jeff Law wrote:
> PR debug/53948
> * emit-rtl.c (reg_is_parm_p): New function.
> * regs.h (reg_is_parm_p): New prototype.
> * ira-conflicts.c (ira_build_conflicts): Allow parameters in
> callee-clobbered registers.
This looks good to me.
> * gcc.dg/debug/dwarf2/pr53948.c: New test.
But the testcase is problematic.
> --- /dev/null
> +++ b/gcc/testsuite/gcc.dg/debug/dwarf2/pr53948.c
> @@ -0,0 +1,10 @@
> +/* Test that we emit a .line directive for the line
> + with local variable initializations. */
> +/* { dg-options "-O0 -g" } */
> +/* { dg-final { scan-assembler ".loc 1 8 0" } } */
It will fail on any target which doesn't support .loc
directives.
I'd say better would be to scan the -fdump-rtl-final
for pr53948.c:8 regex.
Jakub