Re: [PATCH] Fix ICEs when -fsanitize={undefined,address,thread} is used during -c -flto but not during the final link (PR sanitizer/78708)

2016-12-09 Thread Richard Biener
On December 9, 2016 8:30:26 PM GMT+01:00, Jakub Jelinek wrote: >Hi! > >If some TUs are compiled with -flto -fsanitize={undefined,address} or >-fsanitize=thread, there might be some internal calls in them which >require >sanopt pass to lower them. If the final link is performed

Re: [PATCH] Fix tree-object-size for Fortran ARRAY_REFs (PR fortran/78758)

2016-12-09 Thread Richard Biener
On December 10, 2016 1:39:52 AM GMT+01:00, Jakub Jelinek wrote: >Hi! > >tree-object-size.c has been written with C-family FEs in mind (which is >where it has been solely used for initially), and as the testcases show >it doesn't work well with Fortran or other FE when it

[Bug c++/78761] New: C++14, compiler generate mangled name that can not be demangled

2016-12-09 Thread chihin.ko at oracle dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78761 Bug ID: 78761 Summary: C++14, compiler generate mangled name that can not be demangled Product: gcc Version: 6.1.0 Status: UNCONFIRMED Severity: normal

[Bug target/71112] [6/7 Regression] ICE with -fpie on aarch64 ILP32 big-endian

2016-12-09 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71112 Andrew Pinski changed: What|Removed |Added Status|ASSIGNED|NEW Known to work|

Re: Java language not being included as a possible language in --enable-languages argument to configure?

2016-12-09 Thread Andrew Pinski
On Fri, Dec 9, 2016 at 6:53 PM, Ethin Probst wrote: > I'm trying to run the GCC configure script with: > [configure input] > ~/gcc/configure --disable-nls --enable-gold=yes --enable-libquadmath > --enable-libquadmath-support --enable-libada --enable-libssp >

Java language not being included as a possible language in --enable-languages argument to configure?

2016-12-09 Thread Ethin Probst
I'm trying to run the GCC configure script with: [configure input] ~/gcc/configure --disable-nls --enable-gold=yes --enable-libquadmath --enable-libquadmath-support --enable-libada --enable-libssp --enable-libcxx --enable-liboffloadmic=target --enable-bootstrap --enable-lto --enable-static-libjava

Re: C++ PATCH to reject initializating flexible array members in constructors (PR c++/72775)

2016-12-09 Thread Martin Sebor
On 12/09/2016 02:49 PM, Jason Merrill wrote: On Fri, Dec 9, 2016 at 11:33 AM, Martin Sebor wrote: For flexible array members, because they're not in C++, we get to make up the rules that make the most sense to us. IMO, they should fit in well with the rest of the language.

Re: [PATCH] Fix A < 0 ? C : 0 optimization (PR tree-optimization/78720)

2016-12-09 Thread Jakub Jelinek
On Sat, Dec 10, 2016 at 12:04:25AM +0100, Marc Glisse wrote: > On Fri, 9 Dec 2016, Jakub Jelinek wrote: > > >This patch fixes the recently added A < 0 ? C : 0 for power of 2 C > >optimization. The if (!TYPE_UNSIGNED (TREE_TYPE (@0))) part is just > >for safety, I'd hope that unsigned < 0 is

[PATCH] Fix tree-object-size for Fortran ARRAY_REFs (PR fortran/78758)

2016-12-09 Thread Jakub Jelinek
Hi! tree-object-size.c has been written with C-family FEs in mind (which is where it has been solely used for initially), and as the testcases show it doesn't work well with Fortran or other FE when it started to be used everywhere recently. Fixed thusly, bootstrapped/regtested on x86_64-linux

[Bug fortran/70863] [F03] Finalization of array of derived type causes segfault

2016-12-09 Thread janus at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70863 --- Comment #3 from janus at gcc dot gnu.org --- (In reply to Dominique d'Humieres from comment #1) > related to/duplicate of pr69298? Yes, I think it's the same problem.

[Bug fortran/70863] [F03] Finalization of array of derived type causes segfault

2016-12-09 Thread janus at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70863 --- Comment #2 from janus at gcc dot gnu.org --- Slightly reduced test case: module b_module implicit none type :: b character(len=4) :: name = "none" contains final :: destroy_b end type contains impure elemental

[Bug middle-end/78760] New: strcpy buffer overflow not diagnosed with -Wstringop-overflow

2016-12-09 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78760 Bug ID: 78760 Summary: strcpy buffer overflow not diagnosed with -Wstringop-overflow Product: gcc Version: 7.0 Status: UNCONFIRMED Severity: normal

[Bug fortran/78737] [OOP] linking error with deferred, undefined user-defined derived-type I/O

