Re: [Patch, fortran] PR58754 - [4.9/5 Regression] ICE on allocating character array with source

2015-10-22 Thread FX
> 2015-10-22 Paul Thomas > >PR fortran/58754 >* trans-stmt.c (gfc_trans_allocate): Do not use the scalar >character assignment if the allocate expression sis an array >descriptor. > > 2015-10-22 Paul Thomas > >PR fortran/58754 >

Re: [OpenACC 1/11] UNIQUE internal function

2015-10-22 Thread Nathan Sidwell
On 10/22/15 10:26, Richard Biener wrote: On Thu, Oct 22, 2015 at 4:01 PM, Nathan Sidwell wrote: On 10/22/15 04:07, Richard Biener wrote: Yeah, please make them either end or start a BB so we have to check at most a single stmt. ECF_RETURNS_TWICE should achieve that, it also

Re: [gomp4 00/14] NVPTX: further porting

2015-10-22 Thread Bernd Schmidt
On 10/22/2015 08:08 PM, Alexander Monakov wrote: On Thu, 22 Oct 2015, Bernd Schmidt wrote: I'm not really familiar with OpenMP and what it allows, so take all my comments with a grain of salt. On 10/22/2015 06:41 PM, Alexander Monakov wrote: The second approach is to run all threads in the

RE: [PATCH] Disable -mbranch-likely for -Os when targetting generic architecture

2015-10-22 Thread Moore, Catherine
> -Original Message- > From: Robert Suchanek [mailto:robert.sucha...@imgtec.com] > Sent: Friday, September 04, 2015 10:21 AM > To: Matthew Fortune; Richard Sandiford > Cc: Moore, Catherine; gcc-patches@gcc.gnu.org > Subject: RE: [PATCH] Disable -mbranch-likely for -Os when targetting

Re: UTF-8 quotation marks in diagnostics

2015-10-22 Thread Martin Sebor
On 10/22/2015 10:53 AM, Joseph Myers wrote: On Wed, 21 Oct 2015, Martin Sebor wrote: That would go against the usual (i.e., POSIX) expected effect of the environment variable. Specifically for GCC (or the c99 utility), POSIX requires LC_CTYPE to determine the locale used to parse the input,

[Bug c++/67870] ICE: in type_throw_all_p, at cp/except.c:1306

2015-10-22 Thread paolo.carlini at oracle dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67870 Paolo Carlini changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug fortran/58754] [4.9/5 Regression] ICE on allocating character array with source

2015-10-22 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58754 Paul Thomas changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

Re: [PATCH] Fix sanitizer/68042

2015-10-22 Thread Jeff Law
On 10/22/2015 10:45 AM, Maxim Ostapenko wrote: Hi, currently we have memcmp-1.c and sanity-check-pure-c-1.c (ASan testsuite) output pattern test failures on x86_64-apple-darwin14. This patch adjusts their expecting patterns to match real ones. Tested by me on x86_64-unknown-linux-gnu and by

Re: [PATCH] Fix PR rtl-optimization/67736 in combine.c

2015-10-22 Thread Steve Ellcey
On Thu, 2015-10-22 at 13:46 -0500, Segher Boessenkool wrote: > One nit and maybe a problem: > > > --- a/gcc/testsuite/gcc.dg/combine-subregs.c > > +++ b/gcc/testsuite/gcc.dg/combine-subregs.c > > @@ -0,0 +1,36 @@ > > +/* { dg-do run } */ > > +/* { dg-options "-O2 -fexpensive-optimizations" } */

Re: [PATCH v2 08/13] Add TARGET_ADDR_SPACE_ZERO_ADDRESS_VALID

2015-10-22 Thread Richard Biener
On Thu, Oct 22, 2015 at 5:12 AM, Sandra Loosemore wrote: > On 10/20/2015 03:27 PM, Richard Henderson wrote: >> >> >> +@deftypefn {Target Hook} bool TARGET_ADDR_SPACE_ZERO_ADDRESS_VALID >> (addr_space_t @var{as}) >> +Define this to modify the default handling of address 0

Re: [OpenACC 1/11] UNIQUE internal function

2015-10-22 Thread Jakub Jelinek
On Thu, Oct 22, 2015 at 09:49:29AM +0200, Richard Biener wrote: > >> Jakub, IYR I originally had IFN_FORK and IFN_JOIN as such distinct internal > >> fns. This replaces that scheme. > >> > >> ok? > > > > Hmm, I'd just have used gimple_has_volatile_ops on the call? That > > should have the > >

[Bug c++/67989] [REGRESSION] g++ ICE on armel valid code

2015-10-22 Thread paolo.carlini at oracle dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67989 Paolo Carlini changed: What|Removed |Added CC||rguenther at suse dot de --- Comment #3

