[Bug tree-optimization/81292] [8 regression] ICE in zero_length_string, at tree-ssa-strlen.c:822

2017-07-04 Thread dimhen at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81292

--- Comment #10 from Dmitry G. Dyachenko  ---
r249961 PASS for me

[Bug target/81317] builtin_vec_ld fails for powerpc with altivec

2017-07-04 Thread randy.macleod at windriver dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81317

--- Comment #6 from Randy MacLeod  ---
Created attachment 41682
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41682=edit
smaller reproducer with manual work-around

The work-around clearly isn't a viable one. 
I believe that builtin_vec_ld(a,b) can take 'a' as a variable but perhaps not.

[Bug target/81317] builtin_vec_ld fails for powerpc with altivec

2017-07-04 Thread randy.macleod at windriver dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81317

--- Comment #5 from Randy MacLeod  ---
I agree that the creduce output should be ignored. :(

I ran 'delta' again with a manually cleaned up input and I got a minimal
reproducer that is shorter:

wc -l output:

old:
  4756 /tmp/jdcolor-altivec.i
   269 /tmp/jdcolor-altivec-testcase-min.i
new:
   104 /tmp/bar.i

Note the comment in bar.i:
unaligned_shift_index =
  __builtin_vec_ld( num_cols , outptr); 
 /* replacing num_cols above with a constant such as 0, passes compile
*/

[Bug rtl-optimization/81290] [8 Regression] ICE in update_br_prob_note

2017-07-04 Thread asolokha at gmx dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81290

--- Comment #8 from Arseny Solokha  ---
Sorry, my bad. I've messed up my build environment for that target. Indeed
r249924 fixes the ICE everywhere.

[Bug middle-end/79257] spurious -Wformat-overflow=1 warning with -O2 and sanitizer

2017-07-04 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79257

--- Comment #8 from Martin Sebor  ---
(In reply to Pierre Chapuis from comment #5)

That's a different bug, the same one as the root cause behind the false
positive in bug 78969, comment 4.  The range information available outside the
VRP pass via get_range_info() is available (unlike in the test case in comment
#0) but it's off by one.  It probably deserves a separate bug report, distinct
from both this bug and pr78969, and this one can be resolved as a duplicate of
the latter.

[Bug target/81317] builtin_vec_ld fails for powerpc with altivec

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81317

Andrew Pinski  changed:

   What|Removed |Added

 Target||powerpc-oe-linux
  Component|c   |target

--- Comment #4 from Andrew Pinski  ---
I suspect there are two separate issues here.  The reduced testcase looks like
invalid code while the original looks valid.

[Bug c/81317] builtin_vec_ld fails for powerpc with altivec

2017-07-04 Thread randy.macleod at windriver dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81317

--- Comment #3 from Randy MacLeod  ---
Created attachment 41681
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41681=edit
full preprocessor output when building jdcolor-altivec.c

[Bug c/81317] builtin_vec_ld fails for powerpc with altivec

2017-07-04 Thread randy.macleod at windriver dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81317

--- Comment #2 from Randy MacLeod  ---
Created attachment 41680
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41680=edit
minimal testcase produced by the delta utility

[Bug c/81317] builtin_vec_ld fails for powerpc with altivec

2017-07-04 Thread randy.macleod at windriver dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81317

--- Comment #1 from Randy MacLeod  ---
I also used the delta utility to produce a larger reproducer. I'll attach it as
jdcolor-altivec-testcase-min.i along with the full preprocessor output in case
it's useful to have additional context.

[Bug c/81317] New: builtin_vec_ld fails for powerpc with altivec

2017-07-04 Thread randy.macleod at windriver dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81317

Bug ID: 81317
   Summary: builtin_vec_ld fails for powerpc with altivec
   Product: gcc
   Version: unknown
Status: UNCONFIRMED
  Severity: normal
  Priority: P3
 Component: c
  Assignee: unassigned at gcc dot gnu.org
  Reporter: randy.macleod at windriver dot com
  Target Milestone: ---

When building libjpeg-turbo using oe-core's build system with:
   master with HEAD at:
   325f087 image.bbclass: create root symlinks in nativesdk target sysroot
   on an Ubuntu-16.04.1 LTS host, 
I encountered a toolchain ICE. 

I used creduce to produce a minimal test case:
$ cat testcase.i
jsimd_ycc_extbgrx_convert_altivec_outptr;
__attribute__ ( ( altivec
  ( __builtin_vec_ld
( 1, jsimd_ycc_extbgrx_convert_altivec_outptr )
--
The compile line and error are:

$ powerpc-oe-linux-gcc  -m32 -mhard-float -mcpu=7400 -mno-spe -Wall -Wextra
--sysroot=/buildarea/rmacleod/src/distro/yocto/b/sd-ppc/tmp-glibc/work/ppc7400-oe-linux/libjpeg-turbo/1_1.5.2-r0/recipe-sysroot
-DHAVE_CONFIG_H -I. -I../../libjpeg-turbo-1.5.2/simd -I..
-I../../libjpeg-turbo-1.5.2 -Wall -maltivec -O2 -pipe -g
-feliminate-unused-debug-types
-fdebug-prefix-map=/buildarea/rmacleod/src/distro/yocto/b/sd-ppc/tmp-glibc/work/ppc7400-oe-linux/libjpeg-turbo/1_1.5.2-r0=/usr/src/debug/libjpeg-turbo/1_1.5.2-r0
-fdebug-prefix-map=/buildarea/rmacleod/src/distro/yocto/b/sd-ppc/tmp-glibc/work/ppc7400-oe-linux/libjpeg-turbo/1_1.5.2-r0/recipe-sysroot-native=
-fdebug-prefix-map=/buildarea/rmacleod/src/distro/yocto/b/sd-ppc/tmp-glibc/work/ppc7400-oe-linux/libjpeg-turbo/1_1.5.2-r0/recipe-sysroot=
-c testcase.i  -fPIC -DPIC -o .libs/libsimd_altivec_la-jdcolor-altivec.o
testcase.i:1:1: warning: data definition has no type or storage class
 jsimd_ycc_extbgrx_convert_altivec_outptr;
 ^~~~
testcase.i:1:1: warning: type defaults to 'int' in declaration of
'jsimd_ycc_extbgrx_convert_altivec_outptr' [-Wimplicit-int]
testcase.i:4:21: internal compiler error: Segmentation fault
 ( 1, jsimd_ycc_extbgrx_convert_altivec_outptr )
 ^ 
Please submit a full bug report, 
with preprocessed source if appropriate. 
See  for instructions. 



I have also confirmed that using 7.1.1-beta aka 7-20170629 does not fix the
ICE.

I can't easily test the gcc master branch so I await your suggestion.

oe-core builds it's own toolchain but we're close to this version:
$ powerpc-oe-linux-gcc -v 
Using built-in specs.
COLLECT_GCC=powerpc-oe-linux-gcc
COLLECT_LTO_WRAPPER=/buildarea/rmacleod/src/distro/yocto/b/sd-ppc/tmp-glibc/work/ppc7400-oe-linux/libjpeg-turbo/1_1.5.2-r0/recipe-sysroot-native/usr/bin/powerpc-oe-linux/../../libexec/powerpc-oe-linux/gcc/powerpc-oe-linux/7.1.0/lto-wrapper
Target: powerpc-oe-linux
Configured with: ../../../../../../work-shared/gcc-7.1.0-r0/gcc-7.1.0/configure
--build=x86_64-linux --host=x86_64-linux --target=powerpc-oe-linux
--prefix=/buildarea/rmacleod/src/distro/yocto/b/sd-ppc/tmp-glibc/work/x86_64-linux/gcc-cross-powerpc/7.1.0-r0/recipe-sysroot-native/usr
--exec_prefix=/buildarea/rmacleod/src/distro/yocto/b/sd-ppc/tmp-glibc/work/x86_64-linux/gcc-cross-powerpc/7.1.0-r0/recipe-sysroot-native/usr
--bindir=/buildarea/rmacleod/src/distro/yocto/b/sd-ppc/tmp-glibc/work/x86_64-linux/gcc-cross-powerpc/7.1.0-r0/recipe-sysroot-native/usr/bin/powerpc-oe-linux
--sbindir=/buildarea/rmacleod/src/distro/yocto/b/sd-ppc/tmp-glibc/work/x86_64-linux/gcc-cross-powerpc/7.1.0-r0/recipe-sysroot-native/usr/bin/powerpc-oe-linux
--libexecdir=/buildarea/rmacleod/src/distro/yocto/b/sd-ppc/tmp-glibc/work/x86_64-linux/gcc-cross-powerpc/7.1.0-r0/recipe-sysroot-native/usr/libexec/powerpc-oe-linux
--datadir=/buildarea/rmacleod/src/distro/yocto/b/sd-ppc/tmp-glibc/work/x86_64-linux/gcc-cross-powerpc/7.1.0-r0/recipe-sysroot-native/usr/share
--sysconfdir=/buildarea/rmacleod/src/distro/yocto/b/sd-ppc/tmp-glibc/work/x86_64-linux/gcc-cross-powerpc/7.1.0-r0/recipe-sysroot-native/etc
--sharedstatedir=/buildarea/rmacleod/src/distro/yocto/b/sd-ppc/tmp-glibc/work/x86_64-linux/gcc-cross-powerpc/7.1.0-r0/recipe-sysroot-native/com
--localstatedir=/buildarea/rmacleod/src/distro/yocto/b/sd-ppc/tmp-glibc/work/x86_64-linux/gcc-cross-powerpc/7.1.0-r0/recipe-sysroot-native/var
--libdir=/buildarea/rmacleod/src/distro/yocto/b/sd-ppc/tmp-glibc/work/x86_64-linux/gcc-cross-powerpc/7.1.0-r0/recipe-sysroot-native/usr/lib/powerpc-oe-linux
--includedir=/buildarea/rmacleod/src/distro/yocto/b/sd-ppc/tmp-glibc/work/x86_64-linux/gcc-cross-powerpc/7.1.0-r0/recipe-sysroot-native/usr/include
--oldincludedir=/buildarea/rmacleod/src/distro/yocto/b/sd-ppc/tmp-glibc/work/x86_64-linux/gcc-cross-powerpc/7.1.0-r0/recipe-sysroot-native/usr/include

[Bug middle-end/79257] spurious -Wformat-overflow=1 warning with -O2 and sanitizer

2017-07-04 Thread vincent-gcc at vinc17 dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79257

--- Comment #7 from Vincent Lefèvre  ---
(In reply to Pierre Chapuis from comment #5)
> I think I can reproduce something similar *without* the sanitizer.
> 
> Using GCC 7.1.1, with:
> 
> #include 
> int main () {
> int i; char obuf[3];
> int start = 0x00;
> for (i = start; i <= 0xff; ++i) {
> sprintf(obuf, "%02x", i);
> }
> return 0;
> }

If 0xff is replaced by 0xfe, the warning disappears. Is the issue due to the
fact that i can be 0x100 in the loop (thus with one additional character), but
GCC doesn't notice that this value will not be printed?

Similarly, I get the warning with

  for (i = start; i <= 0xfe; i += 2)

but not with

  for (i = start; i <= 0xfd; i += 2)

> I get:
> 
> demo.c: In function ‘main’:
> demo.c:6:23: warning: ‘sprintf’ may write a terminating nul past the end of
> the destination [-Wformat-overflow=]
>  sprintf(obuf, "%02x", i);
>^~
> demo.c:6:9: note: ‘sprintf’ output between 3 and 4 bytes into a destination
> of size 3
>  sprintf(obuf, "%02x", i);
>  ^~~~

IMHO, this seems to be a different bug, otherwise instead of "4 bytes", you
would get the number of bytes corresponding to INT_MAX-1 in the message, as in
comment 0.

> If I set `start` to `0xfb` or `0xfc`, I get the same warning. If I set it to
> `0xfe` or `0xff` I don't get a warning. If I set it to `0xfd` I get the
> warning *twice*:

This might be due to loop unrolling. You can see with -S that the generated asm
code has a loop for start = 0xfc, but not for start = 0xfd, 0xfe or 0xff. With
start = 0xfd, 3 sprintf are generated, and perhaps there's a warning for 2 of
them. Then I wonder why start = 0xfe no longer triggers the warning.

[Bug c++/68939] ICE or wrong code with OpenMP privatization of reference to VLAs

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68939

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug sanitizer/67513] ASAN: Not optimal shadow value check (unlikely condition checked in fast path)

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67513

Andrew Pinski  changed:

   What|Removed |Added

   Keywords||missed-optimization
   Target Milestone|6.5 |---

[Bug target/71419] interrupt attribute silently dropped for Thumb code on armv7-a

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71419

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug libstdc++/69331] FAIL: 20_util/shared_ptr/thread/default_weaktoshared.cc execution test

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69331

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

--- Comment #18 from Andrew Pinski  ---
I don't see this failure on either of my aarch64-linux-gnu testers:
https://gcc.gnu.org/ml/gcc-testresults/2017-07/msg00410.html (Ubuntu 1604
distro)
https://gcc.gnu.org/ml/gcc-testresults/2017-07/msg00367.html (CentOS 7.3
distro)

[Bug target/81300] -fpeephole2 breaks __builtin_ia32_sbb_u64, _subborrow_u64 on AMD64

2017-07-04 Thread uros at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81300

--- Comment #3 from uros at gcc dot gnu.org ---
Author: uros
Date: Tue Jul  4 21:05:17 2017
New Revision: 249978

URL: https://gcc.gnu.org/viewcvs?rev=249978=gcc=rev
Log:
PR target/81300
* config/i386/i386.md (setcc + movzbl/and to xor + setcc peepholes):
Require dead FLAGS_REG at the beginning of a peephole.

PR target/81294
* config/i386/adxintrin.h (_subborrow_u32): Swap _X and _Y
arguments in the call to __builtin_ia32_sbb_u32.
(_subborrow_u64): Swap _X and _Y arguments in the call to
__builtin_ia32_sbb_u64.

testsuite/ChangeLog:

PR target/81300
* gcc.target/i386/pr81300.c: New test.

PR target/81294
* gcc.target/i386/adx-addcarryx32-2.c (adx_test): Swap
x and y arguments in the call to _subborrow_u32.
* gcc.target/i386/adx-addcarryx64-2.c (adx_test): Swap
x and y arguments in the call to _subborrow_u64.
* gcc.target/i386/pr81294-1.c: New test.
* gcc.target/i386/pr81294-2.c: Ditto.


Added:
branches/gcc-7-branch/gcc/testsuite/gcc.target/i386/pr81294-1.c
branches/gcc-7-branch/gcc/testsuite/gcc.target/i386/pr81294-2.c
branches/gcc-7-branch/gcc/testsuite/gcc.target/i386/pr81300.c
Modified:
branches/gcc-7-branch/gcc/ChangeLog
branches/gcc-7-branch/gcc/config/i386/adxintrin.h
branches/gcc-7-branch/gcc/config/i386/i386.md
branches/gcc-7-branch/gcc/testsuite/ChangeLog
branches/gcc-7-branch/gcc/testsuite/gcc.target/i386/adx-addcarryx32-2.c
branches/gcc-7-branch/gcc/testsuite/gcc.target/i386/adx-addcarryx64-2.c

[Bug target/81294] _subborrow_u64 argument order inconsistent with intrinsic reference, icc

2017-07-04 Thread uros at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81294

--- Comment #4 from uros at gcc dot gnu.org ---
Author: uros
Date: Tue Jul  4 21:05:17 2017
New Revision: 249978

URL: https://gcc.gnu.org/viewcvs?rev=249978=gcc=rev
Log:
PR target/81300
* config/i386/i386.md (setcc + movzbl/and to xor + setcc peepholes):
Require dead FLAGS_REG at the beginning of a peephole.

PR target/81294
* config/i386/adxintrin.h (_subborrow_u32): Swap _X and _Y
arguments in the call to __builtin_ia32_sbb_u32.
(_subborrow_u64): Swap _X and _Y arguments in the call to
__builtin_ia32_sbb_u64.

testsuite/ChangeLog:

PR target/81300
* gcc.target/i386/pr81300.c: New test.

PR target/81294
* gcc.target/i386/adx-addcarryx32-2.c (adx_test): Swap
x and y arguments in the call to _subborrow_u32.
* gcc.target/i386/adx-addcarryx64-2.c (adx_test): Swap
x and y arguments in the call to _subborrow_u64.
* gcc.target/i386/pr81294-1.c: New test.
* gcc.target/i386/pr81294-2.c: Ditto.


Added:
branches/gcc-7-branch/gcc/testsuite/gcc.target/i386/pr81294-1.c
branches/gcc-7-branch/gcc/testsuite/gcc.target/i386/pr81294-2.c
branches/gcc-7-branch/gcc/testsuite/gcc.target/i386/pr81300.c
Modified:
branches/gcc-7-branch/gcc/ChangeLog
branches/gcc-7-branch/gcc/config/i386/adxintrin.h
branches/gcc-7-branch/gcc/config/i386/i386.md
branches/gcc-7-branch/gcc/testsuite/ChangeLog
branches/gcc-7-branch/gcc/testsuite/gcc.target/i386/adx-addcarryx32-2.c
branches/gcc-7-branch/gcc/testsuite/gcc.target/i386/adx-addcarryx64-2.c

[Bug target/67352] [avr] incorrect warning with -Waddr-space-convert and array in struct in __flash

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67352

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug c++/70019] VLA size overflow not detected

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70019

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug c++/65642] [C++11] GCC rejects valid constant expression

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65642

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug testsuite/66404] gcc.target/i386/pad-10.c etc. FAIL with PIE

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66404

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug target/81300] -fpeephole2 breaks __builtin_ia32_sbb_u64, _subborrow_u64 on AMD64