2016-12-09 Thread damian at sourceryinstitute dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78737 --- Comment #6 from Damian Rouson --- Janus, thanks for your additional investigation and comments. Jerry, below is a better example in which I extend the abstract type and implement the deferred binding in the non-abstract child type but

[Bug fortran/59781] [5/6 Regression] [F03] Incorrect initialisation of derived type

2016-12-09 Thread janus at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59781 janus at gcc dot gnu.org changed: What|Removed |Added Known to work||7.0 Summary|[5/6/7

Re: [PATCH] Fix A < 0 ? C : 0 optimization (PR tree-optimization/78720)

2016-12-09 Thread Marc Glisse
On Fri, 9 Dec 2016, Jakub Jelinek wrote: This patch fixes the recently added A < 0 ? C : 0 for power of 2 C optimization. The if (!TYPE_UNSIGNED (TREE_TYPE (@0))) part is just for safety, I'd hope that unsigned < 0 is folded into 0 earlier, but just in case this is handled first. The issue

Re: [Patch][i386] PR 70118: Fix ubsan warning on SSE2 loadl_epi64 and storel_epi64

2016-12-09 Thread Allan Sandfeld Jensen
On Tuesday 06 December 2016, Uros Bizjak wrote: > Hello! > > > 2016-11-30 Allan Sandfeld Jensen > > > >PR target/70118 > >* gcc/config/i386/mmintrin.h (__m64_u): New type > >* gcc/config/i386/emmintrin.h (_mm_loadl_epi64, _mm_storel_epi64): > >

[Bug fortran/78758] [7 Regression] Warning: '__builtin_memcpy' ... overflows the destination for string assignment

2016-12-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78758 Jakub Jelinek changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

[Bug fortran/77903] [F08] gfortran 6.1.0/7.0.0 accept invalid code with conflicting module/submodule interfaces

2016-12-09 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77903 Paul Thomas changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug fortran/77903] [F08] gfortran 6.1.0/7.0.0 accept invalid code with conflicting module/submodule interfaces

2016-12-09 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77903 --- Comment #12 from Paul Thomas --- Author: pault Date: Fri Dec 9 22:25:26 2016 New Revision: 243507 URL: https://gcc.gnu.org/viewcvs?rev=243507=gcc=rev Log: 2016-12-09 Paul Thomas PR fortran/77903 *

[Bug libgcc/78759] New: __gcc_qadd could have been "more commutative" when a+c is infinity, but a+c+aa+cc is not

2016-12-09 Thread timshen at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78759 Bug ID: 78759 Summary: __gcc_qadd could have been "more commutative" when a+c is infinity, but a+c+aa+cc is not Product: gcc Version: 7.0 Status: UNCONFIRMED

[Bug fortran/78758] [7 Regression] Warning: '__builtin_memcpy' ... overflows the destination for string assignment

2016-12-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78758 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #2

[Bug testsuite/78740] test case powerpc/pr78691-ppc.c fails starting with its introduction in r243335

2016-12-09 Thread wschmidt at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78740 Bill Schmidt changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug testsuite/78740] test case powerpc/pr78691-ppc.c fails starting with its introduction in r243335

2016-12-09 Thread wschmidt at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78740 --- Comment #7 from Bill Schmidt --- Author: wschmidt Date: Fri Dec 9 22:02:04 2016 New Revision: 243506 URL: https://gcc.gnu.org/viewcvs?rev=243506=gcc=rev Log: 2016-12-09 Bill Schmidt PR

Re: libgo patch committed: Copy hash code from Go 1.7 runtime

2016-12-09 Thread Ian Lance Taylor
On Fri, Dec 9, 2016 at 2:09 AM, Sebastian Huber wrote: > On 08/12/16 17:37, Ian Lance Taylor wrote: >> >> +// For gccgo, use go:linkname to rename compiler-called functions to >> +// themselves, so that the compiler will export them. >> +// >> +//go:linkname

[Bug fortran/78758] [7 Regression] Warning: '__builtin_memcpy' ... overflows the destination for string assignment

2016-12-09 Thread anlauf at gmx dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78758 --- Comment #1 from Harald Anlauf --- Slightly reduced variant: integer function get_satid (name) character(len=*), intent(in) :: name character(len=16):: longname integer, external:: satid_longname ! longname(1:)

Re: [C++ Patch] PR 78637 ("ICE on invalid C++ code (internal compiler error: in pop_namespace, at cp/name-lookup.c:3826)")

2016-12-09 Thread Jason Merrill
OK. On Fri, Dec 2, 2016 at 9:20 AM, Paolo Carlini wrote: > Hi, > > this regression is caused by my fix for c++/60385, where I changed > push_namespace to early return (a bool) when pushdecl fails. In the present > testcase, a different push_namespace call in

Re: C++ PATCH to reject initializating flexible array members in constructors (PR c++/72775)

