Re: [patch] Remove -fconserve-space

2012-11-04 Thread Jason Merrill
That looks good to me. Jason

Re: [patch] Remove -fconserve-space

2012-11-04 Thread Gerald Pfeifer
Hi there, any comments on my proposed release notes entry below? Gerald On Sun, 26 Aug 2012, Gerald Pfeifer wrote: >> The attached patch removes the -fconserve-space flag, as discussed last >> week. Bootstrapped&tested on powerpc64-unknown-linux-gnu. OK for trunk? > > How about the following f

Re: [patch] Remove -fconserve-space

2012-08-25 Thread Gerald Pfeifer
On Wed, 6 Jun 2012, Steven Bosscher wrote: > The attached patch removes the -fconserve-space flag, as discussed last > week. Bootstrapped&tested on powerpc64-unknown-linux-gnu. OK for trunk? How about the following for the release notes? (Happy to use a different/better rationale. This one's bo

Re: [patch] Remove -fconserve-space

2012-06-06 Thread Jason Merrill
On 06/06/2012 02:06 PM, Steven Bosscher wrote: -// { dg-options "-fconserve-space -fcommon" } +// { dg-options "-fcommon" } I think we can just remove the dg-options; -fcommon has no effect for C++. OK with that change. Jason

[patch] Remove -fconserve-space

2012-06-06 Thread Steven Bosscher
Hello, The attached patch removes the -fconserve-space flag, as discussed last week. Bootstrapped&tested on powerpc64-unknown-linux-gnu. OK for trunk? Ciao! Steven gcc/ * doc/invoke.texi (fconserve-space): Remove documentation. c-family/ * c.opt (fconserve-space): Turn into a no-