2017-07-04 Thread uros at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81300

--- Comment #2 from uros at gcc dot gnu.org ---
Author: uros
Date: Tue Jul  4 20:52:32 2017
New Revision: 249977

URL: https://gcc.gnu.org/viewcvs?rev=249977=gcc=rev
Log:
2017-07-04  Uros Bizjak  

PR target/81300
* config/i386/i386.md (setcc + movzbl/and to xor + setcc peepholes):
Require dead FLAGS_REG at the beginning of a peephole.

testsuite/ChangeLog:

PR target/81300
* gcc.target/i386/pr81300.c: New test.


Added:
trunk/gcc/testsuite/gcc.target/i386/pr81300.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/i386/i386.md
trunk/gcc/testsuite/ChangeLog

[Bug target/81294] _subborrow_u64 argument order inconsistent with intrinsic reference, icc

2017-07-04 Thread uros at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81294

--- Comment #3 from uros at gcc dot gnu.org ---
Author: uros
Date: Tue Jul  4 20:46:38 2017
New Revision: 249976

URL: https://gcc.gnu.org/viewcvs?rev=249976=gcc=rev
Log:
PR target/81294
* config/i386/adxintrin.h (_subborrow_u32): Swap _X and _Y
arguments in the call to __builtin_ia32_sbb_u32.
(_subborrow_u64): Swap _X and _Y arguments in the call to
__builtin_ia32_sbb_u64.