2016-12-09 Thread Jason Merrill
On Fri, Dec 9, 2016 at 11:33 AM, Martin Sebor wrote: > For flexible array members, because they're not in C++, we get to > make up the rules that make the most sense to us. IMO, they should > fit in well with the rest of the language. I disagree; we should support C code, but

Re: strange test failures

2016-12-09 Thread Martin Sebor
On 12/08/2016 05:32 PM, Jeff Law wrote: On 12/08/2016 04:47 PM, Martin Sebor wrote: I'm seeing failures in a few C tests that I'm not sure what to make out of. The tests fail due to undefined symbols while linking even though they're not meant to link. Among these are a number (but not all)

[Bug fortran/78758] New: [7 Regression] Warning: '__builtin_memcpy' ... overflows the destination for string assignment

2016-12-09 Thread anlauf at gmx dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78758 Bug ID: 78758 Summary: [7 Regression] Warning: '__builtin_memcpy' ... overflows the destination for string assignment Product: gcc Version: 7.0 Status: UNCONFIRMED

[Bug fortran/78757] New: ICE with function returning a pointer to a character

2016-12-09 Thread jb at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78757 Bug ID: 78757 Summary: ICE with function returning a pointer to a character Product: gcc Version: unknown Status: UNCONFIRMED Severity: normal Priority: P3

[PATCH] Add ISA 3.0 PowerPC support for VEXTU{B,H,W}{L,R}X instructions

2016-12-09 Thread Michael Meissner
This patch adds support to use the ISA 3.0 VEXTU{B,H,W}{L,R}X byte extract instructions in the PowerPC. These instructions extract a byte, 16-bit half-word, or 32-bit word from a vector register using a variable index in a general purpose register, and put the result in a general purpose

Re: [RFC] combine: Improve change_zero_ext, call simplify_set afterwards.

2016-12-09 Thread Segher Boessenkool
On Fri, Dec 09, 2016 at 08:32:01PM +0100, Dominik Vogt wrote: > > > This is a work in progress with the goal of fixing the first > > > problem and similar ones by calling simplify_set after > > > change_zero_ext to get rid of the overly complex code. That > > > works fine in principle,

Re: BITS_PER_UNIT issue in port....

2016-12-09 Thread Jeff Law
On 12/08/2016 01:37 PM, Steve Silva wrote: Hi, I was not sure where to look but I thought I would try this. I am in the middle of porting GCC 6.2 to an internal processor we are experimenting with. I have about 2 months into it and I have made some decent progress. The biggest issue I have

[Bug target/72742] [7 Regression] ICE in extract_insn, at recog.c:2309 (error: unrecognizable insn) w/ -Os -mlra

2016-12-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72742 Jakub Jelinek changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

Re: [PATCH] PR fortran/65173 -- kill off old_cl_list from gfc_namespace.

2016-12-09 Thread Steven G. Kargl
On Fri, Dec 09, 2016 at 01:54:34PM +0200, Janne Blomqvist wrote: > On Fri, Dec 9, 2016 at 1:47 PM, Paul Richard Thomas > wrote: > > I'm seeing the same thing, I guess. Or rather that the ts->type == 81 > which is non-sensical. No idea where that comes from.. > >

[Bug tree-optimization/78721] [7 Regression] ICE on valid code at -O2 and -O3 on x86_64-linux-gnu: in set_value_range, at tree-vrp.c:371

2016-12-09 Thread kugan at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78721 --- Comment #4 from kugan at gcc dot gnu.org --- Author: kugan Date: Fri Dec 9 19:47:10 2016 New Revision: 243501 URL: https://gcc.gnu.org/viewcvs?rev=243501=gcc=rev Log: gcc/testsuite/ChangeLog: 2016-12-09 Kugan Vivekanandarajah

Re: [PATCH] warn on overflow in calls to allocation functions (bugs 77531 and 78284)

2016-12-09 Thread Jakub Jelinek
On Fri, Dec 09, 2016 at 10:44:17AM -0700, Martin Sebor wrote: > On 12/09/2016 06:26 AM, Andreas Schwab wrote: > >FAIL: gcc.dg/attr-alloc_size-3.c (test for warnings, line 445) > >FAIL: gcc.dg/attr-alloc_size-3.c (test for excess errors) > >Excess errors: > >/daten/

Re: [PATCH] Add support for Fuchsia (OS)

2016-12-09 Thread Josh Conner
Sorry, that should have been: #define CC1_SPEC "%{!fno-pic:%{!fno-PIC:%{!fpic:%{!fPIC:" \ "%{!fno-pie:%{!fno-PIE:%{!fpie:%{!fPIE: -fPIE" (default to PIE, not PIC) On Fri, Dec 9, 2016 at 11:33 AM, Josh Conner wrote: > On 12/9/16 12:26 AM, Andrew