[Bug target/68015] ICE in s390_emit_compare

2015-10-22 Thread krebbel at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68015 --- Comment #3 from Andreas Krebbel --- Author: krebbel Date: Thu Oct 22 08:24:01 2015 New Revision: 229163 URL: https://gcc.gnu.org/viewcvs?rev=229163=gcc=rev Log: S/390: PR68015 Fix ICE in s390_emit_compare gcc/ChangeLog: 2015-10-22

[Committed] S/390: PR68015 Fix ICE in s390_emit_compare

2015-10-22 Thread Andreas Krebbel
Committed to head and GCC 5 branch gcc/ChangeLog: 2015-10-22 Andreas Krebbel PR target/68015 * config/s390/s390.md (movcc): Emit compare only if we don't already have a comparison result. gcc/testsuite/ChangeLog: 2015-10-22 Andreas

[Bug sanitizer/68042] [6 Regression] c-c++-common/asan/(memcmp-1.c|sanity-check-pure-c-1.c) failures on x86_64-apple-darwin14 after r229111

2015-10-22 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68042 Richard Biener changed: What|Removed |Added Target Milestone|--- |6.0

[Bug c++/60976] Compilation with G++ 4.9.0 is 2-3 times slower than with 4.8.2

2015-10-22 Thread rene.koec...@wincor-nixdorf.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60976 --- Comment #28 from Rene Koecher --- (In reply to Giuseppe Ottaviano from comment #26) Giuseppe, is there an easy way you could provide me with your changes to alloc_traits.h? I'd really like to give it a shot

Re: Proposed doc update for Explicit Reg Vars 3/3

2015-10-22 Thread David Wohlferd
Given this, I'm going to go ahead and re-work the local register variables page (probably tomorrow) stating extended asm is the only supported usage. Although I also think it's important to mention Andrew's point. If someone sees it in code somewhere, at least the docs will give them some

Re: Add non-constant vector ctors to operand_equal_p

2015-10-22 Thread Richard Biener
On Thu, 22 Oct 2015, Jan Hubicka wrote: > Hi, > this patch adds matching of non-constant CONSTRUCTOR expressions into > operand_equal_p. As discussed with Richard, those can happen when we are > building vectors out of components. I also added a testcase that triggers > this > path and gets

[Bug middle-end/68046] -ftrapv doesn't catch leaq-based overflows on x86-64

2015-10-22 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68046 --- Comment #4 from Richard Biener --- Note that -fsanitize-undefined doesn't properly handle volatile int x = __INT_MAX__ + 1; which is in PR61893, it handles the case in this PR fine and with trap-on-error more efficiently than -ftrapv:

[Bug c++/67989] [REGRESSION] g++ ICE on armel valid code

2015-10-22 Thread sylvestre at debian dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67989 Sylvestre Ledru changed: What|Removed |Added CC||sylvestre at debian dot org ---

Re: [PATCH 1/9] ENABLE_CHECKING refactoring

2015-10-22 Thread Richard Biener
On Wed, Oct 21, 2015 at 6:27 PM, Jakub Jelinek wrote: > On Wed, Oct 21, 2015 at 06:22:37PM +0200, Bernd Schmidt wrote: >> On 10/21/2015 06:18 PM, Jeff Law wrote: >> >To avoid conditionally compiled code. I'm of the opinion we should be >> >stomping out as much as we reasonably

Re: [OpenACC 1/11] UNIQUE internal function

2015-10-22 Thread Richard Biener
On Thu, Oct 22, 2015 at 10:04 AM, Jakub Jelinek wrote: > On Wed, Oct 21, 2015 at 03:00:47PM -0400, Nathan Sidwell wrote: >> To distinguish different uses of the UNIQUE function, I use the first >> argument, which is expected to be an INTEGER_CST. I figured this better >> than

[Bug middle-end/61893] Constant folding inhibits trapping with -ftrapv

2015-10-22 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61893 Richard Biener changed: What|Removed |Added CC||eggert at gnu dot org --- Comment #3

[Bug middle-end/68046] -ftrapv doesn't catch leaq-based overflows on x86-64

2015-10-22 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68046 Richard Biener changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug target/68015] ICE in s390_emit_compare

2015-10-22 Thread krebbel at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68015 --- Comment #4 from Andreas Krebbel --- Author: krebbel Date: Thu Oct 22 08:28:57 2015 New Revision: 229165 URL: https://gcc.gnu.org/viewcvs?rev=229165=gcc=rev Log: S/390: PR68015 Fix ICE in s390_emit_compare gcc/ChangeLog: 2015-10-22

Re: Handle CONSTRUCTOR in operand_equal_p