testsuite/ChangeLog:

PR target/81294
* gcc.target/i386/adx-addcarryx32-2.c (adx_test): Swap
x and y arguments in the call to _subborrow_u32.
* gcc.target/i386/adx-addcarryx64-2.c (adx_test): Swap
x and y arguments in the call to _subborrow_u64.
* gcc.target/i386/pr81294-1.c: New test.
* gcc.target/i386/pr81294-2.c: Ditto.


Added:
trunk/gcc/testsuite/gcc.target/i386/pr81294-1.c
trunk/gcc/testsuite/gcc.target/i386/pr81294-2.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/i386/adxintrin.h
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/gcc.target/i386/adx-addcarryx32-2.c
trunk/gcc/testsuite/gcc.target/i386/adx-addcarryx64-2.c

[Bug fortran/67420] gfortran.dg/norm2_3.f90 FAILs

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67420

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug target/61551] [NEON] alter costs to allow use of post-indexed addressing modes for VLD{2..4}/VST{2..4}

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61551

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug c++/68510] [concepts] ICE: in gimplify_var_or_parm_decl, at gimplify.c:1827

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68510

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug fortran/70598] Fortran OpenACC host_data construct ICE

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70598

Andrew Pinski  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED
   Target Milestone|6.5 |7.0

--- Comment #7 from Andrew Pinski  ---
Fixed.

[Bug ipa/64253] IPA inline analysis processes a code transform operation

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64253

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug c/67819] -Wduplicated-cond should take macros into account

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67819

Andrew Pinski  changed:

   What|Removed |Added

   Keywords||diagnostic
   Target Milestone|6.5 |---

[Bug c++/58796] throw nullptr not caught by catch(type*)

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58796

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug target/69940] gcc.c-torture/execute/alias-3.c FAILs with Solaris/x86 as

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69940

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug c++/66398] g++.dg/abi/anon1.C FAILs with PIE

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66398

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug middle-end/28628] Not forcing alignment of arrays in structs with -fsection-anchors

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=28628

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug target/49857] Put constant switch-tables into flash

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49857

Andrew Pinski  changed:

   What|Removed |Added

  Component|rtl-optimization|target
   Target Milestone|6.5 |---

[Bug middle-end/65534] tailcall not optimized away

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65534

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug target/35514] Gcc shoud generate symbol type for undefined symbol

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=35514

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug target/65846] Optimize data access in PIE with copy reloc

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65846

Andrew Pinski  changed:

   What|Removed |Added

   Keywords||missed-optimization
   Target Milestone|6.5 |---

[Bug middle-end/69513] LTO bootstrap fails with bootstrap-profiled during linking gnat1 in stagefeedback

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69513

Andrew Pinski  changed:

   What|Removed |Added

   Keywords||build
  Component|bootstrap   |middle-end
   Target Milestone|6.5 |---

[Bug ada/81114] GNAT mishandles filenames with UTF8 chars on case-insensitive filesystems

2017-07-04 Thread simon at pushface dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81114

--- Comment #3 from simon at pushface dot org ---
Just for interest, this not-very-good code will successfully convert
the uppercase-a-acute input c381 to uppercase-a/combining-acute 41cc81:

#include 
#include 
#include 
#include 

int main(void)
{
  uint8_t codepoint[] = {0xc3, 0x81, 0};
  char *input = (char *) 
  size_t in_size = 2;

  char output_buffer[10];
  memset(output_buffer, 0, sizeof(output_buffer));
  char *output = output_buffer;
  size_t out_size = 10;

  iconv_t cd = iconv_open("utf8-mac", "UTF-8");

  iconv(cd, , _size, , _size);

  printf("in %d out %d result \"%s\"\n", in_size, out_size, output_buffer);

  return 0;
}

but of course only on macOS - https://stackoverflow.com/a/23159081/40851

[Bug ada/81114] GNAT mishandles filenames with UTF8 chars on case-insensitive filesystems

2017-07-04 Thread ebotcazou at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81114

Eric Botcazou  changed:

   What|Removed |Added

 Status|UNCONFIRMED |SUSPENDED
   Last reconfirmed||2017-07-04
 CC||ebotcazou at gcc dot gnu.org
 Ever confirmed|0   |1

