On Fri, Oct 02, 2015 at 08:43:30AM -0700, H.J. Lu wrote:
> On Fri, Oct 2, 2015 at 3:23 AM, Marek Polacek <[email protected]> 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.
Oh well, sorry about that. I don't think this easily fixable at present :(.
I opened PR67819 for this problem. Until that is resolved, I will have to
move -Wduplicated-cond out of -Wall and -Wextra.
Marek