[PATCH] Fix ppc ICE with and3_imm*_dot (PR target/72742)

2016-12-09 Thread Jakub Jelinek
Hi! rs6000.md has 2 sets of patterns for and with comparison, the non-_mask ones have !rs6000_is_valid_and_mask (operands[2], mode) predicate and have an extra clobber for the case where the destination is not first cc reg, while the second set implicitly assumes the operand is a valid mask

[Bug target/72742] [7 Regression] ICE in extract_insn, at recog.c:2309 (error: unrecognizable insn) w/ -Os -mlra

2016-12-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72742 --- Comment #3 from Jakub Jelinek --- Author: jakub Date: Fri Dec 9 19:35:18 2016 New Revision: 243500 URL: https://gcc.gnu.org/viewcvs?rev=243500=gcc=rev Log: PR target/72742 * config/rs6000/rs6000.md (*and3_imm_mask_dot,

Re: [PATCH] Add support for Fuchsia (OS)

2016-12-09 Thread Josh Conner
On 12/9/16 12:26 AM, Andrew Pinski wrote: On Thu, Dec 8, 2016 at 2:55 PM, Josh Conner wrote: This patch adds support to gcc for the Fuchsia OS (https://fuchsia.googlesource.com/). OK for mainline? A few comments: +/* Build with PIC by default. */ +#undef CC1_SPEC

PR target/78213 revisited (was Re: [PATCH 5/9] Introduce selftest::locate_file (v4))

2016-12-09 Thread David Malcolm
On Fri, 2016-12-09 at 02:48 +0100, Bernd Schmidt wrote: > On 12/08/2016 10:47 PM, David Malcolm wrote: > > > Is (A) OK, or would you prefer (B)? (I prefer (A) fwiw) > > Oh well, just keep it as it is. > > > Bernd Thanks. Unfortunately, applying the "locate_file" patch

Re: [RFC] combine: Improve change_zero_ext, call simplify_set afterwards.

2016-12-09 Thread Dominik Vogt
On Fri, Dec 09, 2016 at 12:25:04PM -0600, Segher Boessenkool wrote: > Hi Dominik, > > On Fri, Dec 09, 2016 at 04:23:44PM +0100, Dominik Vogt wrote: > > 0001-* > > > > Deal with mode expanding zero_extracts in change_zero_ext. The > > patch looks good to me, but not sure whether endianness

[Bug target/78683] [ppc] __builtin_ctzl code gen can be improved for -mcpu=power8 and earlier

2016-12-09 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78683 --- Comment #1 from Segher Boessenkool --- Author: segher Date: Fri Dec 9 19:31:06 2016 New Revision: 243499 URL: https://gcc.gnu.org/viewcvs?rev=243499=gcc=rev Log: rs6000: clz/ctz/ffs improvement (PR78683) On CPUs that implement popcnt[wd]

[PATCH] Fix ICEs when -fsanitize={undefined,address,thread} is used during -c -flto but not during the final link (PR sanitizer/78708)

2016-12-09 Thread Jakub Jelinek
Hi! If some TUs are compiled with -flto -fsanitize={undefined,address} or -fsanitize=thread, there might be some internal calls in them which require sanopt pass to lower them. If the final link is performed without the corresponding -fsanitize=* options (user bug), that pass might not be run at

[PATCH] Fix A < 0 ? C : 0 optimization (PR tree-optimization/78720)

2016-12-09 Thread Jakub Jelinek
Hi! This patch fixes the recently added A < 0 ? C : 0 for power of 2 C optimization. The if (!TYPE_UNSIGNED (TREE_TYPE (@0))) part is just for safety, I'd hope that unsigned < 0 is folded into 0 earlier, but just in case this is handled first. The issue which breaks the testcase is that the

[Bug preprocessor/78680] [7 Regression] ICE in get_substring_ranges_for_loc, at input.c:1398

2016-12-09 Thread egall at gwmail dot gwu.edu
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78680 Eric Gallager changed: What|Removed |Added CC||egall at gwmail dot gwu.edu --- Comment

[Bug middle-end/78750] [7 Regression] ICE in get_range_info, at tree-ssanames.c:375

2016-12-09 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78750 Marek Polacek changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug middle-end/78750] [7 Regression] ICE in get_range_info, at tree-ssanames.c:375

2016-12-09 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78750 --- Comment #8 from Marek Polacek --- Author: mpolacek Date: Fri Dec 9 19:05:28 2016 New Revision: 243498 URL: https://gcc.gnu.org/viewcvs?rev=243498=gcc=rev Log: PR middle-end/78750 * builtins.c (check_sizes): Use

Re: Fix -Wstringop-overflow ICE with references (PR middle-end/78750)