--- Comment #2 from Eric Botcazou  ---
Right.  And people should use sane filesystems (and sane OSes to begin with).

[Bug target/81316] New: Missing memory side effect on __atomic_store

2017-07-04 Thread mfcc64 at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81316

Bug ID: 81316
   Summary: Missing memory side effect on __atomic_store
   Product: gcc
   Version: 6.3.1
Status: UNCONFIRMED
  Severity: normal
  Priority: P3
 Component: target
  Assignee: unassigned at gcc dot gnu.org
  Reporter: mfcc64 at gmail dot com
  Target Milestone: ---

Created attachment 41679
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41679=edit
Testcase

Missing memory side effect on __atomic_store using __ATOMIC_RELEASE.

How to reproduce (atomic-fail.c attached):
gcc atomic-fail.c -O2 -pthread -o atomic-fail
./atomic-fail
Error: val != 1, missing memory side effect.


The bug is not reproduced with __atomic_store using __ATOMIC_SEQ_CST. Probably
this is similar to Bug80640.

[Bug bootstrap/81315] New: powerpc64 vs building lang/gcc7-devel (on FreeBSD head): xgcc gets segmentation fault

2017-07-04 Thread markmigm at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81315

Bug ID: 81315
   Summary: powerpc64 vs building lang/gcc7-devel (on FreeBSD
head): xgcc gets segmentation fault
   Product: gcc
   Version: 7.1.1
Status: UNCONFIRMED
  Severity: normal
  Priority: P3
 Component: bootstrap
  Assignee: unassigned at gcc dot gnu.org
  Reporter: markmigm at gmail dot com
  Target Milestone: ---

Created attachment 41678
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41678=edit
class_type_info.ii file from a -save-temps run

I was trying to build lang/gcc7-devel on FreeBSD head -r320570
on a powerpc64. The xgcc stage got the following segmentation
fault. (By contrast 32-bit powerpc's build of it under head
-r320570 completed without having this problem: somehow
powerpc64 specific.)

/usr/obj/portswork/usr/ports/lang/gcc7-devel/work/gcc-7-20170629/libstdc++-v3/libsupc++/class_type_info.cc:
In member function 'virtual bool
__cxxabiv1::__class_type_info::__do_upcast(const
__cxxabiv1::__class_type_info*, void**) const':
/usr/obj/portswork/usr/ports/lang/gcc7-devel/work/gcc-7-20170629/libstdc++-v3/libsupc++/class_type_info.cc:45:6:
internal compiler error: Segmentation fault
 bool __class_type_info::
  ^

The rest of the -save-temps output was:

xgcc: warning: -pipe ignored because -save-temps specified
Reading specs from
/usr/obj/portswork/usr/ports/lang/gcc7-devel/work/.build/./gcc/specs
COLLECT_GCC=/usr/obj/portswork/usr/ports/lang/gcc7-devel/work/.build/./gcc/xgcc
Target: powerpc64-portbld-freebsd12.0
Configured with:
/usr/obj/portswork/usr/ports/lang/gcc7-devel/work/gcc-7-20170629/configure
--enable-multilib --with-build-config=bootstrap-debug --disable-nls
--enable-gnu-indirect-function --libdir=/usr/local/lib/gcc7
--libexecdir=/usr/local/libexec/gcc7 --program-suffix=7
--with-as=/usr/local/bin/as --with-gmp=/usr/local
--with-gxx-include-dir=/usr/local/lib/gcc7/include/c++/
--with-ld=/usr/local/bin/ld --with-pkgversion='FreeBSD Ports Collection'
--with-system-zlib --enable-languages=c,c++,objc,fortran --prefix=/usr/local
--localstatedir=/var --mandir=/usr/local/man --infodir=/usr/local/info/gcc7
--build=powerpc64-portbld-freebsd12.0
Thread model: posix
gcc version 7.1.1 20170629 (FreeBSD Ports Collection) 
COLLECT_GCC_OPTIONS='-v' '-save-temps' '-shared-libgcc' '-B'
'/usr/obj/portswork/usr/ports/lang/gcc7-devel/work/.build/./gcc' '-nostdinc++'
'-L/usr/obj/portswork/usr/ports/lang/gcc7-devel/work/.build/powerpc64-portbld-freebsd12.0/libstdc++-v3/src'
'-L/usr/obj/portswork/usr/ports/lang/gcc7-devel/work/.build/powerpc64-portbld-freebsd12.0/libstdc++-v3/src/.libs'
'-L/usr/obj/portswork/usr/ports/lang/gcc7-devel/work/.build/powerpc64-portbld-freebsd12.0/libstdc++-v3/libsupc++/.libs'
'-B' '/usr/local/powerpc64-portbld-freebsd12.0/bin/' '-B'
'/usr/local/powerpc64-portbld-freebsd12.0/lib/' '-isystem'
'/usr/local/powerpc64-portbld-freebsd12.0/include' '-isystem'
'/usr/local/powerpc64-portbld-freebsd12.0/sys-include' '-I'
'/usr/obj/portswork/usr/ports/lang/gcc7-devel/work/gcc-7-20170629/libstdc++-v3/../libgcc'
'-I'
'/usr/obj/portswork/usr/ports/lang/gcc7-devel/work/.build/powerpc64-portbld-freebsd12.0/libstdc++-v3/include/powerpc64-portbld-freebsd12.0'
'-I'
'/usr/obj/portswork/usr/ports/lang/gcc7-devel/work/.build/powerpc64-portbld-freebsd12.0/libstdc++-v3/include'
'-I'
'/usr/obj/portswork/usr/ports/lang/gcc7-devel/work/gcc-7-20170629/libstdc++-v3/libsupc++'
'-D' '_GLIBCXX_SHARED' '-fno-implicit-templates' '-Wall' '-Wextra'
'-Wwrite-strings' '-Wcast-qual' '-Wabi' '-fdiagnostics-show-location=once'
'-ffunction-sections' '-fdata-sections' '-frandom-seed=class_type_info.lo'
'-O2' '-pipe' '-B' '/usr/local/bin/' '-D' 'LIBICONV_PLUG' '-g'
'-fno-strict-aliasing' '-B' '/usr/local/bin/' '-D' 'LIBICONV_PLUG' '-c' '-fPIC'
'-D' 'PIC' '-D' '_GLIBCXX_SHARED' '-o' 'class_type_info.o'
 /usr/obj/portswork/usr/ports/lang/gcc7-devel/work/.build/./gcc/cc1plus -E
-quiet -nostdinc++ -v -I
/usr/obj/portswork/usr/ports/lang/gcc7-devel/work/gcc-7-20170629/libstdc++-v3/../libgcc
-I
/usr/obj/portswork/usr/ports/lang/gcc7-devel/work/.build/powerpc64-portbld-freebsd12.0/libstdc++-v3/include/powerpc64-portbld-freebsd12.0
-I
/usr/obj/portswork/usr/ports/lang/gcc7-devel/work/.build/powerpc64-portbld-freebsd12.0/libstdc++-v3/include
-I
/usr/obj/portswork/usr/ports/lang/gcc7-devel/work/gcc-7-20170629/libstdc++-v3/libsupc++
-iprefix
/usr/obj/portswork/usr/ports/lang/gcc7-devel/work/.build/gcc/../lib/gcc7/gcc/powerpc64-portbld-freebsd12.0/7.1.1/
-isystem /usr/obj/portswork/usr/ports/lang/gcc7-devel/work/.build/./gcc/include
-isystem
/usr/obj/portswork/usr/ports/lang/gcc7-devel/work/.build/./gcc/include-fixed -D
_GLIBCXX_SHARED -D LIBICONV_PLUG -D LIBICONV_PLUG -D PIC -D _GLIBCXX_SHARED
-isystem /usr/local/powerpc64-portbld-freebsd12.0/include -isystem
/usr/local/powerpc64-portbld-freebsd12.0/sys-include

