Re: GCC 4.2.0 Status Report (2007-03-04)

2007-03-09 Thread Mark Mitchell
Andrew MacLeod wrote:

 There are a handful I've been involved with which are labeled as
 4.0/4.1/4.2/4.3 regressions which I don't see ever being fixed in 4.0
 through 4.2. There is perhaps some hope for 4.3, but 4.4 is the more
 likely case. They require new development work that I think is unlikely
 to ever be backported to these releases for just these testcases.   

Thank you very much for this email.  I apologize for not acknowledging
this earlier.

 The PRs are 
 - 21596  I made a simple attempt at this one, but the simple approach
 isn't going to work.  
 - 23200  fixed in 4.3 by the new version of TER. Unlikely to port the
 entire TER rewrite back to 4.2 at this stage (it could be done tho :-).

This was still marked as a bug in 4.3 in the Summary section of the
bug.  Given that this fixed, I've removed the 4.3 marker there.  In
future, when you fix a bug, please remove the release from the Summary
section, if you remember.  (I forget too...)

 - 27986  This is in fact a variation of the issue in 21596, except this
 one crosses basic blocks. It will require all new work to get this case.
 - 27877  Mark Shinwell applied a patch for this to 4.3. I pinged him
 about applying it to 4.2. If I haven't heard from him by thursday, I
 will apply it friday.

I think you meant 29877?  In any case, thank you!

 I would say you can mark the first 3 as 'will not fix', or whatever you
 do to indicate they aren't going to be resolved in 4.2. 

I'm going to leave them open, but I've added a link to your message to
the audit trails.  Optimistically, someone might miraculously fix them,
but it's very helpful to know that that's going to be unlikley.

Thanks,

-- 
Mark Mitchell
CodeSourcery
[EMAIL PROTECTED]
(650) 331-3385 x713


Re: GCC 4.2.0 Status Report (2007-03-04)

2007-03-07 Thread Manuel López-Ibáñez

On 06 Mar 2007 21:48:14 -0600, Gabriel Dos Reis [EMAIL PROTECTED] wrote:

Manuel López-Ibáñez [EMAIL PROTECTED] writes:

| On 06/03/07, Mark Mitchell [EMAIL PROTECTED] wrote:
|  Manuel López-Ibáñez wrote:
|   On 05/03/07, Mark Mitchell [EMAIL PROTECTED] wrote:
|   After reviewing all of the traffic[1] that stemmed from my previous
|   status report, I've decided that, indeed, it makes sense to steam ahead
|   with GCC 4.2.0 based on current GCC 4.2.0 release branch.
|  
|  
|   I ask special permission to apply this patch:
|  
|   http://gcc.gnu.org/ml/gcc-patches/2007-02/msg01461.html
|  
|   to GCC 4.2.0. This patch is trivial and it has been in GCC 4.3 for a
|   few weeks already.
| 
|  This patch is OK for 4.2.0, on the grounds that the -Walways-true option
|  has not yet appeared in any FSF release, and we don't want to release
|  the option with semantics different than what are eventually intended.
| 
|  Thanks,
| 
|
| As far as I know, it was introduced in GCC 4.2 revision 108489.
|
| Also:
|
| $ svn log svn://gcc.gnu.org/svn/gcc/branches/gcc-4_2-branch | grep -n
| -e Walways.true
| 63424:  * gcc.dg/20031012-1.c: Add -Walways-true option.
| 63978:  * doc/invoke.texi (Warning Options): Document -Walways-true being
| 64532:  * common.opt (Walways-true): New option.
|
| while
|
| $ svn log svn://gcc.gnu.org/svn/gcc/branches/gcc-4_1-branch | grep -n
| -e Walways.true
|
| shows nothing.

I strongly encourage the removal brefore 4.2 gets out.



I am bootstrapping and regression testing the backport of the patch
and I will commit it and post it here as soon as it finishes.

Should we mention Waddress in the GCC 4.2 release notes? Neither
Wliteral-string-comparison nor Walways-true were mention. That is,
should I prepare a patch?

Cheers,

Manuel.


Re: GCC 4.2.0 Status Report (2007-03-04)

