[issue24324] Remove -Wunreachable-code flag

Ned Deily added the comment: Thanks for the suggested patches. I used a somewhat different approach in an attempt to retain the benefit of the unreachable code warning when using other (non-GCC) compilers, in particular, llvm clang where the warnings do seem to be useful. -- nosy: +n

[issue24324] Remove -Wunreachable-code flag

Roundup Robot added the comment: New changeset 8fecf6e17616 by Ned Deily in branch 'default': Issue #24324: merge from 3.5 https://hg.python.org/cpython/rev/8fecf6e17616 -- ___ Python tracker __

[issue24324] Remove -Wunreachable-code flag

Roundup Robot added the comment: New changeset 61f8f7610a88 by Ned Deily in branch '3.5': Issue #24324: Do not enable unreachable code warnings when using https://hg.python.org/cpython/rev/61f8f7610a88 -- nosy: +python-dev ___ Python tracker

[issue24324] Remove -Wunreachable-code flag

Changes by Jake Garver : Added file: http://bugs.python.org/file41663/Python351-no-unreachable-check.diff ___ Python tracker ___ ___ Python-bu

[issue24324] Remove -Wunreachable-code flag

Skip Montanaro added the comment: This seems to work for me. Diff against default. I have no idea how old the autoconf setup is here at work. I imagine you'll want to only apply the configure.ac patch and regenerate configure. -- keywords: +patch Added file: http://bugs.python.org/file

[issue24324] Remove -Wunreachable-code flag

STINNER Victor added the comment: Can you please propose a patch? -- nosy: +haypo ___ Python tracker ___ ___ Python-bugs-list mailing

[issue24324] Remove -Wunreachable-code flag

Erik Bray added the comment: This would definitely be nice to fix. I panicked a bit because of this when I compiled my extension modules against Python 3.5 for the first time. -- nosy: +erik.bray ___ Python tracker

[issue24324] Remove -Wunreachable-code flag

Changes by Raymond Hettinger : -- nosy: +pitrou ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.py

[issue24324] Remove -Wunreachable-code flag

Changes by Raymond Hettinger : -- nosy: +loewis ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.py

[issue24324] Remove -Wunreachable-code flag

Skip Montanaro added the comment: I guess this is an autoconf thing. "@BASECFLAGS@" in Makefile.pre.in seems to expand to "-Wsign-compare -Wunreachable-code" in Makefile.pre. -- ___ Python tracker

[issue24324] Remove -Wunreachable-code flag

New submission from Skip Montanaro: Makefile.pre defines BASECFLAGS to include -Wunreachable-code. When building in the Linux environment available to me (GCC 4.4.6), compilation spews tons of warnings about "warning: will never be executed". According to this StackOverflow thread: http://sta