On Fri, Oct 2, 2015 at 3:23 AM, Marek Polacek <pola...@redhat.com> wrote: > On Wed, Sep 30, 2015 at 12:45:35PM -0600, Jeff Law wrote: >> On 09/30/2015 09:47 AM, Joseph Myers wrote: >> >The C front-end changes are OK. >> The rest are OK as well. > > Thanks Jeff & Joseph. > > I'm going to apply the patch soon; should it draw the ire of users, I'll > move the option to -Wextra.
It breaks bootstrap: https://gcc.gnu.org/ml/gcc-regression/2015-10/msg00031.html ../../src-trunk/gcc/fortran/trans-intrinsic.c: In function âvoid gfc_conv_intrinsic_leadz(gfc_se*, gfc_expr*)â: ../../src-trunk/gcc/fortran/trans-intrinsic.c:4886:8: error: duplicated âifâ condition [-Werror=duplicated-cond] else if (argsize <= LONG_TYPE_SIZE) ^ ../../src-trunk/gcc/fortran/trans-intrinsic.c:4881:3: note: previously used here if (argsize <= INT_TYPE_SIZE) ^ ../../src-trunk/gcc/fortran/trans-intrinsic.c: In function âvoid gfc_conv_intrinsic_trailz(gfc_se*, gfc_expr*)â: ../../src-trunk/gcc/fortran/trans-intrinsic.c:5003:8: error: duplicated âifâ condition [-Werror=duplicated-cond] else if (argsize <= LONG_TYPE_SIZE) ^ ../../src-trunk/gcc/fortran/trans-intrinsic.c:4998:3: note: previously used here if (argsize <= INT_TYPE_SIZE) ^ ../../src-trunk/gcc/fortran/trans-intrinsic.c: In function âvoid gfc_conv_intrinsic_popcnt_poppar(gfc_se*, gfc_expr*, int)â: ../../src-trunk/gcc/fortran/trans-intrinsic.c:5109:8: error: duplicated âifâ condition [-Werror=duplicated-cond] else if (argsize <= LONG_TYPE_SIZE) ^ ../../src-trunk/gcc/fortran/trans-intrinsic.c:5102:3: note: previously used here if (argsize <= INT_TYPE_SIZE) ^ since int may have the same size as long and long may have the same size as long long. -- H.J.