2007-03-07 Thread Gabriel Dos Reis
On Wed, 7 Mar 2007, Manuel López-Ibáñez wrote:

| Should we mention Waddress in the GCC 4.2 release notes?

Proper documentation is sufficient I believe.

-- Gaby


Re: GCC 4.2.0 Status Report (2007-03-04)

2007-03-07 Thread Joe Buck
On Wed, Mar 07, 2007 at 04:13:08AM -0600, Gabriel Dos Reis wrote:
 On Wed, 7 Mar 2007, Manuel López-Ibáñez wrote:
 
 | Should we mention Waddress in the GCC 4.2 release notes?
 
 Proper documentation is sufficient I believe.

Or the release notes could just say something like:

 * New warning options: -Wfoo, -Wbar (see the GCC manual for details)


Re: GCC 4.2.0 Status Report (2007-03-04)

2007-03-06 Thread Manuel López-Ibáñez

On 05/03/07, Manuel López-Ibáñez [EMAIL PROTECTED] wrote:

On 05/03/07, Mark Mitchell [EMAIL PROTECTED] wrote:
 After reviewing all of the traffic[1] that stemmed from my previous
 status report, I've decided that, indeed, it makes sense to steam ahead
 with GCC 4.2.0 based on current GCC 4.2.0 release branch.


I ask special permission to apply this patch:

http://gcc.gnu.org/ml/gcc-patches/2007-02/msg01461.html

to GCC 4.2.0. This patch is trivial and it has been in GCC 4.3 for a
few weeks already.

It replaces two options that are introduced in GCC 4.2 with a single
switch -Waddress. I think we should replace them before releasing, to
minimise the effect of the change in users.



Mark,

Do you have an opinion about this? Whatever decision is made, I would
like to delete it from my TODO list.

Cheers,

Manuel.


Re: GCC 4.2.0 Status Report (2007-03-04)

2007-03-06 Thread Mark Mitchell
Manuel López-Ibáñez wrote:
 On 05/03/07, Mark Mitchell [EMAIL PROTECTED] wrote:
 After reviewing all of the traffic[1] that stemmed from my previous
 status report, I've decided that, indeed, it makes sense to steam ahead
 with GCC 4.2.0 based on current GCC 4.2.0 release branch.

 
 I ask special permission to apply this patch:
 
 http://gcc.gnu.org/ml/gcc-patches/2007-02/msg01461.html
 
 to GCC 4.2.0. This patch is trivial and it has been in GCC 4.3 for a
 few weeks already.

This patch is OK for 4.2.0, on the grounds that the -Walways-true option
has not yet appeared in any FSF release, and we don't want to release
the option with semantics different than what are eventually intended.

Thanks,

-- 
Mark Mitchell
CodeSourcery
[EMAIL PROTECTED]
(650) 331-3385 x713


Re: GCC 4.2.0 Status Report (2007-03-04)

2007-03-06 Thread Manuel López-Ibáñez

On 06/03/07, Mark Mitchell [EMAIL PROTECTED] wrote:

Manuel López-Ibáñez wrote:
 On 05/03/07, Mark Mitchell [EMAIL PROTECTED] wrote:
 After reviewing all of the traffic[1] that stemmed from my previous
 status report, I've decided that, indeed, it makes sense to steam ahead
 with GCC 4.2.0 based on current GCC 4.2.0 release branch.


 I ask special permission to apply this patch:

 http://gcc.gnu.org/ml/gcc-patches/2007-02/msg01461.html

 to GCC 4.2.0. This patch is trivial and it has been in GCC 4.3 for a
 few weeks already.

This patch is OK for 4.2.0, on the grounds that the -Walways-true option
has not yet appeared in any FSF release, and we don't want to release
the option with semantics different than what are eventually intended.

Thanks,



As far as I know, it was introduced in GCC 4.2 revision 108489.

Also:

$ svn log svn://gcc.gnu.org/svn/gcc/branches/gcc-4_2-branch | grep -n
-e Walways.true
63424:  * gcc.dg/20031012-1.c: Add -Walways-true option.
63978:  * doc/invoke.texi (Warning Options): Document -Walways-true being
64532:  * common.opt (Walways-true): New option.

