RE: [PATCH] PR 60586

2015-09-01 Thread Iyer, Balaji V
Hi Jeff, I thought about this for a minute and I don't think I need to use the lang_hooks. I could do this change right before calling gimplify_cilk_spawn. I have attached the fixed patch and have answered your questions below. Here are the ChangeLog entries: gcc/c-family/ChangeLog: 201

[PATCH GCC]Look into unnecessary conversion when checking mult_op in get_shiftadd_cost

2015-09-01 Thread Bin Cheng
Hi, When calling get_shiftadd_cost, the mult_op is stripped at caller places. We should look into unnecessary conversion in op1 before checking equality, otherwise it computes wrong shiftadd cost. This patch picks this small issue up. Bootstrap and test on x86_64 and aarch64 along with other patc

[PATCH GCC]Try to fold (long)(A-B) into (long)A - (long)B for canonicalization in tree affine

2015-09-01 Thread Bin Cheng
Hi, Generally we don't try to fold (long)(A-B) into (long)A - (long)B because it results in more operations. On the other hand, this fold is wanted when we want to explore as many canonical opportunities as possible. Tree affine is definitely such a place. This patch supports this in tree_to_aff

Re: [RS6000] Don't pass --oformat to ld

2015-09-01 Thread David Edelsohn
On Tue, Sep 1, 2015 at 9:35 PM, Alan Modra wrote: > bugzilla.redhat.com/show_bug_cgi?id=1255946 shows that gcc built with > both powerpc64-linux and powerpc64le-linux support passes wrong linker > options when trying to link in the non-default endian. A --oformat > option coming from LINK_TARGET_

[PATCH PR66388]Add sizetype cand for BIV of smaller type if it's used as index of memory ref

2015-09-01 Thread Bin Cheng
Hi, This patch is a new approach to fix PR66388. IVO today computes iv_use with iv_cand which has at least same type precision as the use. On 64bit platforms like AArch64, this results in different iv_cand created for each address type iv_use, and register pressure increased. As a matter of fact

[RS6000] Don't pass --oformat to ld

2015-09-01 Thread Alan Modra
bugzilla.redhat.com/show_bug_cgi?id=1255946 shows that gcc built with both powerpc64-linux and powerpc64le-linux support passes wrong linker options when trying to link in the non-default endian. A --oformat option coming from LINK_TARGET_SPEC is only correct for 32-bit. It turns out that GNU ld

RE: [PATCH] PR 60586

2015-09-01 Thread Iyer, Balaji V
> -Original Message- > From: Iyer, Balaji V > Sent: Tuesday, September 1, 2015 6:17 PM > To: 'Jeff Law'; gcc-patches@gcc.gnu.org > Cc: Zamyatin, Igor > Subject: RE: [PATCH] PR 60586 > > > > > -Original Message- > > From: Jeff Law [mailto:l...@redhat.com] > > Sent: Tuesday, Sept

RE: [PATCH] PR 60586

2015-09-01 Thread Iyer, Balaji V
> -Original Message- > From: Jeff Law [mailto:l...@redhat.com] > Sent: Tuesday, September 1, 2015 3:26 PM > To: Iyer, Balaji V; gcc-patches@gcc.gnu.org > Cc: Zamyatin, Igor > Subject: Re: [PATCH] PR 60586 > > On 08/31/2015 06:04 PM, Iyer, Balaji V wrote: > > Hello Everyone, > > This

Go patch committed: handle out of range constant integer to string conversion

2015-09-01 Thread Ian Lance Taylor
This patch by Chris Manghane fixes the Go compiler to correctly handle an integer to string conversion when the integer is an out of range constant. This fixes https://golang.org/issue/11525 . Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian Index: gcc/g

Re: [PING] Re: [PATCH] c/66516 - missing diagnostic on taking the address of a builtin function

2015-09-01 Thread Joseph Myers
On Tue, 1 Sep 2015, Martin Sebor wrote: > I also noticed uses of DECL_LANG_FLAG_4 in the definitions of > what appear to be C-specific macros in c-family/c-common.h, > and then uses of the same macro in definitions of a C++-specific > macro in cp/cp-tree.h. That seems like a bug waiting to happen

fixed misplaced testcase

2015-09-01 Thread Kenneth Zadeck
2015-09-01 Kenneth Zadeck * gcc.c-torture/execute/ieee/2320-1.c Fixed misplaced test case. This was approved offline by Mike Stump. committed as revision 227389. Kenny --- gcc/testsuite/gcc.c-torture/execute/ieee/2320-1.c (revision 227385) +++ gcc/testsuite/gcc.c-torture/exe

Re: [gomp4] useless reduction locks and other bug fixes

2015-09-01 Thread Cesar Philippidis
[Attaching patch this time.] On 09/01/2015 04:21 PM, Cesar Philippidis wrote: > This patch teaches lower_oacc_reductions not to generate calls to > GOACC_{UN}LOCK if they aren't any reductions. That situation can happen > when there is a fake gang reduction on a private variable. > > I also found

[gomp4] useless reduction locks and other bug fixes

2015-09-01 Thread Cesar Philippidis
This patch teaches lower_oacc_reductions not to generate calls to GOACC_{UN}LOCK if they aren't any reductions. That situation can happen when there is a fake gang reduction on a private variable. I also found a bug where the lower_rec_input_clauses expects there to be a data mapping for the reduc

RE: [PATCH, MIPS] Compact branch support for MIPS32R6/MIPS64R6