2015-10-22 Thread Richard Biener
On Wed, 21 Oct 2015, Jan Hubicka wrote: > Hi, > this is updated patch I am going to commit. As discussed, we also need to > match > non-empty CONSTRUCTOR of vectors, but those should never be having CONSTANT > flags > set, so they need care in the other path trhough operand_equal_p, so I will

Re: [OpenACC 1/11] UNIQUE internal function

2015-10-22 Thread Jakub Jelinek
On Wed, Oct 21, 2015 at 03:00:47PM -0400, Nathan Sidwell wrote: > To distinguish different uses of the UNIQUE function, I use the first > argument, which is expected to be an INTEGER_CST. I figured this better > than using multiple new internal fns, all with the unique property, as the > latter

Re: [OpenACC 1/11] UNIQUE internal function

2015-10-22 Thread Richard Biener
On Thu, Oct 22, 2015 at 9:59 AM, Jakub Jelinek wrote: > On Thu, Oct 22, 2015 at 09:49:29AM +0200, Richard Biener wrote: >> >> Jakub, IYR I originally had IFN_FORK and IFN_JOIN as such distinct >> >> internal >> >> fns. This replaces that scheme. >> >> >> >> ok? >> > >> > Hmm,

Re: Add VIEW_CONVERT_EXPR to operand_equal_p

2015-10-22 Thread Andreas Schwab
+ /* Changes in machine mode are never useless conversions unless. */ Unless what? Andreas. -- Andreas Schwab, SUSE Labs, sch...@suse.de GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7 "And now for something completely different."

Re: [OpenACC 3/11] new target hook

2015-10-22 Thread Jakub Jelinek
On Wed, Oct 21, 2015 at 03:13:26PM -0400, Nathan Sidwell wrote: > 2015-10-20 Nathan Sidwell > > * target.def (fork_join): New GOACC hook. > * targhooks.h (default_goacc_fork_join): Declare. > * omp-low.c (default_goacc_forkjoin): New. > *

Re: [PATCH v2 10/13] Avoid CSE of MEMs in different address spaces

2015-10-22 Thread Richard Biener
On Wed, Oct 21, 2015 at 7:12 PM, Richard Henderson wrote: > On 10/21/2015 03:37 AM, Jeff Law wrote: >> >> On 10/20/2015 03:27 PM, Richard Henderson wrote: >>> >>> --- >>> gcc/cselib.c | 22 >>> +- >>> gcc/fold-const.c

[Bug target/68018] [6 Regression] ICE: in ix86_compute_frame_layout, at config/i386/i386.c:11308 with -mstackrealign

2015-10-22 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68018 Uroš Bizjak changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[PATCH] Move MINUS of pointer-plus expr foldings to match.pd

2015-10-22 Thread Richard Biener
Bootstrapped and tested on x86_64-unknown-linux-gnu, the OMP changes were approved by Jakub on IRC. Richard. 2015-10-22 Richard Biener * fold-const.c (fold_addr_of_array_ref_difference): Properly convert operands before folding a MINUS_EXPR.

Re: [OpenACC 5/11] C++ FE changes

2015-10-22 Thread Jakub Jelinek
On Wed, Oct 21, 2015 at 03:18:55PM -0400, Nathan Sidwell wrote: > This patch is the C++ changes matching the C ones of patch 4. In > finish_omp_clauses, the gang, worker, & vector clauses are handled the same > as OpenMP's 'num_threads' clause. One change to num_threads is the > augmentation of

[Bug target/67989] [5/6 Regression] g++ ICE on armel valid code

2015-10-22 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67989 Richard Biener changed: What|Removed |Added Target||arm* Priority|P3

[Bug target/68018] [6 Regression] ICE: in ix86_compute_frame_layout, at config/i386/i386.c:11308 with -mstackrealign

2015-10-22 Thread uros at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68018 --- Comment #6 from uros at gcc dot gnu.org --- Author: uros Date: Thu Oct 22 06:52:00 2015 New Revision: 229162 URL: https://gcc.gnu.org/viewcvs?rev=229162=gcc=rev Log: Backport from mainline 2015-10-21 Uros Bizjak

Re: [OpenACC 2/11] PTX backend changes

