This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, ARM] Fix ICE due to out of bound.
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Ramana Radhakrishnan <ramrad01 at arm dot com>
- Cc: Zhenqiang Chen <zhenqiang dot chen at linaro dot org>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Richard Earnshaw <Richard dot Earnshaw at arm dot com>, Richard Biener <rguenther at suse dot de>
- Date: Wed, 19 Mar 2014 10:50:33 +0100
- Subject: Re: [PATCH, ARM] Fix ICE due to out of bound.
- Authentication-results: sourceware.org; auth=none
- References: <CACgzC7BOpXKS_4mVk1-p-Xc7Fh9nyC2rjAWZDopMY1u+OhYOVA at mail dot gmail dot com> <53296790 dot 9040007 at arm dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Wed, Mar 19, 2014 at 09:46:56AM +0000, Ramana Radhakrishnan wrote:
> On 03/19/14 08:42, Zhenqiang Chen wrote:
> >ICE when compiling gcc.target/arm/neon-modes-3.c with "-g" in
> >arm_dwarf_register_span since parts[8] is out of bound for XImode.
> >GET_MODE_SIZE (XImode) / 4 is 16. "rtx parts[8]" can not hold all the
> >registers.
> >
> >According to arm-modes.def, 16 should be the biggest number. So the
> >patch updates parts to
> >
> >rtx parts[16];
> >
> >Bootstrap and no make check regression on ARM Chrome book.
> >
> >OK for trunk?
> >
>
> It may be time in 4.10 or 5.0 (whatever we call it :)), to deal with
> the FIXME in arm_dwarf_register_span to deal with DW_OP_piece. I'm
> surprised that it's taken so long to hit this.
>
> This is OK for stage4 - it looks sane to me but this needs an RM ack
> before applying.
Ok.
Jakub