2016-12-09 Thread Richard Biener
On December 9, 2016 4:27:19 PM GMT+01:00, Marek Polacek wrote: >The attached test crashed because we wound up with an SSA_NAME of >a reference type in get_range_info which has > > gcc_assert (!POINTER_TYPE_P (TREE_TYPE (name))); > >I think this could be fixed by treating

Re: Fix PR78725

2016-12-09 Thread Richard Biener
On December 9, 2016 4:29:04 PM GMT+01:00, Michael Matz wrote: >Hi, > >if the induction variable on which we want to partition the loop >iterations for loop splitting might overflow we would either need >runtime >tests checking if an overflow in fact does happen, or we can simply

[Bug bootstrap/26500] [4.2 Regression] info/gfortran.info is no longer being installed

2016-12-09 Thread mojca at macports dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=26500 Mojca Miklavec changed: What|Removed |Added CC||mojca at macports dot org --- Comment

[PATCH] PR78056: Finish fixing build failure on Power7

2016-12-09 Thread Kelvin Nilsen
This patch adds warning messages and test cases to an initial patch already submitted and committed to the trunk on October 26, 2016. The earlier patch disables initialization of built-in functions which depend on assembler capabilities that are not supported by the associated tool chain. The

[Bug target/70321] [6/7 Regression] STV generates less optimized code

2016-12-09 Thread ienkovich at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70321 --- Comment #14 from Ilya Enkovich --- (In reply to Jakub Jelinek from comment #13) > So shall we still consider moving 32-bit stv pass before the combiner > instead of after it? The argument about andn is no longer relevant, we have > a

Re: [RFC] combine: Improve change_zero_ext, call simplify_set afterwards.

2016-12-09 Thread Segher Boessenkool
Hi Dominik, On Fri, Dec 09, 2016 at 04:23:44PM +0100, Dominik Vogt wrote: > 0001-* > > Deal with mode expanding zero_extracts in change_zero_ext. The > patch looks good to me, but not sure whether endianness is > handled properly. Is the second argument of gen_rtx_SUBREG > correct?

Re: [0/67] Add wrapper classes for machine_modes

2016-12-09 Thread Sandra Loosemore
On 12/09/2016 05:48 AM, Richard Sandiford wrote: This series includes most of the changes in group C from: https://gcc.gnu.org/ml/gcc/2016-11/msg00033.html The idea is to add wrapper classes around machine_mode_enum for specific groups of modes, such as scalar integers, scalar floats,

[PATCH] correct %g handling with unknown arguments in -fprintf-return-value (PR 78696)

2016-12-09 Thread Martin Sebor
Bug 78696 points out a bug in the handling of the %g directive in the gimple-ssa-sprintf pass where precision wasn't being correctly handled. In the process of developing and testing a fix for it I noticed a few other subtle problems in the floating point handling of unknown values. The

[Bug fortran/78746] charlen_15.f90, charlen_03, charlen_10 ICE

2016-12-09 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78746 Dominique d'Humieres changed: What|Removed |Added Target|arm aarch64 | Status|UNCONFIRMED

[arm-embedded] [PATCH, GCC/ARM] Define arm_arch_core_flags in a single file

2016-12-09 Thread Thomas Preudhomme
We have decided to apply the patch proposed for trunk to solve the bloating of arm_arch_core_flags structure in GCC and files compiled by it to ARM embedded-6-branch. ChangeLog.arm entry is as follows: *** gcc/ChangeLog.arm *** 2016-12-09 Thomas Preud'homme

Re: [PATCH] warn on overflow in calls to allocation functions (bugs 77531 and 78284)

2016-12-09 Thread Martin Sebor
On 12/09/2016 06:26 AM, Andreas Schwab wrote: FAIL: gcc.dg/attr-alloc_size-3.c (test for warnings, line 445) FAIL: gcc.dg/attr-alloc_size-3.c (test for excess errors) Excess errors: /daten/aranym/gcc/gcc-20161209/gcc/testsuite/gcc.dg/attr-alloc_size-3.c:244:3: warning: product '65535 * 65535

Re: [PATCH] warn on overflow in calls to allocation functions (bugs 77531 and 78284)

2016-12-09 Thread Martin Sebor
On 12/09/2016 02:59 AM, Christophe Lyon wrote: On 9 December 2016 at 00:54, Martin Sebor wrote: +enabled with @option{-Wextra}. So I think we should in the immediate term not enable this in Wextra. However, I think for gcc-8 we should revisit after fixing GCC to be cleaner

Re: [PATCH] Fix PR78027