while

$ svn log svn://gcc.gnu.org/svn/gcc/branches/gcc-4_1-branch | grep -n
-e Walways.true

shows nothing.

Cheers,

Manuel.


Re: GCC 4.2.0 Status Report (2007-03-04)

2007-03-06 Thread Gabriel Dos Reis
Manuel López-Ibáñez [EMAIL PROTECTED] writes:

| On 06/03/07, Mark Mitchell [EMAIL PROTECTED] wrote:
|  Manuel López-Ibáñez wrote:
|   On 05/03/07, Mark Mitchell [EMAIL PROTECTED] wrote:
|   After reviewing all of the traffic[1] that stemmed from my previous
|   status report, I've decided that, indeed, it makes sense to steam ahead
|   with GCC 4.2.0 based on current GCC 4.2.0 release branch.
|  
|  
|   I ask special permission to apply this patch:
|  
|   http://gcc.gnu.org/ml/gcc-patches/2007-02/msg01461.html
|  
|   to GCC 4.2.0. This patch is trivial and it has been in GCC 4.3 for a
|   few weeks already.
| 
|  This patch is OK for 4.2.0, on the grounds that the -Walways-true option
|  has not yet appeared in any FSF release, and we don't want to release
|  the option with semantics different than what are eventually intended.
| 
|  Thanks,
| 
| 
| As far as I know, it was introduced in GCC 4.2 revision 108489.
| 
| Also:
| 
| $ svn log svn://gcc.gnu.org/svn/gcc/branches/gcc-4_2-branch | grep -n
| -e Walways.true
| 63424:  * gcc.dg/20031012-1.c: Add -Walways-true option.
| 63978:  * doc/invoke.texi (Warning Options): Document -Walways-true being
| 64532:  * common.opt (Walways-true): New option.
| 
| while
| 
| $ svn log svn://gcc.gnu.org/svn/gcc/branches/gcc-4_1-branch | grep -n
| -e Walways.true
| 
| shows nothing.

I strongly encourage the removal brefore 4.2 gets out.

-- Gaby


Re: GCC 4.2.0 Status Report (2007-03-04)

2007-03-05 Thread Brooks Moses

Mark Mitchell wrote:

However, I do think that it's important to eliminate some of the 139
open P2 and P1 regressions [2], especially those P1 regressions which
did not appear in GCC 4.1.x.


133, not 139.  Your search url returns six P3 bugs, one of which (29441) 
is not even a regression.


Does that count as six for my tally?  :)

- Brooks



Re: GCC 4.2.0 Status Report (2007-03-04)

2007-03-05 Thread Manuel López-Ibáñez

On 05/03/07, Mark Mitchell [EMAIL PROTECTED] wrote:

After reviewing all of the traffic[1] that stemmed from my previous
status report, I've decided that, indeed, it makes sense to steam ahead
with GCC 4.2.0 based on current GCC 4.2.0 release branch.



I ask special permission to apply this patch:

http://gcc.gnu.org/ml/gcc-patches/2007-02/msg01461.html

to GCC 4.2.0. This patch is trivial and it has been in GCC 4.3 for a
few weeks already.

It replaces two options that are introduced in GCC 4.2 with a single
switch -Waddress. I think we should replace them before releasing, to
minimise the effect of the change in users.

In addition, there are two trivial doc fixes for GCC 4.2, one in
invoke.texi and another in opts.c, where it says -Werror- and it
should say -Werror=
Can I commit them as obvious or do I need to submit a patch?

Cheers,

Manuel.


Re: GCC 4.2.0 Status Report (2007-03-04)

2007-03-05 Thread Gerald Pfeifer
On Mon, 5 Mar 2007, Manuel López-Ibáñez wrote:
 In addition, there are two trivial doc fixes for GCC 4.2, one in
 invoke.texi and another in opts.c, where it says -Werror- and it
 should say -Werror=
 Can I commit them as obvious or do I need to submit a patch?

Both. :-)  That is, go ahead and commit the patch but post it to the
gcc-patches list as well.

Gerald