[Bug middle-end/79257] spurious -Wformat-overflow=1 warning with -O2 and sanitizer

2017-07-04 Thread catwell-gcc at catwell dot info
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79257

--- Comment #6 from Pierre Chapuis  ---
(Note this is my first time reporting something to GCC, if you think I should
have opened a new ticked because it may be a slightly different bug please tell
me.)

[Bug debug/81278] [8 Regression] -fcompare-debug failure (length)

2017-07-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81278

Jakub Jelinek  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #19 from Jakub Jelinek  ---
Fixed.

[Bug middle-end/79257] spurious -Wformat-overflow=1 warning with -O2 and sanitizer

2017-07-04 Thread catwell-gcc at catwell dot info
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79257

Pierre Chapuis  changed:

   What|Removed |Added

 CC||catwell-gcc at catwell dot info

--- Comment #5 from Pierre Chapuis  ---
I think I can reproduce something similar *without* the sanitizer.

Using GCC 7.1.1, with:


#include 
int main () {
int i; char obuf[3];
int start = 0x00;
for (i = start; i <= 0xff; ++i) {
sprintf(obuf, "%02x", i);
}
return 0;
}


I get:


demo.c: In function ‘main’:
demo.c:6:23: warning: ‘sprintf’ may write a terminating nul past the end of the
destination [-Wformat-overflow=]
 sprintf(obuf, "%02x", i);
   ^~
demo.c:6:9: note: ‘sprintf’ output between 3 and 4 bytes into a destination of
size 3
 sprintf(obuf, "%02x", i);
 ^~~~


At O1 I don't get the warning.

If I set `start` to `0xfb` or `0xfc`, I get the same warning. If I set it to
`0xfe` or `0xff` I don't get a warning. If I set it to `0xfd` I get the warning
*twice*:


demo.c: In function ‘main’:
demo.c:6:23: warning: ‘sprintf’ may write a terminating nul past the end of the
destination [-Wformat-overflow=]
 sprintf(obuf, "%02x", i);
   ^~
demo.c:6:9: note: ‘sprintf’ output between 3 and 4 bytes into a destination of
size 3
 sprintf(obuf, "%02x", i);
 ^~~~
demo.c:6:23: warning: ‘sprintf’ may write a terminating nul past the end of the
destination [-Wformat-overflow=]
 sprintf(obuf, "%02x", i);
   ^~
demo.c:6:9: note: ‘sprintf’ output between 3 and 4 bytes into a destination of
size 3
 sprintf(obuf, "%02x", i);
 ^~~~

[Bug debug/81278] [8 Regression] -fcompare-debug failure (length)

2017-07-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81278

--- Comment #18 from Jakub Jelinek  ---
Author: jakub
Date: Tue Jul  4 18:51:02 2017
New Revision: 249975

URL: https://gcc.gnu.org/viewcvs?rev=249975=gcc=rev
Log:
PR debug/81278
* tree-vrp.c (compare_assert_loc): Turn into a function template
with stable template parameter.  Only test if a->e is NULL,
!a->e == !b->e has been verified already.  Use e == NULL or
e != NULL instead of e or ! e tests.  If stable is true, don't use
iterative_hash_expr, on the other side allow a or b or both NULL
and sort the NULLs last.
(process_assert_insertions): Sort using compare_assert_loc
instead of compare_assert_loc, later sort using
compare_assert_loc before calling process_assert_insertions_for
in a loop.  Use break instead of continue once seen NULL pointer.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/tree-vrp.c

[Bug target/66207] Switch alpha to LRA

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66207

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug target/67412] gfortran.dg/execute_command_line_2.f90 FAILs on Solaris 10

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67412

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug ipa/81261] [8 Regression] ICE with -fno-guess-branch-probability

2017-07-04 Thread dcb314 at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81261

David Binderman  changed:

   What|Removed |Added

 CC||dcb314 at hotmail dot com

--- Comment #3 from David Binderman  ---
Did this one get into trunk ?

I am still seeing this at date 20170704.

[Bug sanitizer/68065] Size calculations for VLAs can overflow

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68065

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug testsuite/70150] Additonal test failures with --enable-default-pie

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70150

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug sanitizer/66401] g++.dg/ubsan/vla-1.C FAILs with PIE

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66401

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug c/65891] -Wlogical-op now warns about logical ‘and’ of equal expressions even when different types/sizeofs are involved

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65891

Andrew Pinski  changed:

   What|Removed |Added

   Keywords||diagnostic
   Target Milestone|6.5 |---

[Bug target/67710] FAIL: gcc.dg/darwin-*version-*.c (test for excess errors) with Xcode 7

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67710

Andrew Pinski  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED
   Target Milestone|6.5 |6.3

--- Comment #17 from Andrew Pinski  ---
.

[Bug fortran/65024] [OOP] unlimited polymorphic pointer structure not built when it should be

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65024

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug other/66259] Combined gcc and binutils build from git-master fails, with gas/as-new not existing

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66259

Andrew Pinski  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED
   Target Milestone|6.5 |6.0

--- Comment #15 from Andrew Pinski  ---
Fixed a long time ago.

[Bug c++/66399] g++.dg/abi/anon4.C FAILs with PIE

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66399

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug other/66250] Can't adjust complex nor decimal floating point modes

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66250

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug rtl-optimization/68749] FAIL: gcc.dg/ifcvt-4.c scan-rtl-dump ce1 "2 true changes made"

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68749

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug target/81291] [5/6/7/8 Regression] wrong code with -O2 -fno-rerun-cse-after-loop -fno-tree-ter -fno-tree-vrp -funroll-loops due to improper carry

2017-07-04 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81291

--- Comment #2 from Segher Boessenkool  ---
To answer your last question (from comment 1) first: at expand
time we have:

  d.3_10 = d;
  _11 = (long long unsigned int) d.3_10;
  c.4_12 = c;
  e.5_14 = e;
  _15 = (long long unsigned int) e.5_14;
  _20 = _11 + _15;
  _19 = c.4_12 + _20;
  return _19;

so it is adding a bunch of u64's.  We could add them as 32-bit
words (it cannot overflow) and only then convert to 64-bit, but
the gimple optimisers do not figure that out.  combine is left
to pick up the pieces, but it does something wrong it seems.

[Bug c++/70588] SIGBUS on a VLA larger than SIZE_MAX / 2

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70588

Andrew Pinski  changed:

   What|Removed |Added

  Known to work|6.0 |
   Target Milestone|6.5 |---

[Bug c/65430] Missing -Wsequence-point warning with COMPOUND_EXPRs

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65430

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug tree-optimization/70315] FAIL: gcc.dg/tree-ssa/sra-17.c scan-tree-dump-times esra

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70315

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug target/66960] Add interrupt attribute to x86 backend

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66960

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug c++/77967] ICE in cp_parser_type_name, at cp/parser.c:16419 (trying to add a method to a template alias)

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77967

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug c++/60336] empty struct value is passed differently in C and C++

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60336

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug c/65452] strcmp (foo, foo) could give a warning

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65452

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug c++/66487] sanitizer/warnings for lifetime DSE

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66487

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug testsuite/66403] gcc.dg/torture/builtin-self.c FAILs with PIE

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66403

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug target/65139] Improve register allocation for aarch64_*_sisd_or_int3 patterns

2017-07-04 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65139

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|6.5 |---

[Bug target/81312] Enable X86_TUNE_ACCUMULATE_OUTGOING_ARGS

2017-07-04 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81312

