Re: PATCH: Correct alloca length in dump_gimple_bb_header

2012-10-30 Thread Jakub Jelinek
On Mon, Oct 29, 2012 at 10:53:42PM -0700, H.J. Lu wrote: > ---2012-10-30 H.J. Lu > > * gimple-pretty-print.c (dump_gimple_bb_header): Avoid alloca. Ok, thanks. Jakub

Re: PATCH: Correct alloca length in dump_gimple_bb_header

2012-10-29 Thread H.J. Lu
On Mon, Oct 29, 2012 at 6:01 AM, H.J. Lu wrote: > On Mon, Oct 29, 2012 at 5:18 AM, Jakub Jelinek wrote: >> On Mon, Oct 29, 2012 at 05:06:01AM -0700, H.J. Lu wrote: >>> When indent == 0, we call alloca with -1 bytes. This patch changes >>> it to indent + 1. This is a trunk only regression. OK t

Re: PATCH: Correct alloca length in dump_gimple_bb_header

2012-10-29 Thread H.J. Lu
On Mon, Oct 29, 2012 at 5:18 AM, Jakub Jelinek wrote: > On Mon, Oct 29, 2012 at 05:06:01AM -0700, H.J. Lu wrote: >> When indent == 0, we call alloca with -1 bytes. This patch changes >> it to indent + 1. This is a trunk only regression. OK to install? > >> 2012-10-29 H.J. Lu >> >> * gi

Re: PATCH: Correct alloca length in dump_gimple_bb_header

2012-10-29 Thread Jakub Jelinek
On Mon, Oct 29, 2012 at 05:06:01AM -0700, H.J. Lu wrote: > When indent == 0, we call alloca with -1 bytes. This patch changes > it to indent + 1. This is a trunk only regression. OK to install? > 2012-10-29 H.J. Lu > > * gimple-pretty-print.c (dump_gimple_bb_header): Correct alloca >

PATCH: Correct alloca length in dump_gimple_bb_header

2012-10-29 Thread H.J. Lu
Hi, When indent == 0, we call alloca with -1 bytes. This patch changes it to indent + 1. This is a trunk only regression. OK to install? Thanks. H.J. --- 2012-10-29 H.J. Lu * gimple-pretty-print.c (dump_gimple_bb_header): Correct alloca length. diff --git a/gcc/gimple-pr