2015-09-01 Thread Matthew Fortune
Moore, Catherine writes: > Hi Matthew: > > > -Original Message- > > From: Matthew Fortune [mailto:matthew.fort...@imgtec.com] > > Sent: Monday, August 17, 2015 6:47 PM > > To: Moore, Catherine; 'gcc-patches@gcc.gnu.org' (gcc-patches@gcc.gnu.org) > > Subject: RE: [PATCH, MIPS] Compact bran

Re: [Fortran, committed] XFAIL read_dir.f90 on FreeBSD

2015-09-01 Thread Jerry DeLisle
On 09/01/2015 11:18 AM, Steve Kargl wrote: > On Tue, Sep 01, 2015 at 11:16:27AM -0700, Steve Kargl wrote: >> open(unit=10, file='junko.dir',iostat=ios,action='read',access='stream') >> if (ios.ne.0) call abort >> read(10, iostat=ios) c >> - if (ios.ne.21) call abort >> + if (ios.ne.

Re: [PATCH] PR 60586

2015-09-01 Thread Jeff Law
On 08/31/2015 06:04 PM, Iyer, Balaji V wrote: Hello Everyone, This patch will fix the bug reported in Bugzilla, PR 60586. The issue was that the spawned function's function arguments must not be pushed into the nested/lambda function. This patch should fix that issue. I have tested thi

Re: [PING] Re: [PATCH] c/66516 - missing diagnostic on taking the address of a builtin function

2015-09-01 Thread Martin Sebor
On 09/01/2015 11:29 AM, Joseph Myers wrote: On Tue, 1 Sep 2015, Martin Sebor wrote: Attached is an updated patch that avoids diagnosing taking the address of implicitly declared library builtins like abs, bootstrapped and tested on ppc64le with no regressions. The tweak below was added to reje

Re: [PATCH][optabs][ifcvt][1/3] Define negcc, notcc optabs

2015-09-01 Thread Jeff Law
On 09/01/2015 09:04 AM, Kyrill Tkachov wrote: Hi all, This first patch introduces the negcc and notcc optabs that should expand to a conditional negate or a conditional bitwise complement operation. These are used in ifcvt.c to transform code of the form: if (test) x = -A; else x = A; into: x =

Re: [PATCH v2] [libstdc++] Run tests on RTEMS

2015-09-01 Thread Jeff Law
On 09/01/2015 05:02 AM, Sebastian Huber wrote: v2: Include all options and not only "dg-do run ...". libstdc++-v3/ChangeLog 2015-09-01 Sebastian Huber testsuite/*: Use 's/\*-\*-cygwin\* /&*-*-rtems* /' to add RTEMS target selector to all tests that run on Cygwin. So presumab

Re: [PATCH] [ARM, Callgraph] Fix PR67280: function incorrectly marked as nothrow

2015-09-01 Thread Jeff Law
On 08/28/2015 09:03 AM, Charles Baylis wrote: Hi This patch is an attempt to fix https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67280. I have written up an analysis of the bug there. When cgraph_node::create_wrapper() updates the callgraph for the new function, it sets the can_throw_external flag

Re: [Patch, libfortran] PR 67414 Improve error handling

2015-09-01 Thread Janne Blomqvist
On Tue, Sep 1, 2015 at 1:04 AM, FX wrote: >> the attached patch improves the error handling for backtrace failing, >> by printing the error number or the error string in addition to the >> message. It also fixes a potential null pointer crash in gf_strerror. >> >> Regtested on x86_64-pc-linux-gnu,

patch for PR61578

2015-09-01 Thread Vladimir Makarov
The following patch is for https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61578 The patch was bootstrapped and tested on x86 and x86-64. Committed as rev. 227382. 2015-09-01 Vladimir Makarov PR target/61578 * lra-lives.c (process_bb_lives): Process move pseudos with the

Re: [Fortran, committed] XFAIL read_dir.f90 on FreeBSD

2015-09-01 Thread Steve Kargl
On Tue, Sep 01, 2015 at 11:16:27AM -0700, Steve Kargl wrote: > open(unit=10, file='junko.dir',iostat=ios,action='read',access='stream') > if (ios.ne.0) call abort > read(10, iostat=ios) c > - if (ios.ne.21) call abort > + if (ios.ne.21) then > + close(10) I forgot to mention

[Fortran, committed] XFAIL read_dir.f90 on FreeBSD

2015-09-01 Thread Steve Kargl
I've committed the patch that follows my .sig. 2015-09-01 Steven G. Kargl * gfortran.dg/read_dir.f90: XFAIL this testcase on FreeBSD. Clean-up a created directory if testcase fails. I suspect that this testcase will fail on all *BSD OS's. -- Steve Index: gfortran.dg/read_di

Re: [PATCH] [libstdc++] Run tests on RTEMS

2015-09-01 Thread Mike Stump
On Sep 1, 2015, at 3:05 AM, Sebastian Huber wrote: > libstdc++-v3/ChangeLog > 2015-09-01 Sebastian Huber > >testsuite/*: Use 's/dg-do run.*\*-\*-cygwin\* /&*-*-rtems* /' to >add RTEMS target selector to all tests that run on Cygwin. Wow, seems unfortunate. Why not define a selector

Re: [PING] Re: [PATCH] c/66516 - missing diagnostic on taking the address of a builtin function

2015-09-01 Thread Joseph Myers
On Tue, 1 Sep 2015, Martin Sebor wrote: > Attached is an updated patch that avoids diagnosing taking the address > of implicitly declared library builtins like abs, bootstrapped and > tested on ppc64le with no regressions. > > The tweak below was added to reject_gcc_builtin make it possible. > Si

Re: [PATCH][wwwdocs][AArch64] Add entry for target attributes and pragmas

2015-09-01 Thread Gerald Pfeifer
On Tue, 1 Sep 2015, Kyrill Tkachov wrote: This wwwdocs patch adds an entry to the GCC 6 changes page about the aarch64 target attributes and pragmas support. Thanks for thinking of this, Kyrill. Index: htdocs/gcc-6/changes.html ==

Re: [PATCH] add initial support for J2 core to sh target

2015-09-01 Thread Rich Felker
On Wed, Sep 02, 2015 at 01:24:55AM +0900, Oleg Endo wrote: > > I'm not sure what the best way to achieve multiple goals is, but the > > current behavior makes it so you need --isa=any (and a final binary > > with weird ABI tag) to have a binary that supports atomic operations > > on any SH model. m

Re: Location of "dg-final" directives? (was Re: [PATCH][GCC] Algorithmic optimization in match and simplify)

2015-09-01 Thread Marek Polacek
On Tue, Sep 01, 2015 at 12:50:27PM -0400, David Malcolm wrote: > I can't comment on the patch itself, but I noticed that in the testsuite > addition, you've gathered all the "dg-final" clauses at the end. > > I think that this is consistent with existing practice in gcc, but > AFAIK, the dg-final

Location of "dg-final" directives? (was Re: [PATCH][GCC] Algorithmic optimization in match and simplify)

2015-09-01 Thread David Malcolm
On Fri, 2015-08-28 at 17:56 +0100, Andre Vieira wrote: [..snip..] > diff --git a/gcc/testsuite/gcc.dg/tree-ssa/forwprop-33.c > b/gcc/testsuite/gcc.dg/tree-ssa/forwprop-33.c > new file mode 100644 > index > ..984d8b37a01defe0e6852070a7dfa7ace5027c51 > --- /d

Re: [gomp4] declare directive

2015-09-01 Thread Tom de Vries
On 12/08/15 20:31, James Norris wrote: diff --git a/gcc/cp/pt.c b/gcc/cp/pt.c index 056b2c1..8ace93c 100644 --- a/gcc/cp/pt.c +++ b/gcc/cp/pt.c @@ -13907,6 +13907,7 @@ tsubst_expr (tree t, tree args, tsubst_flags_t complain, tree in_decl, integral_constant_expression_p) tree

Re: [gomp4, wip] remove references to ganglocal shared memory inside gcc

2015-09-01 Thread Tom de Vries
On 27/08/15 03:37, Cesar Philippidis wrote: - tree ganglocal_size - = gimple_call_arg (goacc_kernels_internal, /* TODO */ 6); - gimple_omp_target_set_ganglocal_size (stmt, ganglocal_size); This caused a bootstrap failure. Committed as attached. Thanks, - Tom Remove unused var i