--- Comment #4 from H.J. Lu  ---
(In reply to Jan Hubicka from comment #3)
> > Doesn't it improve performance in 64-bit mode?
> 
> It used to be important for CPUs without ESP prediction engines, but most of
> those are
> history now.  The codegen is quite different so it may be the case that it
> still helps
> somehow.  Do you have some data on the perfomrance?
> 

It is related to PR 81313.  Without it, GCC generates awful stack realignment
code.

[Bug target/81291] [5/6/7/8 Regression] wrong code with -O2 -fno-rerun-cse-after-loop -fno-tree-ter -fno-tree-vrp -funroll-loops due to improper carry

2017-07-04 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81291

Segher Boessenkool  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2017-07-04
 Ever confirmed|0   |1

[Bug target/81312] Enable X86_TUNE_ACCUMULATE_OUTGOING_ARGS

2017-07-04 Thread hubicka at ucw dot cz
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81312

--- Comment #3 from Jan Hubicka  ---
> Doesn't it improve performance in 64-bit mode?

It used to be important for CPUs without ESP prediction engines, but most of
those are
history now.  The codegen is quite different so it may be the case that it
still helps
somehow.  Do you have some data on the perfomrance?

Honza

[Bug c++/81314] New: In function `star1(int, std::vector<double, std::allocator >&) [clone ._omp_cpyfn.1]': $(PWD)/stencil-vector-taskloop-gccbug2.cc:4: undefined reference to `std::vector

2017-07-04 Thread jeff.science at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81314

Bug ID: 81314
   Summary: In function `star1(int, std::vector&) [clone ._omp_cpyfn.1]':
$(PWD)/stencil-vector-taskloop-gccbug2.cc:4: undefined
reference to `std::vector::vector(std::vector&) [clone ._omp_cpyfn.1]':
$(PWD)/stencil-vector-taskloop-gccbug2.cc:4: undefined reference to
`std::vector::vector(std::vector const&)'
collect2: error: ld returned 1 exit status
FAIL

$ g++-7 -std=c++11 -g -O2 -fopenmp stencil-vector-taskloop-gccbug2.cc -o
stencil-vector-taskloop-gccbug2 && echo SUCCESS || echo FAIL
/tmp/ccuIhlTa.o: In function `star1(int, std::vector&) [clone ._omp_cpyfn.1]':
$(PWD)/stencil-vector-taskloop-gccbug2.cc:4: undefined reference to
`std::vector::vector(std::vector const&)'
collect2: error: ld returned 1 exit status
FAIL

## compile-only

$ g++-8 -std=c++11 -g -O2 -fopenmp -c stencil-vector-taskloop-gccbug2.cc -o
stencil-vector-taskloop-gccbug2.o && echo SUCCESS || echo FAIL
SUCCESS

## without OpenMP

$ g++-8 -std=c++11 -g -O2 stencil-vector-taskloop-gccbug2.cc -o
stencil-vector-taskloop-gccbug2 && echo SUCCESS || echo FAIL
SUCCESS

## other compilers

$ /opt/llvm/4.0.0/bin/clang++ -std=c++11 -g -O2 -fopenmp -Wunused
stencil-vector-taskloop-gccbug2.cc -o stencil-vector-taskloop-gccbug2
/opt/llvm/4.0.0/lib/libomp.so && echo SUCCESS || echo FAIL
SUCCESS

$ icpc -std=c++11 -g -O2 -fopenmp stencil-vector-taskloop-gccbug2.cc -o
stencil-vector-taskloop-gccbug2 && echo SUCCESS || echo FAIL
SUCCESS

# SOURCE

$ cat stencil-vector-taskloop-gccbug2.cc
#include 

void star1(const int n, std::vector & out) {
  _Pragma("omp taskloop")
  for (auto i=0; i<(n*n); ++i) out[i] = 1.0;
}

int main()
{
  const int n = 1000;
  std::vector out;
  out.resize(n*n,0.0);
  _Pragma("omp parallel")
  _Pragma("omp master")
  {
star1(n, out);
_Pragma("omp taskwait")
  }
  return 0;
}

# GCC VERSION

$ g++-8 -v
Using built-in specs.
COLLECT_GCC=g++-8
COLLECT_LTO_WRAPPER=/opt/gcc/HEAD/bin/../libexec/gcc/x86_64-pc-linux-gnu/8.0.0/lto-wrapper
Target: x86_64-pc-linux-gnu
Configured with: /opt/gcc//git/configure --program-suffix=-8 --disable-multilib
--enable-threads=posix --enable-checking=release --with-system-zlib
--enable-__cxa_atexit --enable-languages=c,c++,fortran --with-tune=native
--enable-bootstrap --enable-lto --enable-gold=yes --enable-ld=yes
--prefix=/opt/gcc//HEAD : (reconfigured) /opt/gcc//git/configure
--program-suffix=-8 --disable-multilib --enable-threads=posix
--enable-checking=release --with-system-zlib --enable-__cxa_atexit
--enable-languages=c,c++,fortran --with-tune=native --enable-bootstrap
--enable-lto --enable-gold=yes --enable-ld=yes --prefix=/opt/gcc//HEAD
Thread model: posix
gcc version 8.0.0 20170627 (experimental) (GCC)

$ g++-7 -v
Using built-in specs.
COLLECT_GCC=g++-7
COLLECT_LTO_WRAPPER=/opt/gcc/7.1.0/libexec/gcc/x86_64-pc-linux-gnu/7.1.0/lto-wrapper
Target: x86_64-pc-linux-gnu
Configured with: ../configure --prefix=/opt/gcc/7.1.0 --program-suffix=-7
--enable-shared --enable-static --enable-threads=posix
--enable-checking=release --with-system-zlib --enable-__cxa_atexit
--enable-languages=c,c++,fortran --with-tune=native --enable-bootstrap
--enable-lto --enable-gold=yes --enable-ld=yes --disable-multilib
Thread model: posix
gcc version 7.1.0 (GCC)

[Bug tree-optimization/71016] [6 Regression] Redundant sign extension with conditional __builtin_clzl

2017-07-04 Thread ramana at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71016

Ramana Radhakrishnan  changed:

   What|Removed |Added

 CC||ramana at gcc dot gnu.org

--- Comment #13 from Ramana Radhakrishnan  ---
(In reply to Richard Biener from comment #12)
> GCC 6.4 is being released, adjusting target milestone.

Should we just close this one as wont backport and fix it at GCC-7 ?

[Bug target/81295] [7 Regression] bootstrap broken on powerpc-linux-gnu

2017-07-04 Thread doko at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81295

--- Comment #8 from Matthias Klose  ---
failure mode? an empty main program is segfaulting.

looks like --enable-default-pie and --enable-secureplt are only used for my
build.

 --enable-languages=c,ada,c++,go,d,fortran,objc,obj-c++
 --prefix=/usr
 --with-gcc-major-version-only
 --program-suffix=-7
 --program-prefix=powerpc-linux-gnu-
 --enable-shared
 --enable-linker-build-id
 --libexecdir=/usr/lib
 --without-included-gettext
 --enable-threads=posix
 --libdir=/usr/lib
 --enable-nls
 --with-sysroot=/
 --enable-clocale=gnu
 --enable-libstdcxx-debug
 --enable-libstdcxx-time=yes
 --with-default-libstdcxx-abi=new
 --enable-gnu-unique-object
 --disable-libitm
 --disable-libquadmath
 --enable-plugin
 --enable-default-pie
 --with-system-zlib
 --enable-objc-gc=auto
 --enable-secureplt
 --disable-softfloat
 --with-cpu=default32
 --disable-softfloat
 --enable-targets=powerpc-linux,powerpc64-linux
 --enable-multiarch
 --disable-werror
 --with-long-double-128
 --enable-multilib
 --enable-checking=release
 --build=powerpc-linux-gnu
 --host=powerpc-linux-gnu
 --target=powerpc-linux-gnu

[Bug libstdc++/71556] set::get_time() requires leading 0s for %H and friends

2017-07-04 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71556

Jonathan Wakely  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|unassigned at gcc dot gnu.org  |redi at gcc dot gnu.org

[Bug target/81312] Enable X86_TUNE_ACCUMULATE_OUTGOING_ARGS

2017-07-04 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81312

--- Comment #2 from H.J. Lu  ---
(In reply to Jan Hubicka from comment #1)
> I am planning to look into Zen/generic tunning soon. But why would you like
> to
> enable it for generic? The code size cost is quite large.
> 
> Honza

Doesn't it improve performance in 64-bit mode?

[Bug c++/81259] Class template deduction cannot work on Constructor without parameters in some cases

2017-07-04 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81259

Jonathan Wakely  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2017-07-04
 Ever confirmed|0   |1

[Bug c++/80451] [6/7/8 Regression] return implicit type conversion to std::experimental::optional does not compile

2017-07-04 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80451

Jonathan Wakely  changed:

   What|Removed |Added

  Component|libstdc++   |c++

--- Comment #11 from Jonathan Wakely  ---
Although this isn't a regression in the front-end, we can't easily fix the
libstdc++regression without a front-end fix for the incomplete feature.

[Bug target/81312] Enable X86_TUNE_ACCUMULATE_OUTGOING_ARGS

2017-07-04 Thread hubicka at ucw dot cz
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81312

--- Comment #1 from Jan Hubicka  ---
I am planning to look into Zen/generic tunning soon. But why would you like to
enable it for generic? The code size cost is quite large.

Honza

[Bug target/81313] New: Bad stack realignment code with -mno-accumulate-outgoing-args

2017-07-04 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81313

Bug ID: 81313
   Summary: Bad stack realignment code with
-mno-accumulate-outgoing-args
   Product: gcc
   Version: 8.0
Status: UNCONFIRMED
  Severity: normal
  Priority: P3
 Component: target
  Assignee: unassigned at gcc dot gnu.org
  Reporter: hjl.tools at gmail dot com
CC: pavel.v.chupin at gmail dot com
  Target Milestone: ---

[hjl@gnu-tools-1 gcc]$ cat /tmp/b.c
extern void foo (void);

void
bar (void)
{
  foo ();
}
[hjl@gnu-tools-1 gcc]$ ./xgcc -B./  /tmp/b.c -S -O2
-fno-asynchronous-unwind-tables -mincoming-stack-boundary=3 
-mno-accumulate-outgoing-args  
[hjl@gnu-tools-1 gcc]$ cat b.s
.file   "b.c"
.text
.p2align 4,,15
.globl  bar
.type   bar, @function
bar:
leaq8(%rsp), %r10
andq$-16, %rsp
pushq   -8(%r10)
pushq   %rbp
movq%rsp, %rbp
pushq   %r10
subq$8, %rsp
callfoo
addq$8, %rsp
popq%r10
popq%rbp
leaq-8(%r10), %rsp
ret
.size   bar, .-bar
.ident  "GCC: (GNU) 7.1.1 20170704"
.section.note.GNU-stack,"",@progbits
[hjl@gnu-tools-1 gcc]$ ./xgcc -B./  /tmp/b.c -S -O2
-fno-asynchronous-unwind-tables -mincoming-stack-boundary=3 
-maccumulate-outgoing-args  
[hjl@gnu-tools-1 gcc]$ cat b.s
.file   "b.c"
.text
.p2align 4,,15
.globl  bar
.type   bar, @function
bar:
pushq   %rbp
movq%rsp, %rbp
andq$-16, %rsp
callfoo
leave
ret
.size   bar, .-bar
    .ident  "GCC: (GNU) 7.1.1 20170704"
.section.note.GNU-stack,"",@progbits
[hjl@gnu-tools-1 gcc]$ 

This is due to

static rtx
ix86_get_drap_rtx (void)
{
  if (ix86_force_drap || !ACCUMULATE_OUTGOING_ARGS)
crtl->need_drap = true; 

Since nothing is passed on stack, need_trap should be false.

[Bug tree-optimization/60510] SLP blocks loop vectorization (with reduction)

2017-07-04 Thread clyon at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60510

Christophe Lyon  changed:

   What|Removed |Added

 CC||clyon at gcc dot gnu.org

--- Comment #11 from Christophe Lyon  ---
FWIW, the new tests fail on arm, and pass on aarch64.

[Bug target/81312] New: Enable X86_TUNE_ACCUMULATE_OUTGOING_ARGS

2017-07-04 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81312

Bug ID: 81312
   Summary: Enable X86_TUNE_ACCUMULATE_OUTGOING_ARGS
   Product: gcc
   Version: 8.0
Status: UNCONFIRMED
  Severity: normal
  Priority: P3
 Component: target
  Assignee: unassigned at gcc dot gnu.org
  Reporter: hjl.tools at gmail dot com
CC: hubicka at ucw dot cz, ubizjak at gmail dot com
  Target Milestone: ---

x86-tune.def has

* X86_TUNE_ACCUMULATE_OUTGOING_ARGS: Allocate stack space for outgoing
   arguments in prologue/epilogue instead of separately for each call
   by push/pop instructions.
   This increase code size by about 5% in 32bit mode, less so in 64bit mode
   because parameters are passed in registers.  It is considerable
   win for targets without stack engine that prevents multple push operations
   to happen in parallel.

   FIXME: the flags is incorrectly enabled for amdfam10, Bulldozer,
   Bobcat and Generic.  This is because disabling it causes large
   regression on mgrid due to IRA limitation leading to unecessary
   use of the frame pointer in 32bit mode.  */
DEF_TUNE (X86_TUNE_ACCUMULATE_OUTGOING_ARGS, "accumulate_outgoing_args",
  m_PPRO | m_P4_NOCONA | m_BONNELL | m_SILVERMONT | m_KNL | m_INTEL
  | m_ATHLON_K8)

Is this still true today for the current AMD processors? If yes, I'd like
to enable it for 64-bit -mtune=generic.

[Bug target/67321] [ARM] Exploit Wide Add operations when appropriate

2017-07-04 Thread ramana at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67321

Ramana Radhakrishnan  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED
   Target Milestone|6.5 |7.0

--- Comment #7 from Ramana Radhakrishnan  ---
This is now fixed.

[Bug rtl-optimization/81308] [8 regression] ICE in calc_dfs_tree, at dominance.c:458

2017-07-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81308

Jakub Jelinek  changed:

   What|Removed |Added

 CC||uros at gcc dot gnu.org

--- Comment #4 from Jakub Jelinek  ---
In *.postreload we still have:
(insn 27 110 111 3 (parallel [
(set (reg:SF 8 st [orig:89 _26 ] [89])
(float_truncate:SF (reg:DF 9 st(1) [116])))
(clobber (mem/c:SF (plus:SI (reg/f:SI 6 bp)
(const_int -76 [0xffb4])) [0  S4 A32]))
]) "pr62079.C":50 160 {*truncdfsf_i387}
 (expr_list:REG_EH_REGION (const_int 12 [0xc])
(nil)))
;;  succ:   10 [always (guessed)]  (FALLTHRU)
;;  5 [never]  count:0 (ABNORMAL,EH)

Now, not sure why is that insn marked as REG_EH_REGION, is that because the
truncation might throw on inexact etc. or because the memory access might
throw?  As the MEM is /c, I'd think the latter should not be the case.
In any case, split2 splits this into:
(insn 117 110 118 3 (set (mem/c:SF (plus:SI (reg/f:SI 6 bp)
(const_int -76 [0xffb4])) [0  S4 A32])
(reg:SF 9 st(1))) "pr62079.C":50 127 {*movsf_internal}
 (nil))
(insn 118 117 111 3 (set (reg:SF 8 st [orig:89 _26 ] [89])
(mem/c:SF (plus:SI (reg/f:SI 6 bp)
(const_int -76 [0xffb4])) [0  S4 A32]))
"pr62079.C":50 127 {*movsf_internal}
 (nil))
which doesn't have any REG_EH_REGION notes and thus we basically need to clean
up the EH edges, but don't do that.

[Bug bootstrap/81033] [8 Regression] Revision r249019 breaks bootstrap on darwin

2017-07-04 Thread simon at pushface dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033

--- Comment #17 from simon at pushface dot org ---
I applied your patches (comment 12, comment 13) to the gcc-8-20170702 snapshot.
Building on darwin 16.6.0 with gcc-7.1.0 as the stage1 compiler was successful
(languages=c,c++,ada).

make check-c:

=== gcc Summary ===

# of expected passes41267
# of unexpected failures28
# of expected failures  120
# of unresolved testcases   4
# of unsupported tests  1049

[Bug rtl-optimization/81308] [8 regression] ICE in calc_dfs_tree, at dominance.c:458

2017-07-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81308

--- Comment #3 from Jakub Jelinek  ---
The ICE is because split2 apparently ends up with various unreachable basic
blocks and the checking code attempts to compute dominance info before those
are actually cleaned up.

[Bug ada/81114] GNAT mishandles filenames with UTF8 chars on case-insensitive filesystems

2017-07-04 Thread simon at pushface dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81114

--- Comment #1 from simon at pushface dot org ---
Further:

$ GNAT_FILE_NAME_CASE_SENSITIVE=1 gnatmake -c p*.ads
gcc -c páck3.ads
páck3.ads:1:10: warning: file name does not match unit name, should be
"páck3.ads"

The reason for this apparently-bizarre message is[1] that macOS takes 
the composed form (lowercase a acute) and converts it under the hood 
to what HFS+ insists on, the fully decomposed form (lowercase a, combining 
acute); thus the names are actually different even though they _look_ 
the same.

I have to say that, great as it would be to have this fixed, the changes 
required would be extensive, and I can’t see that anyone would think it 
worth the trouble.

The recommendation would be "don’t use international characters in the 
names of library units".

[1] https://stackoverflow.com/a/6153713/40851

[Bug c++/81311] New: An std::ref argument calls copy constructor instead of template constructor in C++17 mode

2017-07-04 Thread pdimov at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81311

Bug ID: 81311
   Summary: An std::ref argument calls copy constructor instead of
template constructor in C++17 mode
   Product: gcc
   Version: 8.0
Status: UNCONFIRMED
  Severity: normal
  Priority: P3
 Component: c++
  Assignee: unassigned at gcc dot gnu.org
  Reporter: pdimov at gmail dot com
  Target Milestone: ---

In the following code:

```
#include 
#include 

struct function
{
function()
{
std::cout << "function()\n";
}

template function( F )
{
std::cout << "function(F)\n";
}

function( function const& )
{
std::cout << "function(function const&)\n";
}
};

int main()
{
function f1;
function f2( std::ref(f1) );
}
```

g++ 7.1 and 8 with -std=c++1z/17 call the copy constructor, whereas with
-std=c++14 they call the template constructor (as do other compilers in all
language modes.)

[Bug fortran/81304] [5/6/7/8 Regression] Bogus warning with -Wsurprising and -fopenmp: Type specified for intrinsic function 'min' / 'max'

2017-07-04 Thread janus at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81304

janus at gcc dot gnu.org changed:

   What|Removed |Added

Summary|Bogus warning with  |[5/6/7/8 Regression] Bogus
   |-Wsurprising and -fopenmp:  |warning with -Wsurprising
   |Type specified for  |and -fopenmp: Type
   |intrinsic function 'min' /  |specified for intrinsic
   |'max'   |function 'min' / 'max'
  Known to fail||4.5.0

--- Comment #2 from janus at gcc dot gnu.org ---
(In reply to Dominique d'Humieres from comment #1)
> Confirmed from 4.5 up to trunk (8.0). I don't get any warning with gcc
> 4.4.7: regression?

Yes, seems to be a regression (a pretty old one, though).

[Bug rtl-optimization/81308] [8 regression] ICE in calc_dfs_tree, at dominance.c:458

2017-07-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81308

Jakub Jelinek  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2017-07-04
 Ever confirmed|0   |1

--- Comment #2 from Jakub Jelinek  ---
Started with r249885.

[Bug rtl-optimization/81308] [8 regression] ICE in calc_dfs_tree, at dominance.c:458

2017-07-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81308

Jakub Jelinek  changed:

   What|Removed |Added

 CC||jakub at gcc dot gnu.org

--- Comment #1 from Jakub Jelinek  ---
It fails on i686-linux too.

[Bug c/81306] valgrind error for function warn_for_multistatement_macros in file c-warn.c line 2474

2017-07-04 Thread dcb314 at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81306

--- Comment #2 from David Binderman  ---
I've had a go at chopping down the command line and got this:

$ pwd
/home/dcb/gcc/working/x86_64-pc-linux-gnu/libgcc

$ /home/dcb/gcc/working/./gcc/xgcc -B/home/dcb/gcc/working/./gcc/ -Wall -I.
-I../../../trunk/libgcc/. -I../../../trunk/include -c
../../../trunk/libgcc/soft-fp/divtf3.c

Using -E and capturing the preprocessor output in a temporary file and
then compiling that doesn't seem to work.

[Bug c++/81299] Spurious "set but not used" warning with constexpr variable

2017-07-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81299

Jakub Jelinek  changed:

   What|Removed |Added

 CC||jakub at gcc dot gnu.org,
   ||jason at gcc dot gnu.org

--- Comment #1 from Jakub Jelinek  ---
If fun is just const and not constexpr, then we mark it TREE_USED during
build_capture_proxy, if it is not const nor constexpr, then when force_rvalue
in build_lambda_capture.  But when it is constexpr, it goes through none of
that.  Not sure what to do, for constexpr vars the -Wunused-but-set-variable
warning doesn't make much sense, one can't set them except for the initializer,
and so either the var is unused except for the declaration with the
initializer, then we should emit normal -Wuninitialized warning, or it is not
unused, then we just shouldn't warn.

[Bug bootstrap/81033] [8 Regression] Revision r249019 breaks bootstrap on darwin

2017-07-04 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033

--- Comment #16 from Dominique d'Humieres  ---
Created attachment 41677
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41677=edit
assembly with the fixes before r249930

[Bug bootstrap/81033] [8 Regression] Revision r249019 breaks bootstrap on darwin

2017-07-04 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033

--- Comment #15 from Dominique d'Humieres  ---
Created attachment 41676
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41676=edit
Assembly with the patch for r249930

[Bug bootstrap/81033] [8 Regression] Revision r249019 breaks bootstrap on darwin

2017-07-04 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81033

--- Comment #14 from Dominique d'Humieres  ---
Revision r249930 breaks some objc tests:

% /opt/gcc/gcc8p-249834p3/bin/g++
/opt/gcc/work/gcc/testsuite/objc.dg/torture/forward-1.m
-L/opt/gcc/build_w/x86_64-apple-darwin16.6.0/libobjc/.libs -lobjc -O2
-fgnu-runtime
/var/folders/8q/sh_swgz96r7f5vnn08f7fxr0gn/T//ccQHrZR1.s:100:38: error:
unexpected token in argument list
___cold_sect_of_"-[Receiver display]":
 ^
I'll attach the assembly with/without the patch for r249930.

[Bug c/81310] Is __attribute__ alias on variables valid?

2017-07-04 Thread joseph at codesourcery dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81310

--- Comment #2 from joseph at codesourcery dot com  ---
It's valid on variables (but hidden aliases to variables in shared 
libraries don't work properly, and the static linker has special handling 
for the case where a shared library exports both variable __foo and a weak 
alias foo, to ensure that works properly both for programs that only 
reference foo and for programs that define their own foo).

[Bug tree-optimization/60510] SLP blocks loop vectorization (with reduction)

2017-07-04 Thread ro at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60510

--- Comment #10 from Rainer Orth  ---
Created attachment 41675
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41675=edit
32-bit sparc-sun-solaris2.12 bb-slp-1.c.163t.slp1

  1   2   3   4   >