From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21666 invoked by alias); 17 Apr 2014 13:13:19 -0000 Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-bugs-owner@gcc.gnu.org Received: (qmail 21626 invoked by uid 48); 17 Apr 2014 13:13:13 -0000 From: "manu at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug c++/55189] enable -Wreturn-type by default Date: Thu, 17 Apr 2014 13:13:00 -0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: c++ X-Bugzilla-Version: 4.6.2 X-Bugzilla-Keywords: diagnostic X-Bugzilla-Severity: normal X-Bugzilla-Who: manu at gcc dot gnu.org X-Bugzilla-Status: NEW X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: cc Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-SW-Source: 2014-04/txt/msg01282.txt.bz2 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D55189 Manuel L=C3=B3pez-Ib=C3=A1=C3=B1ez changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |manu at gcc dot gnu.org --- Comment #9 from Manuel L=C3=B3pez-Ib=C3=A1=C3=B1ez --- (In reply to Sylvestre Ledru from comment #8) > The patches to fix this bug are available here: > https://github.com/sylvestre/gcc > I am doing the legal FSF papers to be able to apply it. Cool! Hopefully it goes quickly. After that, I would suggest that you submit the patch to gcc-patches and CC Jason Merrill and Dodji Seketeli, either of them should be able to approve this. I noticed that your patch does not update doc/invoke.texi to reflect the fa= ct that -Wreturn-type is enabled by default instead of by -Wall (it would be g= reat to automatically generate some parts of the manual from the options description). I also noticed that you did not include -Wmissing-return in your patch as discussed here: http://gcc.gnu.org/ml/gcc/2013-11/msg00288.html (Perhaps you did, and I am using github incorrectly?) I think you will get easier approval if you include it (or send two patches= in the same email) and point out to the previous discussion (in particular Dod= ji, Joseph and Jason's approval of this idea). >>From gcc-bugs-return-449263-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org Thu Apr 17 13:18:49 2014 Return-Path: Delivered-To: listarch-gcc-bugs@gcc.gnu.org Received: (qmail 23953 invoked by alias); 17 Apr 2014 13:18:48 -0000 Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-bugs-owner@gcc.gnu.org Delivered-To: mailing list gcc-bugs@gcc.gnu.org Received: (qmail 23901 invoked by uid 48); 17 Apr 2014 13:18:44 -0000 From: "sylvestre at debian dot org" To: gcc-bugs@gcc.gnu.org Subject: [Bug c++/55189] enable -Wreturn-type by default Date: Thu, 17 Apr 2014 13:18:00 -0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: c++ X-Bugzilla-Version: 4.6.2 X-Bugzilla-Keywords: diagnostic X-Bugzilla-Severity: normal X-Bugzilla-Who: sylvestre at debian dot org X-Bugzilla-Status: NEW X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-SW-Source: 2014-04/txt/msg01283.txt.bz2 Content-length: 1392 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D55189 --- Comment #10 from Sylvestre Ledru --- (In reply to Manuel L=C3=B3pez-Ib=C3=A1=C3=B1ez from comment #9) >=20 > Cool! Hopefully it goes quickly. After that, I would suggest that you sub= mit > the patch to gcc-patches and CC Jason Merrill and Dodji Seketeli, either = of > them should be able to approve this. Sure. Thanks for the suggestion. > I noticed that your patch does not update doc/invoke.texi to reflect the > fact that -Wreturn-type is enabled by default instead of by -Wall (it wou= ld > be great to automatically generate some parts of the manual from the opti= ons > description). I will have a look! > I also noticed that you did not include -Wmissing-return in your patch as > discussed here: http://gcc.gnu.org/ml/gcc/2013-11/msg00288.html >=20=20 > (Perhaps you did, and I am using github incorrectly?) I did... removed it :) I've followed Jason suggestion here: http://gcc.gnu.org/ml/gcc-patches/2014-01/msg01033.html The problem that we were trying to address with -Wmissing-return has been f= ixed (AFAIK) by the commit description in comment #7 > I think you will get easier approval if you include it (or send two patch= es > in the same email) and point out to the previous discussion (in particular > Dodji, Joseph and Jason's approval of this idea). Sure. Thanks for the advice. >>From gcc-bugs-return-449264-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org Thu Apr 17 13:26:35 2014 Return-Path: Delivered-To: listarch-gcc-bugs@gcc.gnu.org Received: (qmail 31425 invoked by alias); 17 Apr 2014 13:26:35 -0000 Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-bugs-owner@gcc.gnu.org Delivered-To: mailing list gcc-bugs@gcc.gnu.org Received: (qmail 31375 invoked by uid 48); 17 Apr 2014 13:26:32 -0000 From: "jakub at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug rtl-optimization/60866] [4.7/4.8/4.9/4.10 Regression] ICE: in get_seqno_for_a_jump, at sel-sched-ir.c:4068 with -fselective-scheduling -fno-if-conversion Date: Thu, 17 Apr 2014 13:26:00 -0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: rtl-optimization X-Bugzilla-Version: 4.10.0 X-Bugzilla-Keywords: ice-on-valid-code X-Bugzilla-Severity: normal X-Bugzilla-Who: jakub at gcc dot gnu.org X-Bugzilla-Status: NEW X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: 4.7.4 X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: cc Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-SW-Source: 2014-04/txt/msg01284.txt.bz2 Content-length: 463 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60866 Jakub Jelinek changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jakub at gcc dot gnu.org --- Comment #2 from Jakub Jelinek --- Started with r208165 on the trunk (which can of course mean it has been latent before).