Re: [gomp4, wip] remove references to ganglocal shared memory inside gcc

2015-09-01 Thread Tom de Vries
On 27/08/15 03:37, Cesar Philippidis wrote: - ctx->ganglocal_size_host = align_and_expand (&gl_host, host_size, align); I suspect this caused a bootstrap failure (align_and_expand unused). Worked-around as attached. Thanks, - Tom Mark align_and_expand as unused 2015-09-01 Tom de Vries

Re: [PATCH] add initial support for J2 core to sh target

2015-09-01 Thread Oleg Endo
On 01 Sep 2015, at 23:18, Rich Felker wrote: > On Tue, Sep 01, 2015 at 10:45:10PM +0900, Oleg Endo wrote: >> It seems that this J2 atomic instruction(s ?) is not available to >> the public. I've skimmed through the currently available J2 hardware >> sources, but couldn't find anything about it.

Re: [gomp4] New reduction infrastructure for OpenACC

2015-09-01 Thread Tom de Vries
On 01/09/15 14:53, Tom de Vries wrote: On 19/08/15 21:42, Cesar Philippidis wrote: +DEFHOOK +(reduction, +"This hook is used by the oacc_transform pass to expand calls to the\n\ +internal functions @var{GOACC_REDUCTION_SETUP},\n\ +@var{GOACC_REDUCTION_INIT},\n\ @var{GOACC_REDUCTION_FINI} and\n\

[gomp4] ptx assembler formatting

2015-09-01 Thread Nathan Sidwell
In looking at some ptx output, OCD kicked in and I couldn't tolerate the formatting inconsistencies. Fixed with this patch. Also, the mechanism of providing scratch regs to the spin lock and reset insns, caused the optimizers to want to insert initializations. Fixed by making these patterns

Re: [PATCH] Fix ICE when generating a vector shift by scalar

2015-09-01 Thread Bill Schmidt
On Tue, 2015-09-01 at 11:01 +0200, Richard Biener wrote: > On Mon, Aug 31, 2015 at 10:28 PM, Bill Schmidt > wrote: > > Hi, > > > > The following simple test fails when attempting to convert a vector > > shift-by-scalar into a vector shift-by-vector. > > > > typedef unsigned char v16ui __attribut

Re: [Patch, libstdc++] Fix data races in basic_string implementation

2015-09-01 Thread Dmitry Vyukov
On Tue, Sep 1, 2015 at 5:08 PM, Jonathan Wakely wrote: > On 01/09/15 16:56 +0200, Dmitry Vyukov wrote: >> >> I don't understand how a new gcc may not support __atomic builtins on >> ints. How it is even possible? That's a portable API provided by >> recent gcc's... > > > The built-in function is a

Re: [c++-delayed-folding] fold_simple

2015-09-01 Thread Kai Tietz
2015-09-01 17:31 GMT+02:00 Jason Merrill : > On 09/01/2015 11:27 AM, Kai Tietz wrote: >> >> I rewrote binary/unary overflow-check logic so, that we avoid double >> checking-s. I think this address things as you intend, beside the >> checking for constant value. We would need to check for *_CST >>

Re: [c++-delayed-folding] fold_simple

2015-09-01 Thread Jason Merrill
On 09/01/2015 11:27 AM, Kai Tietz wrote: I rewrote binary/unary overflow-check logic so, that we avoid double checking-s. I think this address things as you intend, beside the checking for constant value. We would need to check for *_CST tree-codes. Is there a macro we could use, which is just

Re: [c++-delayed-folding] fold_simple

2015-09-01 Thread Kai Tietz
2015-09-01 16:47 GMT+02:00 Jason Merrill : > On 08/31/2015 03:43 PM, Kai Tietz wrote: >> >> 2015-08-31 21:29 GMT+02:00 Jason Merrill : >>> >>> On 08/31/2015 03:08 PM, Kai Tietz wrote: I will need to verify that this patch doesn't introduce regressions. The wacky thing here is th

Re: [committed] taking transactional credits

2015-09-01 Thread Aldy Hernandez
On 09/01/2015 08:05 AM, Patrick Marlier wrote: On Tue, Sep 1, 2015 at 4:10 PM, Aldy Hernandez wrote: On 09/01/2015 07:03 AM, Richard Henderson wrote: On 09/01/2015 06:55 AM, Aldy Hernandez wrote: If you don't like it, sue me. =) Share the blame... ;-) That's right! I ain't going dow

[PATCH][ARM][3/3] Implement negsicc, notsicc optabs

2015-09-01 Thread Kyrill Tkachov
[Resending with patch attached] Hi all, This third patch implements the new optabs for arm. Conveniently, we can reuse the recently refactored *if_neg_move pattern and extend it to cover the conditional NOT case. Although arm has conditional execution capabilities I have found that performing th

Re: [PATCH][ARM][3/3] Implement negsicc, notsicc optabs

2015-09-01 Thread pinskia
> On Sep 1, 2015, at 11:04 PM, Kyrill Tkachov wrote: > > Hi all, > > This third patch implements the new optabs for arm. > Conveniently, we can reuse the recently refactored *if_neg_move pattern > and extend it to cover the conditional NOT case. > Although arm has conditional execution capab

Re: [Patch, libstdc++] Fix data races in basic_string implementation

2015-09-01 Thread Jonathan Wakely
On 01/09/15 16:56 +0200, Dmitry Vyukov wrote: I don't understand how a new gcc may not support __atomic builtins on ints. How it is even possible? That's a portable API provided by recent gcc's... The built-in function is always defined, but it might expand to a call to an external function in

Re: [committed] taking transactional credits

2015-09-01 Thread Patrick Marlier
On Tue, Sep 1, 2015 at 4:10 PM, Aldy Hernandez wrote: > On 09/01/2015 07:03 AM, Richard Henderson wrote: >> >> On 09/01/2015 06:55 AM, Aldy Hernandez wrote: >>> >>> If you don't like it, sue me. =) >> >> >> Share the blame... ;-) > > > That's right! I ain't going down in posterity alone on this.

[PATCH][AArch64][2/3] Implement negcc, notcc optabs

2015-09-01 Thread Kyrill Tkachov
Hi all, This second patch implements the new optabs for aarch64. The new expander is almost identical to the movcc expander except that operand 2 has a neg or a not before it to reflect the fact that it should be negated if the comparison in operand 1 holds. These patterns will eventually match t

[PATCH][ARM][3/3] Implement negsicc, notsicc optabs

2015-09-01 Thread Kyrill Tkachov
Hi all, This third patch implements the new optabs for arm. Conveniently, we can reuse the recently refactored *if_neg_move pattern and extend it to cover the conditional NOT case. Although arm has conditional execution capabilities I have found that performing the transformation from patch 1/3 e

[PATCH][optabs][ifcvt][1/3] Define negcc, notcc optabs

2015-09-01 Thread Kyrill Tkachov
Hi all, This first patch introduces the negcc and notcc optabs that should expand to a conditional negate or a conditional bitwise complement operation. These are used in ifcvt.c to transform code of the form: if (test) x = -A; else x = A; into: x = A; if (test) x = -x; where the "if (test) x =

[PATCH][optabs][ifcvt][0/3] Define negcc, notcc optabs and use for if-conversion

2015-09-01 Thread Kyrill Tkachov
Hi all, This patch series aims to improve the use of conditional negate and conditional invert instructions when if-converting code of the form: x := test ? -a : a; or x := test ? ~a : a; I'd like to generate something like: x := a; if (test) x := -x; where the latter if-statement can be implemen

[PATCH, PR67405, committed] Avoid NULL pointer dereference

2015-09-01 Thread Ilya Enkovich
Hi, This fixes an ICE by adding a NULL check. Bootstrapped and regtested for x86_64-unknown-linux-gnu. Applied to trunk. Does this need to be ported to gcc-5-branch? Thanks, Ilya -- gcc/ 2015-09-01 Ilya Enkovich PR target/67405 * tree-chkp.c (chkp_find_bound_slots_1): Ad

Re: [PING] Re: [PATCH] c/66516 - missing diagnostic on taking the address of a builtin function

2015-09-01 Thread Martin Sebor
Attached is an updated patch that avoids diagnosing taking the address of implicitly declared library builtins like abs, bootstrapped and tested on ppc64le with no regressions. The tweak below was added to reject_gcc_builtin make it possible. Since the expression is in c-family/c-common.c, the mo

Re: [Patch, libstdc++] Fix data races in basic_string implementation

2015-09-01 Thread Dmitry Vyukov
On Tue, Sep 1, 2015 at 4:27 PM, Jonathan Wakely wrote: > On 01/09/15 14:51 +0200, Dmitry Vyukov wrote: >> >> Hello, >> >> The refcounted basic_string implementation contains several data races >> on _M_refcount: >> 1. _M_is_leaked loads _M_refcount concurrently with mutations of >> _M_refcount. Th

[PATCH] Add __builtin_argument_pointer

2015-09-01 Thread H.J. Lu
On Wed, Aug 19, 2015 at 3:35 PM, Segher Boessenkool wrote: > On Wed, Aug 19, 2015 at 03:18:46PM -0700, H.J. Lu wrote: >> @deftypefn {Built-in Function} {void *} __builtin_argument_pointer (void) >> This function is similar to @code{__builtin_frame_address} with an >> argument of 0, but it returns

Re: [c++-delayed-folding] fold_simple

2015-09-01 Thread Jason Merrill
On 08/31/2015 03:43 PM, Kai Tietz wrote: 2015-08-31 21:29 GMT+02:00 Jason Merrill : On 08/31/2015 03:08 PM, Kai Tietz wrote: I will need to verify that this patch doesn't introduce regressions. The wacky thing here is the encapsulation of overflowed-arguments in maybe_constant_value function b

[PATCH][wwwdocs][AArch64] Add entry for target attributes and pragmas

2015-09-01 Thread Kyrill Tkachov
Hi all, This wwwdocs patch adds an entry to the GCC 6 changes page about the aarch64 target attributes and pragmas support. Ok for the wwwdocs repo? Thanks, Kyrill Index: htdocs/gcc-6/changes.html === RCS file: /cvs/gcc/wwwdocs/htd

Re: [RFC] Try vector as a new representation for vector masks

2015-09-01 Thread Richard Biener
On Tue, Sep 1, 2015 at 3:08 PM, Ilya Enkovich wrote: > On 27 Aug 09:55, Richard Biener wrote: >> On Wed, Aug 26, 2015 at 5:51 PM, Ilya Enkovich >> wrote: >> > >> > Yes, I want to try it. But getting rid of bool patterns would mean >> > support for all targets currently supporting vec_cond. Would

Re: [Patch, libstdc++] Fix data races in basic_string implementation

2015-09-01 Thread Jonathan Wakely
On 01/09/15 14:51 +0200, Dmitry Vyukov wrote: Hello, The refcounted basic_string implementation contains several data races on _M_refcount: 1. _M_is_leaked loads _M_refcount concurrently with mutations of _M_refcount. This loads needs to be memory_order_relaxed load, as _M_is_leaked predicate do

Re: [PATCH] add initial support for J2 core to sh target

2015-09-01 Thread Rich Felker
On Tue, Sep 01, 2015 at 10:45:10PM +0900, Oleg Endo wrote: > Hi Rich, > > On 01 Sep 2015, at 02:49, Rich Felker wrote: > > > The J2 Core is an open hardware cpu implementing the SH-2 instruction > > set, with the addition of barrel shift instructions and an atomic > > compare-and-swap instructio

Re: [testsuite] Clean up effective_target cache

2015-09-01 Thread Christophe Lyon
On 25 August 2015 at 17:31, Mike Stump wrote: > On Aug 25, 2015, at 1:14 AM, Christophe Lyon > wrote: >> Some subsets of the tests override ALWAYS_CXXFLAGS or >> TEST_ALWAYS_FLAGS and perform effective_target support tests using >> these modified flags. > >> This patch adds a new function 'clear

Re: [committed] taking transactional credits

2015-09-01 Thread Aldy Hernandez
On 09/01/2015 07:03 AM, Richard Henderson wrote: On 09/01/2015 06:55 AM, Aldy Hernandez wrote: If you don't like it, sue me. =) Share the blame... ;-) That's right! I ain't going down in posterity alone on this.

Re: [RS6000] Weak functions may not be file local

2015-09-01 Thread David Edelsohn
On Tue, Sep 1, 2015 at 3:51 AM, Alan Modra wrote: > A weak symbol defined in the current object file may not turn out to > be the definition used at link time, if other copies of the symbol > exist. This means they can't be considered file local; Calls to > weak functions need a nop so that the

Re: [committed] taking transactional credits

2015-09-01 Thread Richard Henderson
On 09/01/2015 06:55 AM, Aldy Hernandez wrote: If you don't like it, sue me. =) Share the blame... ;-) r~

Re: [PATCH][GCC] Algorithmic optimization in match and simplify

2015-09-01 Thread Richard Biener
On Tue, Sep 1, 2015 at 3:40 PM, Andre Vieira wrote: > Hi Marc, > > On 28/08/15 19:07, Marc Glisse wrote: >> >> (not a review, I haven't even read the whole patch) >> >> On Fri, 28 Aug 2015, Andre Vieira wrote: >> >>> 2015-08-03 Andre Vieira >>> >>> * match.pd: Added new patterns: >>> ((X

[committed] taking transactional credits

2015-09-01 Thread Aldy Hernandez
I've had too many sleepless nights caused by this code, not to warrant tattooing my name on it. If you don't like it, sue me. =) Aldy commit 7d92b913093c35cb6fb322d4dc9acb237cecabfe Author: Aldy Hernandez Date: Tue Sep 1 06:51:45 2015 -0700 + * trans-mem.c: Add contributed-by.

Re: [testsuite] Don't xfail gcc.dg/vect/no-scevccp-outer-11.c

2015-09-01 Thread Rainer Orth
Richard Biener writes: > On Tue, 1 Sep 2015, Rainer Orth wrote: > >> It seems that since 20150717, gcc.dg/vect/no-scevccp-outer-11.c XPASSes >> everywhere: >> >> XPASS: gcc.dg/vect/no-scevccp-outer-11.c scan-tree-dump-times vect "OUTER >> LOOP VECTORIZED." 1 >> >> To reduce testsuite noise, I'd

Re: [PATCH] add initial support for J2 core to sh target

2015-09-01 Thread Oleg Endo
Hi Rich, On 01 Sep 2015, at 02:49, Rich Felker wrote: > The J2 Core is an open hardware cpu implementing the SH-2 instruction > set, with the addition of barrel shift instructions and an atomic > compare-and-swap instruction. This patch adds a cpu model option -mj2 > to the sh target. Presently

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

2015-09-01 Thread Chen Gang
On 8/31/15 19:12, Richard Biener wrote: > On Sat, Aug 29, 2015 at 2:57 PM, Chen Gang > wrote: >> >> It is about bug63510: current input_location isn't precise for reporting >> warning. The correct location is gimple location of current statement. > > Looks ok to me. Ok if bootstrapped and tested.

Re: [c++-delayed-folding] fold_simple

2015-09-01 Thread Kai Tietz
2015-09-01 13:17 GMT+02:00 Kai Tietz : > 2015-09-01 10:43 GMT+02:00 Kai Tietz : >> 2015-09-01 10:15 GMT+02:00 Kai Tietz : >>> 2015-08-31 22:19 GMT+02:00 Kai Tietz : 2015-08-31 21:43 GMT+02:00 Kai Tietz : > 2015-08-31 21:29 GMT+02:00 Jason Merrill : >> On 08/31/2015 03:08 PM, Kai Tietz

Re: [PATCH][GCC] Algorithmic optimization in match and simplify

2015-09-01 Thread Andre Vieira
Hi Marc, On 28/08/15 19:07, Marc Glisse wrote: (not a review, I haven't even read the whole patch) On Fri, 28 Aug 2015, Andre Vieira wrote: 2015-08-03 Andre Vieira * match.pd: Added new patterns: ((X {&,<<,>>} C0) {|,^} C1) {^,|} C2) (X {|,^,&} C0) {<<,>>} C1 -> (X {<<,>>} C1) {

Re: [testsuite] Don't xfail gcc.dg/vect/no-scevccp-outer-11.c

2015-09-01 Thread Richard Biener
On Tue, 1 Sep 2015, Rainer Orth wrote: > It seems that since 20150717, gcc.dg/vect/no-scevccp-outer-11.c XPASSes > everywhere: > > XPASS: gcc.dg/vect/no-scevccp-outer-11.c scan-tree-dump-times vect "OUTER > LOOP VECTORIZED." 1 > > To reduce testsuite noise, I'd like to remove the xfail as follo

[RFC] Try vector as a new representation for vector masks

2015-09-01 Thread Ilya Enkovich
On 27 Aug 09:55, Richard Biener wrote: > On Wed, Aug 26, 2015 at 5:51 PM, Ilya Enkovich wrote: > > > > Yes, I want to try it. But getting rid of bool patterns would mean > > support for all targets currently supporting vec_cond. Would it be OK > > to have vector mask co-exist with bool patterns fo

Re: [gomp4] expunge shared_size from launch API

2015-09-01 Thread Nathan Sidwell
On 09/01/15 06:15, Tom de Vries wrote: On 31/08/15 19:39, Nathan Sidwell wrote: * builtin-types.def (DEF_FUNCTION_TYPE_VAR_6): Define. Committed attached follow-up patch to fix the ada build. thanks

Re: [gomp4] New reduction infrastructure for OpenACC

2015-09-01 Thread Tom de Vries
On 19/08/15 21:42, Cesar Philippidis wrote: +DEFHOOK +(reduction, +"This hook is used by the oacc_transform pass to expand calls to the\n\ +internal functions @var{GOACC_REDUCTION_SETUP},\n\ +@var{GOACC_REDUCTION_INIT},\n\ @var{GOACC_REDUCTION_FINI} and\n\ @var{GOACC_REDUCTION_TEARDOWN} into a s

Re: [testsuite] Don't xfail gcc.dg/vect/no-scevccp-outer-11.c

2015-09-01 Thread Alan Lawrence
Rainer Orth wrote: It seems that since 20150717, gcc.dg/vect/no-scevccp-outer-11.c XPASSes everywhere: XPASS: gcc.dg/vect/no-scevccp-outer-11.c scan-tree-dump-times vect "OUTER LOOP VECTORIZED." 1 To reduce testsuite noise, I'd like to remove the xfail as follows. Tested with the appropriate r

[Patch, libstdc++] Fix data races in basic_string implementation

2015-09-01 Thread Dmitry Vyukov
Hello, The refcounted basic_string implementation contains several data races on _M_refcount: 1. _M_is_leaked loads _M_refcount concurrently with mutations of _M_refcount. This loads needs to be memory_order_relaxed load, as _M_is_leaked predicate does not change under the feet. 2. _M_is_shared lo

RE: [PATCH][4/N] Introduce new inline functions for GET_MODE_UNIT_SIZE and GET_MODE_UNIT_PRECISION

2015-09-01 Thread David Sherwood
Hi, Fix/patch committed. Tested: x86_64-linux: bootstrap built fine, no regressions aarch64-none-elf: no regressions sh-elf crossbuild: builds fine ChangeLog: 2015-09-01 David Sherwood gcc/ * genmodes.c: Add CONST_MODE_UNIT_SIZE modifier. Thanks, David. > -Original Messag

[testsuite] Don't xfail gcc.dg/vect/no-scevccp-outer-11.c

2015-09-01 Thread Rainer Orth
It seems that since 20150717, gcc.dg/vect/no-scevccp-outer-11.c XPASSes everywhere: XPASS: gcc.dg/vect/no-scevccp-outer-11.c scan-tree-dump-times vect "OUTER LOOP VECTORIZED." 1 To reduce testsuite noise, I'd like to remove the xfail as follows. Tested with the appropriate runtest invocations on

Re: Fix 61441

2015-09-01 Thread Richard Biener
On Tue, Sep 1, 2015 at 12:23 PM, Sujoy Saraswati wrote: > The following patch fixes 61441. It converts sNaN to qNaN on folding > when -fno-signaling-nans is used. > > Bootstrap and regression tests on x86_64-linux-gnu and > aarch64-unknown-linux-gnu passed with changes done on trunk. > > Is this f

Re: [PATCH] Import liboffloadmic from upstream

2015-09-01 Thread Ilya Verbin
On Tue, Sep 01, 2015 at 09:58:22 +0200, Dodji Seketeli wrote: > Woops. can you send me the exact two libraries so that I can see what's > going wrong? You can quickly file an issue to > https://sourceware.org/bugzilla/enter_bug.cgi?product=libabigail or just > send me the two binaries by email. I

[testsuite, fortran] XFAIL gfortran.dg/execute_command_line_2.f90 (PR libfortran/67412)

2015-09-01 Thread Rainer Orth
As discussed in the PR, gfortran.dg/execute_command_line_2.f90 execution FAILs on Solaris 10 due to a bug in /bin/sh's exit code handling. Since the fix is involved and the issue a corner case, we decided to XFAIL execution instead (at least until an eventual fix is forthcoming). Tested with the

RE: Fix intelmic-mkoffload.c if the temp path contains a '-'

2015-09-01 Thread Hahnfeld, Jonas
> -Original Message- > From: Jakub Jelinek [mailto:ja...@redhat.com] > Sent: Tuesday, September 01, 2015 1:50 PM > To: Hahnfeld, Jonas > Cc: gcc-patches@gcc.gnu.org; Ilya Verbin; Kirill Yukhin > Subject: Re: Fix intelmic-mkoffload.c if the temp path contains a '-' > > On Tue, Sep 01, 2015

Re: Fix intelmic-mkoffload.c if the temp path contains a '-'

2015-09-01 Thread Jakub Jelinek
On Tue, Sep 01, 2015 at 11:35:15AM +, Hahnfeld, Jonas wrote: > >From 884b6199179e7a604474bc6a828a6861d3ff4501 Mon Sep 17 00:00:00 2001 > From: Jonas Hahnfeld > Date: Thu, 20 Aug 2015 12:13:55 +0200 > Subject: [PATCH] Fix intelmic-mkoffload.c if the temp path contains a '-' > > 2015-08-20 Jon

[PATCH] [RTEMS] Update RTEMS thread model

2015-09-01 Thread Sebastian Huber
For synchronization objects use the self-contained objects available via Newlib . https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;a=commit;h=ecaef05f6601f1e8acb78fb65b411a258f39988a Enable the C++11 threads using . So, the threads are only supported in case the POSIX API is enabled in

Re: Fix intelmic-mkoffload.c if the temp path contains a '-'

2015-09-01 Thread Ilya Verbin
On Tue, Sep 01, 2015 at 11:35:15 +, Hahnfeld, Jonas wrote: > > during my test of OpenMP 4.0 offloading features I have found a bug in > > intelmic-mkoffload.c when the temp path contains a '-'. > > objcopy will in this case replace it with a '_' which wasn't reflected in > > the original code a

RE: Fix intelmic-mkoffload.c if the temp path contains a '-'

2015-09-01 Thread Hahnfeld, Jonas
Hopefully CC'ing the right people... > -Original Message- > From: Hahnfeld, Jonas > Sent: Thursday, August 20, 2015 12:25 PM > To: 'gcc-patches@gcc.gnu.org' > Subject: Fix intelmic-mkoffload.c if the temp path contains a '-' > > Hi all, > > during my test of OpenMP 4.0 offloading feature

Re: [PATCH] Updated LTO early debug patch

2015-09-01 Thread Markus Trippelsdorf
On 2015.09.01 at 13:15 +0200, Jan Hubicka wrote: > > > And it breaks quickly during Firefox build, e.g.: > > > > > > trippels@gcc2-power8 angle % cat glslang_lex.ii > > > struct A; > > > template class B { > > > typedef A value_type; > > > void m_fn1(const value_type &); > > > }; > > > struct

Re: [c++-delayed-folding] fold_simple

2015-09-01 Thread Kai Tietz
2015-09-01 10:43 GMT+02:00 Kai Tietz : > 2015-09-01 10:15 GMT+02:00 Kai Tietz : >> 2015-08-31 22:19 GMT+02:00 Kai Tietz : >>> 2015-08-31 21:43 GMT+02:00 Kai Tietz : 2015-08-31 21:29 GMT+02:00 Jason Merrill : > On 08/31/2015 03:08 PM, Kai Tietz wrote: >> >> I will need to verify tha

Re: [PATCH] Updated LTO early debug patch

2015-09-01 Thread Jan Hubicka
> > And it breaks quickly during Firefox build, e.g.: > > > > trippels@gcc2-power8 angle % cat glslang_lex.ii > > struct A; > > template class B { > > typedef A value_type; > > void m_fn1(const value_type &); > > }; > > struct A { > > int staticUse; > > B<> fields; > > }; > > > > trippels@

[PATCH] libstdc++/67408

2015-09-01 Thread Sebastian Huber
2015-09-01 Sebastian Huber PR libstdc++/67408 * include/std/mutex (__timed_mutex_impl): Use __timedlock(). (timed_mutex): Add friend class __timed_mutex_impl. Add __timedlock(). (recursive_timed_mutex): Likewise. --- libstdc++-v3/include/std/mutex | 17

Re: [PATCH][AArch64][1/3] Expand signed mod by power of 2 using CSNEG

2015-09-01 Thread Kyrill Tkachov
Hi James, On 01/09/15 10:25, James Greenhalgh wrote: On Thu, Aug 13, 2015 at 01:36:50PM +0100, Kyrill Tkachov wrote: Some comments below. Thanks, I'll incorporate them, with one clarification inline. diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c index 1394ed7..c

Commit: MSP430: Fix typo in --target-help text

2015-09-01 Thread Nick Clifton
Hi DJ, I am applying the patch below as an obvious fix for a typo in the msp430's --target-help output. Cheers Nick gcc/ChangeLog 2015-09-01 Nick Clifton * config/msp430/msp430.opt (mcpu): Fix typo. Index: gcc/config/msp430/msp430.opt ==

Re: [PATCH] Fix TERed insn locations

2015-09-01 Thread Jakub Jelinek
On Tue, Sep 01, 2015 at 12:09:48PM +0200, Richard Biener wrote: > > So I finally got around to dig into why some testcases I inspected for > early LTO debug (and some existing guality FAILs) happen. They happen > because we screw up locations of insns coming from TERed SSA defs > which end up usi

Fix 61441

2015-09-01 Thread Sujoy Saraswati
The following patch fixes 61441. It converts sNaN to qNaN on folding when -fno-signaling-nans is used. Bootstrap and regression tests on x86_64-linux-gnu and aarch64-unknown-linux-gnu passed with changes done on trunk. Is this fix fine ? Regards, Sujoy 2015-09-01 Sujoy Saraswati PR t

Re: [gomp4] expunge shared_size from launch API

2015-09-01 Thread Tom de Vries
On 31/08/15 19:39, Nathan Sidwell wrote: * builtin-types.def (DEF_FUNCTION_TYPE_VAR_6): Define. Committed attached follow-up patch to fix the ada build. Thanks, - Tom Fix gomp-4_0-branch ada build 2015-09-01 Tom de Vries * gcc-interface/utils.c (DEF_FUNCTION_TYPE_VAR_6): Define

[PATCH] Fix TERed insn locations

2015-09-01 Thread Richard Biener
So I finally got around to dig into why some testcases I inspected for early LTO debug (and some existing guality FAILs) happen. They happen because we screw up locations of insns coming from TERed SSA defs which end up using the TERed-into stmt location. So final re-building the BLOCK tree ends

[PATCH][AArch64] Use preferred aliases for CSNEG, CSINC, CSINV

2015-09-01 Thread Kyrill Tkachov
Hi all, The ARMv8-A reference manual says: "CNEG , , is equivalent to CSNEG , , , invert() and is the preferred disassembly when Rn == Rm && cond != '111x'." That is, when the two input registers are the same we can use the shorter CNEG mnemonic with the inverse condition instead of the longer

Re: [PATCH][AArch64] Fix FAIL: gcc.target/aarch64/target_attr_crypto_ice_1.c (internal compiler error)

2015-09-01 Thread James Greenhalgh
On Wed, Aug 19, 2015 at 02:41:35PM +0100, Kyrill Tkachov wrote: > Hi all, > > This fixes the ICE exposed by Alexandre's patch > (https://gcc.gnu.org/ml/gcc-patches/2015-08/msg00873.html) > The solution I came up with is to re-layout the parameter decls not during > expansion time (when RTL has a

Re: [PATCH][AArch64][1/3] Expand signed mod by power of 2 using CSNEG

2015-09-01 Thread James Greenhalgh
On Thu, Aug 13, 2015 at 01:36:50PM +0100, Kyrill Tkachov wrote: Some comments below. > diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c > index 1394ed7..c8bd8d2 100644 > --- a/gcc/config/aarch64/aarch64.c > +++ b/gcc/config/aarch64/aarch64.c > @@ -6652,6 +6652,25 @@ cost_p

Re: [Patch] PR67351 Implement << N & >> N optimizers

2015-09-01 Thread Richard Biener
On Tue, Sep 1, 2015 at 10:57 AM, Hurugalawadi, Naveen wrote: > Hi, > > Please find attached the patch "pr67351.patch" that implements the > pattern << N & >> N optimizers. + (bit_and @0 (lshift { build_minus_one_cst (type); } @1 please use (bit_and @0 { wide_int_to_tree (type, wi::lshif

  1   2   >