This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix PR bootstrap/63995
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Ilya Enkovich <enkovich dot gnu at gmail dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Wed, 26 Nov 2014 13:46:20 +0100
- Subject: Re: [PATCH] Fix PR bootstrap/63995
- Authentication-results: sourceware.org; auth=none
- References: <20141126124146 dot GC40854 at msticlxl57 dot ims dot intel dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Wed, Nov 26, 2014 at 03:41:46PM +0300, Ilya Enkovich wrote:
> Hi,
>
> This patch makes optimization for bounds lifetime reduction to ignore
> debug stetments. This fixes stage2 and stage3 comparision for
> instrumented boostrap. OK for trunk?
Please add a small testcase (with -fcompare-debug -fcheck-pointer-bounds (or what
other options you need to reproduce it) that fails without the patch and
succeeds with it.
> 2014-11-26 Ilya Enkovich <ilya.enkovich@intel.com>
>
> PR bootstrap/63995
> * tree-chkp-opt.c (chkp_reduce_bounds_lifetime): Ignore
> debug statement when searching for a new position for
> bounds load/creation statement.
>
>
> diff --git a/gcc/tree-chkp-opt.c b/gcc/tree-chkp-opt.c
> index ff390d7..92e0694 100644
> --- a/gcc/tree-chkp-opt.c
> +++ b/gcc/tree-chkp-opt.c
> @@ -1175,6 +1175,9 @@ chkp_reduce_bounds_lifetime (void)
>
> FOR_EACH_IMM_USE_STMT (use_stmt, use_iter, op)
> {
> + if (is_gimple_debug (use_stmt))
> + continue;
> +
> if (dom_bb &&
> dominated_by_p (CDI_DOMINATORS,
> dom_bb, gimple_bb (use_stmt)))
Jakub