2015-10-22 Thread Jakub Jelinek
On Wed, Oct 21, 2015 at 03:09:55PM -0400, Nathan Sidwell wrote: > Bernd, any comments? Just a few questions, otherwise it is a PTX territory you PTX maintainers should review. > (*oacc_ntid_insn, oacc_ntid, *oacc_tid_insn, oacc_tid): Delete. Extra space. > +/* Size of buffer needed to

Re: [OpenACC 6/11] Reduction initialization

2015-10-22 Thread Jakub Jelinek
On Wed, Oct 21, 2015 at 03:24:13PM -0400, Nathan Sidwell wrote: > 2015-10-20 Nathan Sidwell > > * omp-low.c (oacc_init_rediction_array): New. > (oacc_initialize_reduction_data): Initialize array. Ok. Jakub

[Bug c++/68051] copy constructor does not copy default initialized union member

2015-10-22 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68051 Markus Trippelsdorf changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug c++/66583] incorrect implicitly-defined move constructor for class with anonymous union and NSDMI

2015-10-22 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66583 Markus Trippelsdorf changed: What|Removed |Added CC||akju at mailinator dot com ---

[Bug fortran/68054] New: ICE on using protected attribute in program without program statement

2015-10-22 Thread gerhard.steinmetz.fort...@t-online.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68054 Bug ID: 68054 Summary: ICE on using protected attribute in program without program statement Product: gcc Version: 5.2.1 Status: UNCONFIRMED Severity: normal

Re: [PATCH] unconditionally compile most of the delay slot code

2015-10-22 Thread Jeff Law
On 10/22/2015 11:34 AM, Steve Ellcey wrote: On Wed, 2015-10-21 at 11:13 -0400, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders Hi, $subject bootstrapped+ regtested x86_64-linux-gnu, I wouldn't mind a second pair of eyes on this one given its not totally

Re: [gomp4 00/14] NVPTX: further porting

2015-10-22 Thread Alexander Monakov
On Thu, 22 Oct 2015, Julian Brown wrote: > > The second approach is to run all threads in the warp all the time, > > making sure they execute the same code with the same data, and thus > > build up the same local state. In this case we'd need to ensure this > > invariant: if threads in the warp

Re: [PATCH 3/9] ENABLE_CHECKING refactoring: Java and Ada

2015-10-22 Thread Jeff Law
On 10/05/2015 05:30 PM, Mikhail Maltsev wrote: gcc/java/ChangeLog: 2015-10-05 Mikhail Maltsev * decl.c (java_mark_decl_local): Use flag_checking instead of ENABLE_CHECKING. gcc/ada/ChangeLog: 2015-10-05 Mikhail Maltsev *

[Bug fortran/68055] ICE on using unsupported kinds in program without program statement

2015-10-22 Thread gerhard.steinmetz.fort...@t-online.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68055 --- Comment #1 from Gerhard Steinmetz --- Simplified. Still different behaviour : $ cat z3.f90 !program p integer*3 c print *, c end $ gfortran -g -O0 -Wall -fcheck=all z3.f90 z3.f90:2:0:

Re: [PATCH][optabs.c] Fix PR 67989: Handle const0_rtx target in expand_atomic_compare_and_swap

2015-10-22 Thread Jeff Law
On 10/22/2015 10:24 AM, Kyrill Tkachov wrote: Hi all, The ICE in this PR (reduced from LLVM) occurs because the atomic_load builtin expansion ends up calling expand_atomic_compare_and_swap without caring about the result. The convention in the optabs/builtin machinery is to use const0_rtx for

Re: [PATCH] New attribute to create target clones

2015-10-22 Thread Evgeny Stupachenko
PING. On Thu, Oct 15, 2015 at 12:32 AM, Evgeny Stupachenko wrote: > Bootstrap and make check for x86 passed. No new fails. > Please ignore an empty line added to omp-low.c in the patch, the > misprint will be removed prior to a commit. > > Thanks, > Evgeny > > On Tue, Oct 13,

[Bug fortran/58754] [4.9/5 Regression] ICE on allocating character array with source

2015-10-22 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58754 --- Comment #14 from Paul Thomas --- Author: pault Date: Thu Oct 22 18:08:31 2015 New Revision: 229180 URL: https://gcc.gnu.org/viewcvs?rev=229180=gcc=rev Log: 2015-10-22 Paul Thomas PR fortran/58754 *

Re: [gomp4 00/14] NVPTX: further porting

2015-10-22 Thread Alexander Monakov
On Thu, 22 Oct 2015, Bernd Schmidt wrote: > I'm not really familiar with OpenMP and what it allows, so take all my > comments with a grain of salt. > > On 10/22/2015 06:41 PM, Alexander Monakov wrote: > > The second approach is to run all threads in the warp all the time, making > > sure they

Re: [Patch, fortran] PR58754 - [4.9/5 Regression] ICE on allocating character array with source

2015-10-22 Thread Paul Richard Thomas
Thanks FX! Committed to 5 branch as revision 229179 and the testcase to 4_9 as revision 229180. Cheers Paul On 22 October 2015 at 19:32, FX wrote: >> 2015-10-22 Paul Thomas >> >>PR fortran/58754 >>* trans-stmt.c (gfc_trans_allocate): Do not

[OpenACC 0/7] host_data construct

2015-10-22 Thread James Norris
Hi! This patch series adds the handling of OpenACC host_data construct in the C and C++ front-ends, gimple, and supporting functions in libgomp. Commentary on the changes is included in each individual patch. All of the code is in the gomp-4_0-branch. Regtested on

[Bug fortran/67805] ICE on array constructor with wrong character specification

2015-10-22 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67805 kargl at gcc dot gnu.org changed: What|Removed |Added CC||kargl at gcc dot gnu.org

[OpenACC 5/7] host_data construct (gcc tests)

2015-10-22 Thread James Norris
gcc/testsuite/c-c++-common/goacc/host_data-1.c b/gcc/testsuite/c-c++-common/goacc/host_data-1.c gcc/testsuite/c-c++-common/goacc/host_data-2.c b/gcc/testsuite/c-c++-common/goacc/host_data-2.c gcc/testsuite/c-c++-common/goacc/host_data-3.c

[OpenACC 6/7] host_data construct

2015-10-22 Thread James Norris
libgomp/libgomp.map b/libgomp/libgomp.map - Add new symbol GOACC_deviceptr. libgomp/oacc-mem.c b/libgomp/oacc-mem.c - Add new function GOACC_deviceptr() to handle pointer lookup for host_data regions. diff --git a/libgomp/libgomp.map b/libgomp/libgomp.map index

[OpenACC 4/7] host_data construct (middle end)

2015-10-22 Thread James Norris
gcc/gimplify.c b/gcc/gimplify.c - Add new enum for use_device clause handling to gimplify_omp_var_data. - Add new enum for host_data regions to omp_region_type. - Move handling of use_device clause in gimplify_scan_omp_clauses(). - Add new functions

[PATCH] Fix abort in write_eligible_delay

2015-10-22 Thread Jeff Law
Various ports are currently failing to build, faulting in write_eligible_delay. This can happen if the target has delay slots defined, but does not have annul-true or annul-false slots. cris is a reasonable example. This is most easily addressed by writing out a trivial dummy function if

[Bug rtl-optimization/67609] [5/6 Regression] Generates wrong code for SSE2 _mm_load_pd

2015-10-22 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67609 --- Comment #21 from Jeffrey A. Law --- On 10/22/2015 07:40 AM, rguenth at gcc dot gnu.org wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67609 > > --- Comment #20 from Richard Biener --- > (In reply to Jeffrey A. Law from comment #19)

Re: [PATCH] Fix PR rtl-optimization/67736 in combine.c

2015-10-22 Thread Segher Boessenkool
On Thu, Oct 22, 2015 at 09:47:23AM -0700, Steve Ellcey wrote: > OK to checkin? > 2015-10-21 Steve Ellcey > Andrew Pinski > > PR rtl-optimization/67736 > * combine.c (simplify_comparison): Use gen_lowpart_or_truncate instead >

Re: [PATCH v2 08/13] Add TARGET_ADDR_SPACE_ZERO_ADDRESS_VALID

2015-10-22 Thread Richard Henderson
On 10/21/2015 05:12 PM, Sandra Loosemore wrote: On 10/20/2015 03:27 PM, Richard Henderson wrote: +@deftypefn {Target Hook} bool TARGET_ADDR_SPACE_ZERO_ADDRESS_VALID (addr_space_t @var{as}) +Define this to modify the default handling of address 0 for the +address space. Return true if 0 should

Re: [testsuite] Fix potential race conditions in gfortran tests

2015-10-22 Thread Maxim Kuvyrkov
Hi, And this is a similar patch for gfortran.dg/streamio_N.f90 tests. OK to commit? -- Maxim Kuvyrkov www.linaro.org > On Oct 18, 2015, at 4:17 PM, Mikael Morin wrote: > > Le 16/10/2015 10:08, Christophe Lyon a écrit : >> Hi, >> >> We have noticed a few random

[Bug middle-end/67989] [4.9/5/6 Regression] g++ ICE on armel valid code

2015-10-22 Thread ktkachov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67989 --- Comment #10 from ktkachov at gcc dot gnu.org --- Proposed patch at: https://gcc.gnu.org/ml/gcc-patches/2015-10/msg02299.html

Re: [gomp4 00/14] NVPTX: further porting

2015-10-22 Thread Julian Brown
On Thu, 22 Oct 2015 19:41:51 +0300 Alexander Monakov wrote: > On Thu, 22 Oct 2015, Jakub Jelinek wrote: > > Does that apply also to threads within a warp? I.e. is .local > > local to each thread in the warp, or to the whole warp, and if the > > former, how can say at the

[Bug fortran/58754] [4.9/5 Regression] ICE on allocating character array with source

2015-10-22 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58754 --- Comment #13 from Paul Thomas --- Author: pault Date: Thu Oct 22 18:05:21 2015 New Revision: 229179 URL: https://gcc.gnu.org/viewcvs?rev=229179=gcc=rev Log: 2015-10-22 Paul Thomas PR fortran/58754 *

[Bug c++/68056] New: can use keyword as " _Atomic" as an identifier

2015-10-22 Thread zhijian at ca dot ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68056 Bug ID: 68056 Summary: can use keyword as " _Atomic" as an identifier Product: gcc Version: 4.8.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

[OpenACC 1/7] host_data construct (C/C++ common)

2015-10-22 Thread James Norris
gcc/c-family/c-pragma.c b/gcc/c-family/c-pragma.c - Add host_data pragma definition to list in oacc_pragma[]. gcc/c-family/c-pragma.h b/gcc/c-family/c-pragma.h - Add host_data pragma enum to list in pragma_kind[]. - Add use_device clause enum to list in

Re: [PATCH] Fix abort in write_eligible_delay

2015-10-22 Thread Trevor Saunders
On Thu, Oct 22, 2015 at 01:18:29PM -0600, Jeff Law wrote: > > Various ports are currently failing to build, faulting in > write_eligible_delay. > > This can happen if the target has delay slots defined, but does not have > annul-true or annul-false slots. cris is a reasonable example. > > This

[Bug fortran/68055] New: ICE on using unsupported kinds in program without program statement

2015-10-22 Thread gerhard.steinmetz.fort...@t-online.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68055 Bug ID: 68055 Summary: ICE on using unsupported kinds in program without program statement Product: gcc Version: 5.2.1 Status: UNCONFIRMED Severity: normal

Re: [OpenACC 1/11] UNIQUE internal function

2015-10-22 Thread Nathan Sidwell
On 10/22/15 04:07, Richard Biener wrote: Yeah, please make them either end or start a BB so we have to check at most a single stmt. ECF_RETURNS_TWICE should achieve that, it also makes it a code motion barrier. I'm having a hard time making UNIQUE the end of a BB. I'm emitting code to a

[gomp4] OpenACC async clause regressions (was: [gomp4.1] Add new versions of GOMP_target{,_data,_update} and GOMP_target_enter_exit_data)

2015-10-22 Thread Thomas Schwinge
Hi! On Mon, 19 Oct 2015 18:24:35 +0200, I wrote: > Chung-Lin, would you please have a look at the following (on > gomp-4_0-branch)? Also, anyone else got any ideas off-hand? Ilya, Jakub, thanks for your comments! > On Tue, 23 Jun 2015 13:51:39 +0200, Jakub Jelinek wrote: > >

Re: [PATCH 0/4] OpenMP 4.0 offloading to Intel MIC

2015-10-22 Thread Ilya Verbin
On Mon, Dec 22, 2014 at 13:01:40 +0100, Thomas Schwinge wrote: > By chance (when tracking down a different problem), I've found the > following. Would you please check whether that's a real problem in > liboffloadmic, or its libgomp plugin, or just a mis-diagnosis by > Valgrind? > >

[OpenACC 3/7] host_data construct (C front-end)

2015-10-22 Thread James Norris
gcc/cp/cp-tree.h b/gcc/cp/cp-tree.h - Add definition for c_finish_oacc_host_data(). gcc/cp/parser.c b/gcc/cp/parser.c - Add handling of use_device clause in cp_parser_omp_clause_name(). - Add handling of use_device clause in cp_parser_oacc_all_clauses().

[OpenACC 2/7] host_data construct (C FE)

2015-10-22 Thread James Norris
gcc/c/c-parser.c b/gcc/c/c-parser.c - Add handling of use_device clause in c_parser_omp_clause_name(). - Add new function c_parser_oacc_clause_use_device() to handle use_device clause with host_data. - Add handling of use_device clause in

Re: [gomp4 00/14] NVPTX: further porting

2015-10-22 Thread Alexander Monakov
On Thu, 22 Oct 2015, Bernd Schmidt wrote: > On 10/22/2015 08:08 PM, Alexander Monakov wrote: > > On Thu, 22 Oct 2015, Bernd Schmidt wrote: > > > > > I'm not really familiar with OpenMP and what it allows, so take all my > > > comments with a grain of salt. > > > > > > On 10/22/2015 06:41 PM,

Re: [PATCH] gcc/fold-const.c: Correct the report warning position.

2015-10-22 Thread Jeff Law
On 10/21/2015 04:31 PM, Chen Gang wrote: Hello all: It is for bug63510, which reported by another members (not me), I guess, this patch should fix this bug. Welcome any other members' ideas, suggestions, and completions. Note that the call to fold_binary from tree-ssa-sccvn.c has been

Re: [Patch, fortran] PR58754 - [4.9/5 Regression] ICE on allocating character array with source

2015-10-22 Thread Paul Richard Thomas
Dear All, Only the testcase will be applied to 4.9. At some time, it seems to have fixed itself! Cheers Paul On 22 October 2015 at 16:31, Paul Richard Thomas wrote: > Dear All, > > This patch speaks for itself. It is by no means as comprehensive as > the work on

Re: [gomp4 00/14] NVPTX: further porting

2015-10-22 Thread Bernd Schmidt
I'm not really familiar with OpenMP and what it allows, so take all my comments with a grain of salt. On 10/22/2015 06:41 PM, Alexander Monakov wrote: The second approach is to run all threads in the warp all the time, making sure they execute the same code with the same data, and thus build

Re: [PATCH] unconditionally compile most of the delay slot code

2015-10-22 Thread Steve Ellcey
On Wed, 2015-10-21 at 11:13 -0400, tbsaunde+...@tbsaunde.org wrote: > From: Trevor Saunders > > Hi, > > $subject > > bootstrapped+ regtested x86_64-linux-gnu, I wouldn't mind a second pair of > eyes > on this one given its not totally trivial. > > Trev > >

[OpenACC 7/7] host_data construct (runtime tests)

2015-10-22 Thread James Norris
libgomp/testsuite/libgomp.oacc-c-c++-common/host_data-1.c b/libgomp/testsuite/libgomp.oacc-c-c++-common/host_data-1.c libgomp/testsuite/libgomp.oacc-c-c++-common/host_data-2.c b/libgomp/testsuite/libgomp.oacc-c-c++-common/host_data-2.c - New runtime tests for host_data. diff

[Bug c++/60976] Compilation with G++ 4.9.0 is 2-3 times slower than with 4.8.2

2015-10-22 Thread ott at fb dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60976 --- Comment #29 from Giuseppe Ottaviano --- (In reply to Rene Koecher from comment #28) > (In reply to Giuseppe Ottaviano from comment #26) > > Giuseppe, is there an easy way you could provide me with your changes to > alloc_traits.h? > > I'd

Re: [OpenACC 1/11] UNIQUE internal function

2015-10-22 Thread Richard Biener
On Thu, Oct 22, 2015 at 9:48 AM, Richard Biener wrote: > On Wed, Oct 21, 2015 at 9:00 PM, Nathan Sidwell wrote: >> This patch implements a new internal function that has a 'uniqueness' >> property. Jump-threading cannot clone it and tail-merging

[Bug debug/68010] internal compiler error: in tree_to_shwi, at tree.h:3661

2015-10-22 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68010 Markus Trippelsdorf changed: What|Removed |Added CC||jason at gcc dot gnu.org ---

Re: [OpenACC 1/11] UNIQUE internal function

2015-10-22 Thread Richard Biener
On Wed, Oct 21, 2015 at 9:00 PM, Nathan Sidwell wrote: > This patch implements a new internal function that has a 'uniqueness' > property. Jump-threading cannot clone it and tail-merging cannot combine > multiple instances. > > The uniqueness is implemented by a new gimple fn, >

[Bug rtl-optimization/67609] [5/6 Regression] Generates wrong code for SSE2 _mm_load_pd

2015-10-22 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67609 --- Comment #18 from Richard Biener --- (In reply to Jeffrey A. Law from comment #16) > reload has traditionally removed subregs of hardregs and passes after reload > have depended on that behaviour. Doing something similar in lra is >

Re: [OpenACC 4/11] C FE changes

2015-10-22 Thread Jakub Jelinek
On Wed, Oct 21, 2015 at 03:16:20PM -0400, Nathan Sidwell wrote: > 2015-10-20 Cesar Philippidis > Thomas Schwinge > James Norris > Joseph Myers >

[Bug c/67488] Incorrect error message on call of nested functions

2015-10-22 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67488 Marek Polacek changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

Re: [vec-cmp, patch 1/6] Add optabs for vector comparison

2015-10-22 Thread Ilya Enkovich
2015-10-21 20:25 GMT+03:00 Jeff Law : > On 10/08/2015 08:52 AM, Ilya Enkovich wrote: >> >> Hi, >> >> This series introduces autogeneration of vector comparison and its support >> on i386 target. It lets comparison statements to be vectorized into vector >> comparison instead of

Re: [0/7] Type promotion pass and elimination of zext/sext

2015-10-22 Thread Kugan
On 21/10/15 23:45, Richard Biener wrote: > On Tue, Oct 20, 2015 at 10:03 PM, Kugan > wrote: >> >> >> On 07/09/15 12:53, Kugan wrote: >>> >>> This a new version of the patch posted in >>> https://gcc.gnu.org/ml/gcc-patches/2015-08/msg00226.html. I have done >>>

Re: [OpenACC 1/11] UNIQUE internal function

2015-10-22 Thread Julian Brown
On Thu, 22 Oct 2015 10:05:30 +0200 Richard Biener wrote: > On Thu, Oct 22, 2015 at 9:59 AM, Jakub Jelinek > wrote: > > On Thu, Oct 22, 2015 at 09:49:29AM +0200, Richard Biener wrote: > >> >> Jakub, IYR I originally had IFN_FORK and IFN_JOIN as

[Bug c++/67354] [5/6 regression] internal compiler error: in add_to_same_comdat_group, at symtab.c:421

2015-10-22 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67354 Markus Trippelsdorf changed: What|Removed |Added CC||jason at gcc dot gnu.org ---

[Bug c++/67354] [5/6 regression] internal compiler error: in add_to_same_comdat_group, at symtab.c:421

2015-10-22 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67354 --- Comment #6 from Markus Trippelsdorf --- No, the statement is correct: markus@x4 tmp % echo | g++ -fabi-version=9 -dM -E - | grep -i abi #define __GXX_ABI_VERSION 1009 markus@x4 tmp % echo | g++ -fabi-version=0 -dM -E - | grep -i abi #define

[Bug middle-end/68046] -ftrapv doesn't catch leaq-based overflows on x86-64

2015-10-22 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68046 --- Comment #5 from Richard Biener --- Author: rguenth Date: Thu Oct 22 11:44:11 2015 New Revision: 229170 URL: https://gcc.gnu.org/viewcvs?rev=229170=gcc=rev Log: 2015-10-22 Richard Biener PR middle-end/68046

[PATCH] Fix PR68046 (part of PR61893)

2015-10-22 Thread Richard Biener
This splits out the non-constant folding part of an earlier approved patch to make -ftrapv work better. The constant folding case is still broken (also ubsan doesn't handle it correctly). Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2015-10-22 Richard

[Bug middle-end/61893] Constant folding inhibits trapping with -ftrapv

2015-10-22 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61893 --- Comment #4 from Richard Biener --- Author: rguenth Date: Thu Oct 22 11:44:11 2015 New Revision: 229170 URL: https://gcc.gnu.org/viewcvs?rev=229170=gcc=rev Log: 2015-10-22 Richard Biener PR middle-end/68046

[Bug middle-end/61893] Constant folding inhibits trapping with -ftrapv

2015-10-22 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61893 --- Comment #5 from Richard Biener --- Note only the non-constant-folding part is fixed (the REG_EQUAL notes which enable combine to remove the trapping).

Re: [PATCH, VECTOR ABI] Add __attribute__((__simd__)) to GCC.

2015-10-22 Thread Kirill Yukhin
Hello, On 15 Oct 17:47, Kirill Yukhin wrote: > Hi Jakub, > On 15 Oct 16:39, Jakub Jelinek wrote: > > On Thu, Oct 15, 2015 at 05:33:32PM +0300, Kirill Yukhin wrote: > > > --- a/gcc/doc/extend.texi > > > +++ b/gcc/doc/extend.texi > > > @@ -3066,6 +3066,20 @@ This function attribute make a stack

[Bug target/67989] [5/6 Regression] g++ ICE on armel valid code

2015-10-22 Thread ktkachov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67989 --- Comment #6 from ktkachov at gcc dot gnu.org --- Created attachment 36561 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=36561=edit Reduced testcase Attaching reduced testcase, here's the full ICE backtrace. Can be reproduced with -O2

[Bug target/67989] [4.9/5/6 Regression] g++ ICE on armel valid code

2015-10-22 Thread ktkachov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67989 --- Comment #8 from ktkachov at gcc dot gnu.org --- Also, fails only for the arm-linux targets. arm-none-eabi compiles fine. Also, needs an -march option of armv6z or lower. armv6kz, armv7-a and higher don't ICE

Move fold_trunc_transparent_mathfn to match.pd

2015-10-22 Thread Richard Sandiford
This moves the fold rules for trunc, floor, ceil, round, nearbyint and rint in one go, since they're tested as a group. Most of the code is supporting the f(x)->x fold when x is known to be integer-valued. Like with the non-negative test, this is probably more elegantly handled by tracking range

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-22 Thread Martin Liška
On 10/21/2015 04:06 PM, Richard Biener wrote: > On Wed, Oct 21, 2015 at 1:24 PM, Martin Liška wrote: >> On 10/21/2015 11:59 AM, Richard Biener wrote: >>> On Wed, Oct 21, 2015 at 11:19 AM, Martin Liška wrote: On 10/20/2015 03:39 PM, Richard Biener wrote: >

[Bug c++/67354] [5/6 regression] internal compiler error: in add_to_same_comdat_group, at symtab.c:421

2015-10-22 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67354 --- Comment #4 from Markus Trippelsdorf --- It was fixed on trunk by r228017. And indeed -fabi-version=5 ICEs, 6 or higher is fine.

  1   2   3   4   >