Re: [PATCH] Replace a SRA FIXME with an assert

2012-03-23 Thread Martin Jambor
On Wed, Mar 21, 2012 at 08:46:49AM +0100, Richard Guenther wrote: > On Tue, 20 Mar 2012, Martin Jambor wrote: > > > Hi, > > > > On Tue, Mar 20, 2012 at 04:08:31PM +0100, Richard Guenther wrote: > > > On Tue, 20 Mar 2012, Martin Jambor wrote: > > > > > > > Hi, > > > > > > > > this patch which re

Re: [PATCH] Replace a SRA FIXME with an assert

2012-03-21 Thread Richard Guenther
On Tue, 20 Mar 2012, Martin Jambor wrote: > Hi, > > On Tue, Mar 20, 2012 at 04:08:31PM +0100, Richard Guenther wrote: > > On Tue, 20 Mar 2012, Martin Jambor wrote: > > > > > Hi, > > > > > > this patch which removes one of only two FIXMEs in tree-sra.c has been > > > sitting in my patch queue fo

Re: [PATCH] Replace a SRA FIXME with an assert

2012-03-20 Thread Martin Jambor
Hi, On Tue, Mar 20, 2012 at 04:08:31PM +0100, Richard Guenther wrote: > On Tue, 20 Mar 2012, Martin Jambor wrote: > > > Hi, > > > > this patch which removes one of only two FIXMEs in tree-sra.c has been > > sitting in my patch queue for over a year. Yesterday I noticed it > > there, bootstrappe

Re: [PATCH] Replace a SRA FIXME with an assert

2012-03-20 Thread Richard Guenther
On Tue, 20 Mar 2012, Martin Jambor wrote: > Hi, > > this patch which removes one of only two FIXMEs in tree-sra.c has been > sitting in my patch queue for over a year. Yesterday I noticed it > there, bootstrapped and tested it on x86_64-linux and it passed. > > I'd like to either commit it or j

[PATCH] Replace a SRA FIXME with an assert

2012-03-20 Thread Martin Jambor
Hi, this patch which removes one of only two FIXMEs in tree-sra.c has been sitting in my patch queue for over a year. Yesterday I noticed it there, bootstrapped and tested it on x86_64-linux and it passed. I'd like to either commit it or just remove the comment, if there likely still are size in