2016-12-09 Thread Alexander Monakov
On Fri, 9 Dec 2016, Cesar Philippidis wrote: > >> if (cgraph_node::get_create (decl)->offloadable > >> && !lookup_attribute ("omp declare target", > >> DECL_ATTRIBUTES (current_function_decl))) > >>{ > >> const char *target_attr = (is_gimple_omp_offloaded

Re: [nvptx] propagating conditionals in worker-vector partitioned loops

2016-12-09 Thread Bernd Schmidt
On 10/27/2016 12:29 AM, Cesar Philippidis wrote: Currently, the nvptx backend is only neutering the worker axis when propagating variables used in conditional expressions across the worker and vector axes. That's a problem with the worker-state spill and fill propagation implementation because

[Bug rtl-optimization/78255] [5/6/7 regression] Indirect sibling call causing wrong code generation for ARM

2016-12-09 Thread avieira at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78255 --- Comment #13 from avieira at gcc dot gnu.org --- Author: avieira Date: Fri Dec 9 17:22:20 2016 New Revision: 243496 URL: https://gcc.gnu.org/viewcvs?rev=243496=gcc=rev Log: PR78255: Make postreload aware of NO_FUNCTION_CSE

[arm-embedded][committed] PR78255: Make postreload aware of NO_FUNCTION_CSE

2016-12-09 Thread Andre Vieira (lists)
Hi I backported this patch to the embedded-6-branch in revision r243496. Cheers, Andre gcc/ChangeLog.arm: 2016-12-09 Andre Vieira Backport from mainline 2016-12-09 Andre Vieira PR

[Bug target/78748] [7 Regression] ICE in extract_insn, at recog.c:2311 (s390x-linux-gnu)

2016-12-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78748 --- Comment #3 from Jakub Jelinek --- Created attachment 40292 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40292=edit gcc7-pr78748.patch Untested fix (though, I don't have a quick setup to bootstrap/regtest it in the next week or so).

[Bug bootstrap/78756] New: Missing prefix in the name of gfortran.info

2016-12-09 Thread mojca at macports dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78756 Bug ID: 78756 Summary: Missing prefix in the name of gfortran.info Product: gcc Version: 6.2.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

[Bug fortran/78746] charlen_15.f90, charlen_03, charlen_10 ICE

2016-12-09 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78746 kargl at gcc dot gnu.org changed: What|Removed |Added CC||kargl at gcc dot gnu.org ---

[PATCH, GCC/ARM] Define arm_arch_core_flags in a single file

2016-12-09 Thread Thomas Preudhomme
Hi, This patch moves the definition of arm_arch_core_flags along with the declaration of its structure element type to common/config/arm/arm-common.c so that it is not *defined* in all object files including tm.h. Otherwise, GCC gets bloated with as many copy of that array as there is file

[Bug sanitizer/78755] Thread sanitizer reports race with std::promise with -O0, works with -O{1,2,3}

2016-12-09 Thread csaba_22 at yahoo dot co.uk
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78755 --- Comment #1 from Csaba Ráduly --- Created attachment 40291 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40291=edit Preprocessed source

Re: [PATCH, GCC/testsuite/ARM] Fix empty_fiq_handler target selector

2016-12-09 Thread Thomas Preudhomme
On 09/12/16 15:39, Kyrill Tkachov wrote: On 09/12/16 15:28, Thomas Preudhomme wrote: Hi, The current target selector for empty_fiq_handler.c testcase skips the test when targeting Thumb mode on a device with ARM execution state. Because it checks Thumb mode by looking for an -mthumb option it

[Bug sanitizer/78755] New: Thread sanitizer reports race with std::promise with -O0, works with -O{1,2,3}

2016-12-09 Thread csaba_22 at yahoo dot co.uk
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78755 Bug ID: 78755 Summary: Thread sanitizer reports race with std::promise with -O0, works with -O{1,2,3} Product: gcc Version: 6.2.0 Status: UNCONFIRMED Severity:

[Bug rtl-optimization/78255] [5/6/7 regression] Indirect sibling call causing wrong code generation for ARM

2016-12-09 Thread avieira at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78255 --- Comment #12 from avieira at gcc dot gnu.org --- Author: avieira Date: Fri Dec 9 16:46:42 2016 New Revision: 243494 URL: https://gcc.gnu.org/viewcvs?rev=243494=gcc=rev Log: PR78255: Make postreload aware of NO_FUNCTION_CSE gcc/ChangeLog:

[Ping] [C++ Patch] PR 78637 ("ICE on invalid C++ code (internal compiler error: in pop_namespace, at cp/name-lookup.c:3826)")

2016-12-09 Thread Paolo Carlini
Hi, gently pinging, might be rather straightforward... On 02/12/2016 15:20, Paolo Carlini wrote: Hi, this regression is caused by my fix for c++/60385, where I changed push_namespace to early return (a bool) when pushdecl fails. In the present testcase, a different push_namespace call in

Re: C++ PATCH to reject initializating flexible array members in constructors (PR c++/72775)

2016-12-09 Thread Martin Sebor
On 12/09/2016 08:34 AM, Nathan Sidwell wrote: On 12/09/2016 10:18 AM, Martin Sebor wrote: On 12/09/2016 08:09 AM, Jakub Jelinek wrote: On Fri, Dec 09, 2016 at 08:04:11AM -0700, Martin Sebor wrote: then (IMO) so is this: struct Foo { int a; char ary[]; Foo () : ary ("bob") {}

[Bug middle-end/78748] [7 Regression] ICE in extract_insn, at recog.c:2311 (s390x-linux-gnu)

2016-12-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78748 --- Comment #2 from Jakub Jelinek --- Started with r239685.

Re: [PATCH] PR78255: Make postreload aware of NO_FUNCTION_CSE

2016-12-09 Thread Bernd Schmidt
On 12/09/2016 05:16 PM, Andre Vieira (lists) wrote: Regardless, 'reload_cse_simplify' would never perform the opposite transformation. It checks whether it can replace anything within the first argument INSN, with the second argument TESTREG. As the name implies this will always be a register.

Re: [PATCH, rs6000] whitespace fix for p9-dimode tests

2016-12-09 Thread Will Schmidt
On Thu, 2016-12-08 at 20:55 -0600, Segher Boessenkool wrote: > Hello, > > On Thu, Dec 08, 2016 at 10:18:42AM -0600, Will Schmidt wrote: > > I am seeing some failures in the p9-dimode tests. This appears to > > be due to the scan-assembler strings matching comment portions of the > > generated

Re: [PATCH] omp-low.c split

2016-12-09 Thread Alexander Monakov
On Fri, 9 Dec 2016, Jakub Jelinek wrote: > Can you post an incremental patch fixing those issues? A few small nits I found while reading the patch. First of all, please use 'git diff --patience' (or --histogram) when generating such patches, without it the changes in omp-low.c look uglier than

Re: [PATCH] PR78255: Make postreload aware of NO_FUNCTION_CSE

2016-12-09 Thread Andre Vieira (lists)
On 09/12/16 16:02, Ramana Radhakrishnan wrote: > On Fri, Dec 9, 2016 at 3:58 PM, Bernd Schmidt wrote: >> On 12/09/2016 04:34 PM, Andre Vieira (lists) wrote: >> >>> Regardless, the other testcases I add in this patch show a sub-optimal >>> transformation done by postreload,

[Bug tree-optimization/78731] [5 Regression] Possible bug with switch when optimization is turned on.

2016-12-09 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78731 Jeffrey A. Law changed: What|Removed |Added CC||law at redhat dot com --- Comment #4

Re: [PATCH] Fix PR78027

2016-12-09 Thread Cesar Philippidis
On 12/09/2016 07:49 AM, Alexander Monakov wrote: > On Fri, 9 Dec 2016, Cesar Philippidis wrote: >>> Normally all offloaded code has either "omp declare target" (most >>> functions) or >>> "omp target entrypoint" (only toplevel offloaded code). This used to >>> include >>> OpenACC functions:

Re: [PATCH] PR78255: Make postreload aware of NO_FUNCTION_CSE

2016-12-09 Thread Ramana Radhakrishnan
On Fri, Dec 9, 2016 at 3:58 PM, Bernd Schmidt wrote: > On 12/09/2016 04:34 PM, Andre Vieira (lists) wrote: > >> Regardless, the other testcases I add in this patch show a sub-optimal >> transformation done by postreload, turning direct calls into indirect >> calls, for

[Bug c++/68903] missing default initialization of member when combined with virtual inheritance

2016-12-09 Thread spam at trzeci dot pl
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68903 --- Comment #1 from trzeci --- Created attachment 40290 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40290=edit simpler testcase should output: 3 3 3 usually outputs: 3 3 0 C::c remains uninitialized after construction

Re: [PATCH] PR78255: Make postreload aware of NO_FUNCTION_CSE

2016-12-09 Thread Jeff Law
On 12/09/2016 08:02 AM, Bernd Schmidt wrote: On 12/09/2016 03:03 PM, Andre Vieira (lists) wrote: This patch fixes the issue reported in PR78255 by making postreload aware it should not be performing CSE on functions if NO_FUNCTION_CSE is defined to true. Bootstrap and full regression on

Re: [PATCH, rs6000] Fix gcc.target/powerpc/pr78691-ppc.c

2016-12-09 Thread Segher Boessenkool
Hi Bill, On Fri, Dec 09, 2016 at 09:39:27AM -0600, Bill Schmidt wrote: > The fix for PR78691 added a new test case for 32-bit PowerPC. However, the > test needs to be restricted to 32-bit targets to avoid a FAIL. This patch > fixes that. > > Tested on powerpc64-unknown-linux-gnu. Ok for

[Bug target/72742] [7 Regression] ICE in extract_insn, at recog.c:2309 (error: unrecognizable insn) w/ -Os -mlra

2016-12-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72742 Jakub Jelinek changed: What|Removed |Added Status|NEW |ASSIGNED CC|

Re: [PATCH] PR78255: Make postreload aware of NO_FUNCTION_CSE

2016-12-09 Thread Bernd Schmidt
On 12/09/2016 04:34 PM, Andre Vieira (lists) wrote: Regardless, the other testcases I add in this patch show a sub-optimal transformation done by postreload, turning direct calls into indirect calls, for targets which have specifically pointed out that no CSE should be done on functions through

[c++/61636] generic lambda's potential capture of this

2016-12-09 Thread Nathan Sidwell
Jason, looking at 61636 I can't determine what the correct behaviour should be. I've reduced the testcase further to: struct A { void b (); void Foo (int); }; void A::b() { auto lam = [&](auto asdf) { Foo (asdf); }; lam (0); } which avoids nullptr_t. The compiler's confused

Re: parallel_algorithm_assert2.cc

2016-12-09 Thread Jonathan Wakely
On 08/12/16 14:41 -0800, David Edelsohn wrote: On Thu, Dec 8, 2016 at 10:03 AM, Jonathan Wakely wrote: On 8 December 2016 at 17:09, David Edelsohn wrote: Jonathan, I'm seeing a new failure on AIX. FAIL: 25_algorithms/headers/algorithm/parallel_algorithm_assert2.cc

Re: [PATCH] Fix PR78027

2016-12-09 Thread Alexander Monakov
On Fri, 9 Dec 2016, Cesar Philippidis wrote: > > Normally all offloaded code has either "omp declare target" (most > > functions) or > > "omp target entrypoint" (only toplevel offloaded code). This used to > > include > > OpenACC functions: they would have "oacc fnattrib" _in_ _addition_ to

Re: [PATCH] Fix PR78027

2016-12-09 Thread Jakub Jelinek
On Fri, Dec 09, 2016 at 07:37:53AM -0800, Cesar Philippidis wrote: > It was more so to point out that there were cases where OpenACC > offloaded regions may not have omp target attributes. Referring back to > case 2 in the original email: > > 2. Nested offloaded regions inside 'omp declare

[Bug middle-end/78748] [7 Regression] ICE in extract_insn, at recog.c:2311 (s390x-linux-gnu)

2016-12-09 Thread aldyh at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78748 Aldy Hernandez changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

Re: [PATCH] PR78255: Make postreload aware of NO_FUNCTION_CSE

2016-12-09 Thread Wilco Dijkstra
Bernd wrote: > Hmm, it probably doesn't hurt, but looking at the PR I think the originally > reported problem > suggests you need a different fix: a separate register class to be used for > indirect sibling calls. > I remember seeing similar issues on other targets. The only safe way to

[Bug c++/61636] generic lambda: segfault / "cannot call member function without object"

2016-12-09 Thread nathan at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61636 Nathan Sidwell changed: What|Removed |Added CC||nathan at gcc dot gnu.org --- Comment

[Bug target/72861] [7 Regression] 25% tramp3d-v4 performance regression on ppc64le

2016-12-09 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72861 Markus Trippelsdorf changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

Re: [PATCH, GCC/testsuite/ARM] Fix empty_fiq_handler target selector

2016-12-09 Thread Kyrill Tkachov
On 09/12/16 15:28, Thomas Preudhomme wrote: Hi, The current target selector for empty_fiq_handler.c testcase skips the test when targeting Thumb mode on a device with ARM execution state. Because it checks Thumb mode by looking for an -mthumb option it fails to work when GCC was configured

[PATCH, rs6000] Fix gcc.target/powerpc/pr78691-ppc.c

2016-12-09 Thread Bill Schmidt
Hi, The fix for PR78691 added a new test case for 32-bit PowerPC. However, the test needs to be restricted to 32-bit targets to avoid a FAIL. This patch fixes that. Tested on powerpc64-unknown-linux-gnu. Ok for trunk? Thanks, Bill 2016-12-09 Bill Schmidt

Re: [PATCH] Fix PR78027

2016-12-09 Thread Cesar Philippidis
On 12/09/2016 06:56 AM, Alexander Monakov wrote: > On Thu, 8 Dec 2016, Jeff Law wrote: >>> PR fortran/78027 >>> >>> gcc/ >>> * ipa-icf.c (sem_function::parse): Don't process functions with >>> oacc decl attributes, as they may be OpenACC routines. >>> >>> gcc/testsuite/ >>>

[Bug c/25733] missed diagnostic about assignment used as truth value.

2016-12-09 Thread vincent-gcc at vinc17 dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=25733 --- Comment #13 from Vincent Lefèvre --- Another example: int f1 (int a, int b, int c, int d) { if (a = c && b == d) return 0; else return 1; } int f2 (int a, int b, int c, int d) { if (a == b || (a = c && b == d)) return 0;

  1   2   3   4   >