Re: [committed] Update config/pa/linux-atomic.c to use new subword and double word atomic support

2014-11-21 Thread Richard Henderson
On 11/22/2014 01:25 AM, John David Anglin wrote: > #define ABORT_INSTRUCTION asm ("iitlbp %r0,(%sr0, %r0)") ... > +static inline long > +__kernel_cmpxchg2 (void * oldval, void * newval, void *mem, int val_size) > +{ > + register unsigned long lws_mem asm("r26") = (unsigned long) (mem); > + regis

[Go] Use the static chain as closure parameter from Go

2014-11-21 Thread Richard Henderson
This is the non-gofrontend patch corresponding to https://codereview.appspot.com/180890043/ Ian, I thought I'd sent it out at the same time as the codereview, but I guess it never went. r~ commit 5a78f18cf8cd2160c62472693222cc72bd4379b6 Author: Richard Henderson Date: Tue Oct 7 12:28:55 20

Re: [PATCHv4][MIPS] Implement O32 ABI extensions (GCC)

2014-11-21 Thread Andrew Pinski
On Wed, Nov 12, 2014 at 2:56 PM, Matthew Fortune wrote: >> > Moore, Catherine writes: >> > > The patch looks good. Please fix up these couple of nits prior to >> > > committing. >> > >> > OK, thanks for the second read through. One further amendment below, >> > I'll aim to commit later today. >

C++ PATCH for c++/63657 (missed unused reference warning)

2014-11-21 Thread Jason Merrill
The earlier fix for 38958 was too broad; we don't want to suppress the unused warning for all references to type with non-trivial destructor, just references bound to a temporary. Tested x86_64-pc-linux-gnu, applying to trunk. commit 725c1c94f7e14cfc5314a86c3504cb33ec1cb74b Author: Jason Merril

[PING][PATCH] Change contrib/test_installed for testing cross compilers

2014-11-21 Thread Steve Ellcey
I noticed I never got a reply to this patch proposal I sent out back in March. Does anyone have an opinion on this? I think it would make testing of installed cross compilers easier. Steve Ellcey sell...@imgtec.com --- Original Email --- I was doing some testing of an

[committed] Update config/pa/linux-atomic.c to use new subword and double word atomic support

2014-11-21 Thread John David Anglin
The attached change updates the linux atomic support to use the new kernel calls for subword and double word types. Tested on hppa-unknown-linux-gnu and committed to trunk. The majority of this change, including the kernel patch, was contributed by Guy Martin. Dave -- John David Anglin

Re: [RFC] First steps towards segregating types.

2014-11-21 Thread Joseph Myers
On Fri, 21 Nov 2014, Andrew MacLeod wrote: > The biggest issue is what to do with fields which can be either a type or a > tree... ie TREE_VALUE() of a TREE_LIST can be a type, as can a TREE_VEC > element or a DECL_CONTEXT. I think the DECL_INITIAL field is overloaded and > can sometimes be a

Re: [PATCH, MPX wrappers 1/3] Add MPX wrappers library

2014-11-21 Thread Joseph Myers
On Fri, 21 Nov 2014, Ilya Enkovich wrote: > * c-family/c.opt (static-libmpxwrappers): New. New options need documenting in invoke.texi. This includes driver options. > diff --git a/libmpx/mpxwrap/mpx_wrappers.c b/libmpx/mpxwrap/mpx_wrappers.c > new file mode 100644 > index 000..bcff8

Re: [PATCH, MPX runtime 1/2] Integrate MPX runtime library

2014-11-21 Thread Joseph Myers
On Wed, 19 Nov 2014, Ilya Enkovich wrote: > > If it's intended only as the latter - and this is documented - then you > > don't have the libgcc-like requirements, and there's no point in having > > multiple libraries. If it's intended for both, that points the way to > > separate libraries (wh

Re: [PATCH, MPX runtime 1/2] Integrate MPX runtime library

2014-11-21 Thread Joseph Myers
On Fri, 21 Nov 2014, Ilya Enkovich wrote: > +# Disable libmpx on unsupported systems. > +if test -d ${srcdir}/libmpx; then > +if test x$enable_libmpx = x; then > + AC_MSG_CHECKING([for libmpx support]) > + if (srcdir=${srcdir}/libmpx; \ > + . ${srcdir}/configure.tgt;

Re: [RFC] First steps towards segregating types.

2014-11-21 Thread David Malcolm
On Fri, 2014-11-21 at 21:13 +0100, Richard Biener wrote: > On November 21, 2014 8:45:09 PM CET, Diego Novillo > wrote: > >On Fri, Nov 21, 2014 at 1:48 PM, Andrew MacLeod > >wrote: > > > >> 1 - introduce a TYPE_REF tree node, which is effectively just a > >'typed' tree > >> node, and the TREE_TYPE

Re: [RFC] First steps towards segregating types.

2014-11-21 Thread Andrew MacLeod
On 11/21/2014 05:39 PM, Jeff Law wrote: On 11/21/14 11:48, Andrew MacLeod wrote: There are a few issues, of course :-) The biggest issue is what to do with fields which can be either a type or a tree... ie TREE_VALUE() of a TREE_LIST can be a type, as can a TREE_VEC element or a DECL_CONT

Re: [wwwdocs] Document ARM --with-cpu changes for 5.0

2014-11-21 Thread Jeff Law
On 11/21/14 06:30, James Greenhalgh wrote: Hi, As requested by Ramana when he OKed the initial change, the attched patch documents the changes I made to --with-cpu and --with-tune in this patch: https://gcc.gnu.org/ml/gcc-patches/2014-05/msg02618.html in the changes for GCC 5.0. OK? OK. jef

Re: [RFC] First steps towards segregating types.

2014-11-21 Thread Jeff Law
On 11/21/14 11:48, Andrew MacLeod wrote: There are a few issues, of course :-) The biggest issue is what to do with fields which can be either a type or a tree... ie TREE_VALUE() of a TREE_LIST can be a type, as can a TREE_VEC element or a DECL_CONTEXT. I think the DECL_INITIAL field is o

Re: [Patch,MIPS] Add default SYS_futex definition in libgomp

2014-11-21 Thread Jakub Jelinek
On Fri, Nov 21, 2014 at 02:30:06PM -0800, Steve Ellcey wrote: > 2014-11-21 Steve Ellcey > > * config/linux/mips/futex.h (SYS_futex): Define if not already done. Ok. > --- a/libgomp/config/linux/mips/futex.h > +++ b/libgomp/config/linux/mips/futex.h > @@ -25,6 +25,11 @@ > /* Provide ta

[Patch,MIPS] Add default SYS_futex definition in libgomp

2014-11-21 Thread Steve Ellcey
Here is another patch for the MIPS android build. It is in the libgomp subdirectory and it has been in the Android tree for a while but I would like to check it in to the official GCC tree so we do not need to keep porting it to new versions of GCC. This patch defines SYS_futex if it is not alrea

Re: [PATCH v2] gcc/ubsan.c: Use 'pretty_print' for 'pretty_name' to avoid memory overflow

2014-11-21 Thread Chen Gang
Thank you very much for your work. I shall send patch v3 for it. Thanks. On 11/22/2014 05:57 AM, Jakub Jelinek wrote: > On Sat, Nov 22, 2014 at 05:03:37AM +0800, Chen Gang wrote: >> According to the next code, 'pretty_name' may need additional bytes more >> than 16 (may have unlimited length for

Re: [PATCH v2] gcc/ubsan.c: Use 'pretty_print' for 'pretty_name' to avoid memory overflow

2014-11-21 Thread Jakub Jelinek
On Sat, Nov 22, 2014 at 05:03:37AM +0800, Chen Gang wrote: > According to the next code, 'pretty_name' may need additional bytes more > than 16 (may have unlimited length for array type). There is an easy way > to fix it: use 'pretty_print' for 'pretty_name'. > > Let the code meet 2 white spaces a

C++ PATCH for c++/63588 (ICE with empty template args to variable template)

2014-11-21 Thread Jason Merrill
uses_template_parms ought to be able to deal with NULL_TREE. Tested x86_64-pc-linux-gnu, applying to trunk. commit 3a85a7bd7ea664fb594843433c51f3b6cbb320d3 Author: Jason Merrill Date: Fri Nov 21 13:46:34 2014 -0500 PR c++/63588 * pt.c (uses_template_parms): Handle null argument. di

C++ PATCH for c++/63942 (mangling compat alias problems)

2014-11-21 Thread Jason Merrill
In this testcase, the wrong old mangling for one constructor matches the correct mangling for the other constructor, and the testcase was failing to compile as a result. But compatibility aliases should not be treated as conflicting with real declarations; they should be discarded. Tested x86

C++ PATCH for c++/63849 (mangling crash with alias template)

2014-11-21 Thread Jason Merrill
An alias or typedef to a template type parameter is not itself a template type parameter. Tested x86_64-pc-linux-gnu, applying to trunk and 4.9. commit 89ffb512bb8120b0e4bd919a941c44876a6a8ce3 Author: Jason Merrill Date: Fri Nov 21 13:55:11 2014 -0500 PR c++/63849 * mangle.c (decl_

patch to fix PR63897

2014-11-21 Thread Vladimir Makarov
The following patch fixes PR63897. The details can be found on https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63897 The patch was successfully bootstrapped and tested on x86 and x86-64. Committed as rev. 217947. 2014-11-21 Vladimir Makarov PR target/63897 * lra-lives.c (mark

RE: [PATCH,MIPS] Refine configure guard for .module availability

2014-11-21 Thread Matthew Fortune
Matthew Fortune writes: > (I'm not sure if I need approval from someone else for MIPS specific > top level 'configure' changes. I'm cautiously assuming I do for now.) FWIW, MIPS maintainership covers all MIPS-specific code and documentation (except for externally-maintained code) so you don't

Re: [PATCH][OpenMP] Fix named critical sections inside target functions

2014-11-21 Thread Ilya Verbin
> On 22 Nov 2014, at 00:11, H.J. Lu wrote: > > Have you fixed the offloading issue with binutils 2.25? No, I'm still thinking how to make a patch better than the former... Probably will send it on Monday. (Regressions in make check disappeared after disabling offload IR in default configura

Re: [PATCH][OpenMP] Fix named critical sections inside target functions

2014-11-21 Thread Jakub Jelinek
On Sat, Nov 22, 2014 at 12:08:38AM +0300, Ilya Verbin wrote: > > On 21 Nov 2014, at 23:36, Jakub Jelinek wrote: > > > >> On Fri, Nov 21, 2014 at 11:19:26PM +0300, Ilya Verbin wrote: > >> Hi, > >> > >> '#pragma omp critical (name)' can be placed in the function, marked > >> with '#pragma omp decl

Re: [PATCH][OpenMP] Fix named critical sections inside target functions

2014-11-21 Thread H.J. Lu
On Fri, Nov 21, 2014 at 1:08 PM, Ilya Verbin wrote: >> On 21 Nov 2014, at 23:36, Jakub Jelinek wrote: >> >>> On Fri, Nov 21, 2014 at 11:19:26PM +0300, Ilya Verbin wrote: >>> Hi, >>> >>> '#pragma omp critical (name)' can be placed in the function, marked >>> with '#pragma omp declare target', in t

Re: [PATCH][OpenMP] Fix named critical sections inside target functions

2014-11-21 Thread Ilya Verbin
> On 21 Nov 2014, at 23:36, Jakub Jelinek wrote: > >> On Fri, Nov 21, 2014 at 11:19:26PM +0300, Ilya Verbin wrote: >> Hi, >> >> '#pragma omp critical (name)' can be placed in the function, marked >> with '#pragma omp declare target', in this case the corresponding node >> should be marked as off

[PATCH v2] gcc/ubsan.c: Use 'pretty_print' for 'pretty_name' to avoid memory overflow

2014-11-21 Thread Chen Gang
According to the next code, 'pretty_name' may need additional bytes more than 16 (may have unlimited length for array type). There is an easy way to fix it: use 'pretty_print' for 'pretty_name'. Let the code meet 2 white spaces alignment coding styles (originally, some of code is 1 white sapce ali

Re: [RFC] First steps towards segregating types.

2014-11-21 Thread Richard Biener
On November 21, 2014 9:22:08 PM CET, Andrew MacLeod wrote: >On 11/21/2014 03:13 PM, Richard Biener wrote: >> On November 21, 2014 8:45:09 PM CET, Diego Novillo > wrote: >>> On Fri, Nov 21, 2014 at 1:48 PM, Andrew MacLeod > >>> wrote: >>> 1 - introduce a TYPE_REF tree node, which is effectivel

Re: [PATCH 2/2] PR debug/38757 continued. Handle C11, C++11 and C++14.

2014-11-21 Thread Mark Wielaard
On Fri, Nov 21, 2014 at 09:28:45AM +0100, Jakub Jelinek wrote: > I think best would be to tweak > if (value < 2 || value > 4) > error_at (loc, "dwarf version %d is not supported", value); > else > opts->x_dwarf_version = value; > so that we accept value 5 too, and for no

Re: [PATCH][OpenMP] Fix named critical sections inside target functions

2014-11-21 Thread Jakub Jelinek
On Fri, Nov 21, 2014 at 11:19:26PM +0300, Ilya Verbin wrote: > Hi, > > '#pragma omp critical (name)' can be placed in the function, marked > with '#pragma omp declare target', in this case the corresponding node > should be marked as offloadable too. > Bootstrapped/regtested on x86_64-linux and i6

Re: [PATCH] Fix up __builtin_*_overflow expansion on some targets (PR target/63848)

2014-11-21 Thread Richard Biener
On November 21, 2014 8:08:37 PM CET, Jakub Jelinek wrote: >Hi! > >Apparently, emit_cmp_and_jump_insns can silently generate wrong code >for wider modes on some targets, so this patch changes all those calls >in >internal-fn.c to do_compare_rtx_and_jump, which is a wrapper around >emit_cmp_and_jump

Re: [RFC] First steps towards segregating types.

2014-11-21 Thread Andrew MacLeod
On 11/21/2014 03:13 PM, Richard Biener wrote: On November 21, 2014 8:45:09 PM CET, Diego Novillo wrote: On Fri, Nov 21, 2014 at 1:48 PM, Andrew MacLeod wrote: 1 - introduce a TYPE_REF tree node, which is effectively just a 'typed' tree node, and the TREE_TYPE() field of a TYPE_REF node wou

Re: [PATCH] Fix VRP handling of {ADD,SUB,MUL}_OVERFLOW (PR tree-optimization/64006)

2014-11-21 Thread Richard Biener
On November 21, 2014 8:04:39 PM CET, Jakub Jelinek wrote: >Hi! > >As discussed on IRC and in the PR, these internal calls are quite >unique for VRP in that they return _Complex integer result, >which VRP doesn't track, but then extract using >REALPART_EXPR/IMAGPART_EXPR >the two results from that

[PATCH][OpenMP] Fix named critical sections inside target functions

2014-11-21 Thread Ilya Verbin
Hi, '#pragma omp critical (name)' can be placed in the function, marked with '#pragma omp declare target', in this case the corresponding node should be marked as offloadable too. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? -- Ilya gcc/ * omp-low.c (lower_omp_

Re: [PATCH, PR 63551] Use proper type in evaluate_conditions_for_known_args

2014-11-21 Thread Richard Biener
On November 21, 2014 9:07:50 PM CET, Martin Jambor wrote: >Hi, > >the testcase of PR 63551 passes a union between a signed and an >unsigned integer between two functions as a parameter. The caller >initializes to an unsigned integer with the highest order bit set, the >callee loads the data throu

Re: [PATCH, PR 63551] Use proper type in evaluate_conditions_for_known_args

2014-11-21 Thread Martin Jambor
On Fri, Nov 21, 2014 at 09:07:50PM +0100, Martin Jambor wrote: > Hi, > > the testcase of PR 63551 passes a union between a signed and an > unsigned integer between two functions as a parameter. The caller > initializes to an unsigned integer with the highest order bit set, the > callee loads the

Re: [RFC] First steps towards segregating types.

2014-11-21 Thread Andrew MacLeod
On 11/21/2014 02:45 PM, Diego Novillo wrote: On Fri, Nov 21, 2014 at 1:48 PM, Andrew MacLeod wrote: 1 - introduce a TYPE_REF tree node, which is effectively just a 'typed' tree node, and the TREE_TYPE() field of a TYPE_REF node would point to the type node. Any routines which utilize a TYPE n

Re: [RFC] First steps towards segregating types.

2014-11-21 Thread Richard Biener
On November 21, 2014 8:45:09 PM CET, Diego Novillo wrote: >On Fri, Nov 21, 2014 at 1:48 PM, Andrew MacLeod >wrote: > >> 1 - introduce a TYPE_REF tree node, which is effectively just a >'typed' tree >> node, and the TREE_TYPE() field of a TYPE_REF node would point to the >type >> node. Any routin

[PATCH, PR 63551] Use proper type in evaluate_conditions_for_known_args

2014-11-21 Thread Martin Jambor
Hi, the testcase of PR 63551 passes a union between a signed and an unsigned integer between two functions as a parameter. The caller initializes to an unsigned integer with the highest order bit set, the callee loads the data through the signed field and compares with zero. evaluate_conditions_f

Re: [RFC] First steps towards segregating types.

2014-11-21 Thread Diego Novillo
On Fri, Nov 21, 2014 at 1:48 PM, Andrew MacLeod wrote: > 1 - introduce a TYPE_REF tree node, which is effectively just a 'typed' tree > node, and the TREE_TYPE() field of a TYPE_REF node would point to the type > node. Any routines which utilize a TYPE node in a tree list would have to > be modi

Re: [PATCH] PR lto/63968: 175.vpr from cpu2000 fails to build with LTO

2014-11-21 Thread Jan Hubicka
> >Can you verify that the implementation is correct? I tend to remember that I > >introduced the > >lazy incerementation to inliner both for perofrmance and correctness > >reasons. I used to get > >odd orders when keys was increased. > > > >Honza > > Hello. > > What kind of correctness do you

Re: [PATCH 4/4] OpenMP 4.0 offloading to Intel MIC: non-fallback testing

2014-11-21 Thread Jakub Jelinek
On Fri, Nov 21, 2014 at 10:14:21PM +0300, Ilya Verbin wrote: > > On Fri, 21 Nov 2014 21:44:40, Ilya Verbin wrote: > > > You're right. This patch was rebased so many times, that we may forget to > > > regenerate it before committing. > > Build with liboffloadmic passed. OK for trunk? > > -- Ily

[PATCH 2/2, PR 63814] Do not re-create expanded artificial thunks

2014-11-21 Thread Martin Jambor
Hi, when debugging PR 63814 I noticed that when cgraph_node::create_clone was using redirect_edge_duplicating_thunks to redirect two edges to a thunk of a clone, two thunks were created, one for each edge. The reason is that even though duplicate_thunk_for_node attempts to locate an already creat

Re: [PATCH 4/4] OpenMP 4.0 offloading to Intel MIC: non-fallback testing

2014-11-21 Thread Ilya Verbin
Hi Jakub! > On Fri, 21 Nov 2014 21:44:40, Ilya Verbin wrote: > > You're right. This patch was rebased so many times, that we may forget to > > regenerate it before committing. Build with liboffloadmic passed. OK for trunk? -- Ilya * Makefile.in: Regenerate. diff --git a/Makefile.i

[PATCH] Fix up __builtin_*_overflow expansion on some targets (PR target/63848)

2014-11-21 Thread Jakub Jelinek
Hi! Apparently, emit_cmp_and_jump_insns can silently generate wrong code for wider modes on some targets, so this patch changes all those calls in internal-fn.c to do_compare_rtx_and_jump, which is a wrapper around emit_cmp_and_jump_insns that should handle the wider mode comparison expansion. Un

[PATCH] Fix VRP handling of {ADD,SUB,MUL}_OVERFLOW (PR tree-optimization/64006)

2014-11-21 Thread Jakub Jelinek
Hi! As discussed on IRC and in the PR, these internal calls are quite unique for VRP in that they return _Complex integer result, which VRP doesn't track, but then extract using REALPART_EXPR/IMAGPART_EXPR the two results from that _Complex int and to generate good code it is desirable to get prop

RE: [PATCH] MIPS16/GCC: Optimise `__call_stub_fp_' call/return stubs

2014-11-21 Thread Moore, Catherine
> -Original Message- > From: Rozycki, Maciej > Sent: Wednesday, November 19, 2014 8:05 AM > To: gcc-patches@gcc.gnu.org > Cc: Moore, Catherine; Eric Christopher; Matthew Fortune > Subject: [PATCH] MIPS16/GCC: Optimise `__call_stub_fp_' call/return stubs > > > 2014-11-19 Maciej W. Rozyc

Re: [PATCH, i386] Add new arg values for __builtin_cpu_supports

2014-11-21 Thread Ilya Enkovich
2014-11-21 20:45 GMT+03:00 Jeff Law : > On 11/20/14 09:40, Jakub Jelinek wrote: >> >> On Thu, Nov 20, 2014 at 07:36:03PM +0300, Ilya Enkovich wrote: >>> >>> Hi, >>> >>> MPX runtime checks some feature bits in order to check MPX is fully >>> supported. Runtime does it by cpuid calls but there is a

[PATCH 1/2, PR 63814] Strengthen cgraph_edge_brings_value_p

2014-11-21 Thread Martin Jambor
Hi, PR 63814 is caused by cgraph_edge_brings_value_p misidentifying an edge to an expanded artificial thunk as an edge to the original node, which then leads to crazy double-cloning and doubling the thunks along the call. This patch fixes the bug by strengthening the predicate so that it knows wh

RE: [PATCH 4/4] OpenMP 4.0 offloading to Intel MIC: non-fallback testing

2014-11-21 Thread Bernd Edlinger
Hi Ilya, On Fri, 21 Nov 2014 21:44:40, Ilya Verbin wrote: > > Hi, > > On 21 Nov 19:19, Bernd Edlinger wrote: >> so, did you really regenerate Makefile.in in that patch, or am I missing >> something ? > > You're right. This patch was rebased so many times, that we may forget to > regenerate it bef

[RFC] First steps towards segregating types.

2014-11-21 Thread Andrew MacLeod
I've been trying to sort out how to proceed with the gimple_type work, and the first step always come back to figuring out all the places types are used. This has turned out to be non-trivial and is difficult to do in an iterative way. I believe I've found a reasonable way to proceed. Over th

Re: [PATCH 4/4] OpenMP 4.0 offloading to Intel MIC: non-fallback testing

2014-11-21 Thread Ilya Verbin
Hi, On 21 Nov 19:19, Bernd Edlinger wrote: > so, did you really regenerate Makefile.in in that patch, or am I missing > something ? You're right. This patch was rebased so many times, that we may forget to regenerate it before committing. Do you plan to submit any patch for Makefile.in? Or sho

[PATCH 2/2] Pipeline model for APM XGene-1.

2014-11-21 Thread Philipp Tomsich
--- gcc/ChangeLog | 6 + gcc/config/aarch64/aarch64.md | 3 +- gcc/config/arm/xgene1.md | 532 ++ 3 files changed, 540 insertions(+), 1 deletion(-) create mode 100644 gcc/config/arm/xgene1.md diff --git a/gcc/ChangeLog b/gcc/Change

[PATCH 1/2] Core definition for APM XGene-1 and associated cost-table.

2014-11-21 Thread Philipp Tomsich
To keep this change separately buildable from the pipeline model, this patch directs the APM XGene-1 to use the generic scheduling model. --- gcc/ChangeLog| 8 +++ gcc/config/aarch64/aarch64-cores.def | 1 + gcc/config/aarch64/aarch64-tune.md | 2 +- gcc/config/aarc

[PATCH 0/2, AArch64, v3] APM X-Gene 1 cost-table and pipeline model

2014-11-21 Thread Philipp Tomsich
The following patch-series adds optimized support for the APM X-Gene 1 by providing a cost-model and pipeline-model. The pipeline-model has a few long reservation-chains, but looking at the stats for the generated NDA shows that it's well below other AArch64 cores (e.g. Cortex-A53) in overall size

Re: [PATCH] Set goarch to ppc64le where needed for gccgo testing

2014-11-21 Thread Ian Lance Taylor
On Wed, Nov 19, 2014 at 12:55 PM, Lynn A. Boger wrote: > Updated patch: Thanks. Committed. Ian > On 11/19/2014 09:01 AM, Lynn A. Boger wrote: >> >> Hi, >> >> This change goes along with the change to the GOARCH setting in gccgo for >> ppc64le which will be done in gofrontend. The description

libgo patch committed: Use ppc64le for little-endian 64-bit PowerPC architecture

2014-11-21 Thread Ian Lance Taylor
This patch by Lynn A. Boger changes libgo to use ppc64le for little-endian 64-bit PowerPC. Bootstrapped and ran testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 96de84075614 libgo/configure.ac --- a/libgo/configure.acTue Nov 18 09:28:24 2014 -0800 +++ b/libgo/con

Re: [PATCH 4/4] OpenMP 4.0 offloading to Intel MIC: non-fallback testing

2014-11-21 Thread Bernd Edlinger
Aehm Kirill, excuse me please, but if I do autogen Makefile.def I get this from svn diff Index: Makefile.in === --- Makefile.in    (revision 217890) +++ Makefile.in    (working copy) @@ -35238,9 +35238,6 @@ $(SHELL) $(srcdir)

[C++ PATCH] Allow void type as a literal type in C++14

2014-11-21 Thread Marek Polacek
I noticed that C++14 [basic.types] says that "a type is a literal type if it is: void, [...]". Yet our literal_type_p doesn't consider void type as a literal type. The following is an attempt to fix that along with a testcase. It seems that void was only added in C++14, so check for cxx14 as wel

Re: [PATCH, i386] Add new arg values for __builtin_cpu_supports

2014-11-21 Thread Jeff Law
On 11/20/14 09:40, Jakub Jelinek wrote: On Thu, Nov 20, 2014 at 07:36:03PM +0300, Ilya Enkovich wrote: Hi, MPX runtime checks some feature bits in order to check MPX is fully supported. Runtime does it by cpuid calls but there is a __builtin_cpu_supports which may be used for that. Unfortunat

Re: [PATCH][AArch64][4/5] Implement fusion of ARDP+LDR

2014-11-21 Thread Marcus Shawcroft
On 18 November 2014 10:33, Kyrill Tkachov wrote: > 2014-11-18 Kyrylo Tkachov > > * config/aarch64/aarch64.c (AARCH64_FUSE_ADRP_LDR): Define. > (cortexa53_tunings): Specify AARCH64_FUSE_ADRP_LDR in fuseable_ops. > (aarch_macro_fusion_pair_p): Handle AARCH64_FUSE_ADRP_LDR. OK /Marcu

Re: [PATCH][AArch64][3/5] Implement fusion of MOVK+MOVK

2014-11-21 Thread Marcus Shawcroft
On 18 November 2014 10:33, Kyrill Tkachov wrote: > 2014-11-18 Kyrylo Tkachov > > * config/aarch64/aarch64.c (AARCH64_FUSE_MOVK_MOVK): Define. > (cortexa53_tunings): Specify AARCH64_FUSE_MOVK_MOVK in fuseable_ops. > (cortexa57_tunings): Likewise. > (aarch_macro_fusion_pair_p): H

Re: [PATCH][AArch64][2/5] Implement adrp+add fusion

2014-11-21 Thread Marcus Shawcroft
On 18 November 2014 10:33, Kyrill Tkachov wrote: > 2014-11-18 Kyrylo Tkachov > > * config/aarch64/aarch64.c: Include tm-constrs.h > (AARCH64_FUSE_ADRP_ADD): Define. > (cortexa57_tunings): Add AARCH64_FUSE_ADRP_ADD to fuseable_ops. > (cortexa53_tunings): Likewise. > (aarch_m

Re: [PATCH][AArch64][1/5] Implement TARGET_SCHED_MACRO_FUSION_PAIR_P

2014-11-21 Thread Marcus Shawcroft
On 18 November 2014 12:20, Kyrill Tkachov wrote: > > On 18/11/14 10:33, Kyrill Tkachov wrote: >> >> diff --git a/gcc/config/arm/aarch-common-protos.h >> b/gcc/config/arm/aarch-common-protos.h >> index 264bf01..ad7ec43c 100644 >> --- a/gcc/config/arm/aarch-common-protos.h >> +++ b/gcc/config/arm/aa

Re: [PATCH][AArch64] Add vector pattern for __builtin_ctz

2014-11-21 Thread Marcus Shawcroft
On 14 November 2014 16:38, Jiong Wang wrote: > > gcc/ > * config/aarch64/iterators.md (VS): New mode iterator. > (vsi2qi): New mode attribute. > (VSI2QI): Likewise. > * config/aarch64/aarch64-simd-builtins.def: New entry for ctz. > * config/aarch64/aarch64-simd.md (ctz2): New pattern for

Re: [PATCH][AArch64]Tidy up aarch64_simd_expand_args

2014-11-21 Thread Marcus Shawcroft
On 17 November 2014 16:56, Alan Lawrence wrote: > This is a pure tidyup, no new functionality. Changes are > (1) Use op[0] to store the result operand, rather than a separate variable, > thus combining the two large switch statements into one; > (2) The 'arg' and 'mode' arrays were (almost-)only e

Re: [PATCH][AArch64]Add vec_shr pattern for 64-bit vectors using ush{l,r}; enable tests.

2014-11-21 Thread Marcus Shawcroft
On 14 November 2014 15:46, Alan Lawrence wrote: >> gcc/ChangeLog: >> >> * config/aarch64/aarch64-simd.md (vec_shr): New. >> >> gcc/testsuite/ChangeLog: >> >> * lib/target-supports.exp >> (check_effective_target_whole_vector_shift): Add aarch64{,_be}. OK /Marcus

Re: [PATCH][wwwdocs] Add Cortex-A53 erratum workaround note to AArch64 changes for 4.8

2014-11-21 Thread Marcus Shawcroft
On 17 November 2014 11:42, Kyrill Tkachov wrote: > Makes sense. Here are the changes for the 4.9 and 4.8 changes.html pages. > > Ok? This looks ok to me, I'd suggest changing... + Starting with GCC 4.8.4 a workaround for the ARM Cortex-A53 to + As of GCC 4.8.4 OK with that change. /

Re: [PATCH][AArch64] Implement vsqrt_f64 intrinsic

2014-11-21 Thread Marcus Shawcroft
On 17 November 2014 17:35, Kyrill Tkachov wrote: > 2014-11-17 Kyrylo Tkachov > > * config/aarch64/arm_neon.h (vsqrt_f64): New intrinsic. > > 2014-11-17 Kyrylo Tkachov > > * gcc.target/aarch64/simd/vsqrt_f64_1.c OK /Marcus

Re: [PATCH, MPX wrappers 1/3] Add MPX wrappers library

2014-11-21 Thread Ilya Enkovich
On 18 Nov 14:15, Jeff Law wrote: > On 11/18/14 09:48, Ilya Enkovich wrote: > >On 15 Nov 00:10, Jeff Law wrote: > >>On 11/14/14 10:26, Ilya Enkovich wrote: > >>>Hi, > >>> > >>>This patch introduces a simple library with several wrappers to be used > >>>with MPX and Pointer Bounds Checker. Wrappers

Re: [PATCH, MPX runtime 1/2] Integrate MPX runtime library

2014-11-21 Thread Ilya Enkovich
On 19 Nov 21:11, Ilya Enkovich wrote: > 2014-11-19 20:55 GMT+03:00 Jeff Law : > > On 11/19/14 07:15, Ilya Enkovich wrote: > > > >> -- > >> 2014-11-19 Ilya Enkovich > >> > >> * Makefile.def: Add libmpx. > >> * configure.ac: Add libmpx. > >> * Makefile.in: Regenerate. > >>

Re: [PATCH 8/9] Negative numbers added for sreal class.

2014-11-21 Thread Martin Liška
On 11/21/2014 04:02 PM, Richard Biener wrote: On Fri, Nov 21, 2014 at 3:39 PM, Martin Liška wrote: Hello. Ok, this is simplified, one can use sreal a = 12345 and it works ;) that's a new API, right? There is no max () and I think that using LONG_MIN here is asking for trouble (host depend

Re: [PATCH 8/9] Negative numbers added for sreal class.

2014-11-21 Thread Richard Biener
On Fri, Nov 21, 2014 at 3:39 PM, Martin Liška wrote: > Hello. > > Ok, this is simplified, one can use sreal a = 12345 and it works ;) > >> that's a new API, right? There is no max () and I think that using >> LONG_MIN here is asking for trouble (host dependence). The >> comment in the file say

Re: [PATCH 8/9] Negative numbers added for sreal class.

2014-11-21 Thread Martin Liška
On 11/21/2014 01:03 PM, Richard Biener wrote: On Fri, Nov 21, 2014 at 12:21 PM, Martin Liška wrote: On 11/14/2014 11:48 AM, Richard Biener wrote: On Thu, Nov 13, 2014 at 1:35 PM, mliska wrote: gcc/ChangeLog: 2014-11-13 Martin Liska * predict.c (propagate_freq): More elegant

Re: SRA: don't drop clobbers

2014-11-21 Thread Richard Biener
On Thu, Nov 20, 2014 at 7:11 PM, Martin Jambor wrote: > Hi, > > On Mon, Nov 03, 2014 at 10:46:49PM +0100, Marc Glisse wrote: >> On Mon, 3 Nov 2014, Marc Glisse wrote: >> >> >On Mon, 3 Nov 2014, Martin Jambor wrote: >> > >> >>I just applied your patch on top of trunk revision 217032 on my >> > >> >

[PATCH] Improve PR63679

2014-11-21 Thread Richard Biener
This patch picks up work that was in my working tree already and fixes it up. When targets choose to not emitting piecewise aggregate inits during gimplification or when that is disabled for other reasons (like being too large) then even FRE with all its tricks cannot constant fold from them. Th

Re: [rtlanal.c][BE][1/2] Fix vector load/stores to not use ld1/st1

2014-11-21 Thread Alan Hayward
On 14/11/2014 16:48, "Alan Hayward" wrote: >This is a new version of my BE patch from a few weeks ago. >This is part 1 and covers rtlanal.c. The second part will be aarch64 >specific. > >When combined with the second patch, It fixes up movoi/ci/xi for Big >Endian, so that we end up with the lab

[committed] Cherry-pick a libsanitizer bugfix (PR sanitizer/64013)

2014-11-21 Thread Jakub Jelinek
Hi! I've committed this as obvious. 2014-11-21 Jakub Jelinek PR sanitizer/64013 * sanitizer_common/sanitizer_linux.cc (FileExists): Cherry pick upstream r222532. --- libsanitizer/sanitizer_common/sanitizer_linux.cc(revision 222531) +++ libsanitizer/sanitizer_commo

[PATCH,MIPS] Refine configure guard for .module availability

2014-11-21 Thread Matthew Fortune
(I'm not sure if I need approval from someone else for MIPS specific top level 'configure' changes. I'm cautiously assuming I do for now.) Since adding o32 FFPXX support, the MIPS backend uses the .module directive to emit a .module [no]oddspreg when .module support is detected in binutils. The

[patch] Fix tilepro includes

2014-11-21 Thread Andrew MacLeod
During the flattening of optabs.h, I updated all the config/* files which were affected. I've been getting spurious failures with config-list.mk where my changes would "disappear" and tracked down why. I was blissfully unaware that the tilepro ports mul-tables.c file is actually generated fr

Re: [PATCH] VRP: don't assume strict overflow semantics when checking if a loop wraps

2014-11-21 Thread Patrick Palka
On Fri, Nov 21, 2014 at 7:18 AM, Richard Biener wrote: > On Fri, Nov 21, 2014 at 12:29 PM, Patrick Palka wrote: >> When adjusting the value range of an induction variable using SCEV, VRP >> calls scev_probably_wraps_p() with use_overflow_semantics=true. This >> parameter set to true makes scev_p

[wwwdocs] Document ARM --with-cpu changes for 5.0

2014-11-21 Thread James Greenhalgh
Hi, As requested by Ramana when he OKed the initial change, the attched patch documents the changes I made to --with-cpu and --with-tune in this patch: https://gcc.gnu.org/ml/gcc-patches/2014-05/msg02618.html in the changes for GCC 5.0. OK? Thanks, James --- ? .git ? foo.patch ? htdocs/.#ind

Re: FW: [Aarch64][BE][2/2] Fix vector load/stores to not use ld1/st1

2014-11-21 Thread Marcus Shawcroft
On 21 November 2014 12:11, Alan Hayward wrote: > 2014-11-21 Alan Hayward > > PR 57233 > PR 59810 > * config/aarch64/aarch64.c > (aarch64_classify_address): Allow extra addressing modes for BE. > (aarch64_print_operand): New operand for printing a q regis

Re: [PATCH 1/2] PR debug/38757 gcc does not emit DW_LANG_C99.

2014-11-21 Thread Jakub Jelinek
On Fri, Nov 21, 2014 at 02:01:55PM +0100, Mark Wielaard wrote: > gcc/c-family/ChangeLog > > PR debug/38757 > * c-opts.c (set_std_c89): Set lang_hooks.name. > (set_std_c99): Likewise. > (set_std_c11): Likewise. > (set_std_cxx98): Likewise. > (set_std_cxx1

Re: [PATCH 1/2] PR debug/38757 gcc does not emit DW_LANG_C99.

2014-11-21 Thread Mark Wielaard
On Fri, 2014-11-21 at 12:48 +0100, Richard Biener wrote: > On Fri, Nov 21, 2014 at 8:56 AM, Jakub Jelinek wrote: > > On Thu, Nov 20, 2014 at 11:30:11PM +0100, Mark Wielaard wrote: > >> --- a/gcc/config/avr/avr-c.c > >> +++ b/gcc/config/avr/avr-c.c > >> @@ -386,7 +386,8 @@ avr_cpu_cpp_builtins (str

[PATCH, committed] Add fgcse-sm test with scan-rtl-dump

2014-11-21 Thread Tom de Vries
Hi, this patch adds a fgcse-sm test with a scan-rtl-dump directive. The other fgcse-sm tests: ... ./gcc/testsuite/gcc.dg/pr45352-3.c ./gcc/testsuite/gcc.dg/torture/pr24257.c ./gcc/testsuite/gcc.target/i386/movsi-sm-1.c ./gcc/testsuite/g++.dg/opt/pr36185.C ... do not check whether fgcse-sm actual

[AArch64, Obvious] Fix formatting of SHLL and friends

2014-11-21 Thread James Greenhalgh
Hi, I spotted in an assembly dump, that the the SHLL, SHLL2, SADDL, and SSUBL instructions appear out of line, as they are missing a tab between their mnemonic and their operands. I've committed (revision 217917) the attached as the obvious fix to this. Tested with a build-test and a run of aarc

Re: [PATCH 1/4][x86] Add clwb,pcommit,avx512avbmi,avx512ifma.

2014-11-21 Thread Uros Bizjak
On Fri, Nov 21, 2014 at 12:21 PM, Ilya Tocar wrote: > On 20 Nov 09:43, Uros Bizjak wrote: >> On Wed, Nov 19, 2014 at 6:32 PM, Ilya Tocar wrote: >> > Hi, >> > >> > New revision of Intel ISA reference [1] has new instructions: >> > Clwb, pcommit and new flavors of AVX512. Patch bellow adds them. >>

Re: [PATCH 2/4][x86] Add clwb,pcommit,avx512avbmi,avx512ifma.

2014-11-21 Thread Uros Bizjak
On Fri, Nov 21, 2014 at 12:38 PM, Ilya Tocar wrote: > On 20 Nov 09:43, Uros Bizjak wrote: >> On Wed, Nov 19, 2014 at 6:32 PM, Ilya Tocar wrote: >> > Hi, >> > >> > New revision of Intel ISA reference [1] has new instructions: >> > Clwb, pcommit and new flavors of AVX512. Patch bellow adds them. >>

Re: [PATCH] PR ipa/63909 ICE: SIGSEGV in ipa_icf_gimple::func_checker::compare_bb()

2014-11-21 Thread Richard Biener
On Fri, Nov 21, 2014 at 12:52 PM, Martin Liška wrote: > On 11/20/2014 05:41 PM, Richard Biener wrote: >> >> On Thu, Nov 20, 2014 at 5:30 PM, Martin Liška wrote: >>> >>> Hello. >>> >>> Following patch fixes ICE in IPA ICF. Problem was that number of >>> non-debug >>> statements in a BB can >>> cha

Re: [PATCH][x86] Add clwb,pcommit,avx512avbmi,avx512ifma.

2014-11-21 Thread Uros Bizjak
On Fri, Nov 21, 2014 at 12:50 PM, Ilya Tocar wrote: > On 20 Nov 09:43, Uros Bizjak wrote: >> On Wed, Nov 19, 2014 at 6:32 PM, Ilya Tocar wrote: >> > Hi, >> > >> > New revision of Intel ISA reference [1] has new instructions: >> > Clwb, pcommit and new flavors of AVX512. Patch bellow adds them. >>

Re: libsanitizer merge from upstream r221802

2014-11-21 Thread Dmitry Vyukov
On Thu, Nov 13, 2014 at 12:16 PM, Jakub Jelinek wrote: > On Wed, Nov 12, 2014 at 05:35:48PM -0800, Konstantin Serebryany wrote: >> Here is one more merge of libsanitizer (last one was in Sept). >> >> Tested on x86_64 Ubuntu 14.04 like this: >> rm -rf */{*/,}libsanitizer && make -j 50 >> make -j 40

Re: [PATCH] VRP: don't assume strict overflow semantics when checking if a loop wraps

2014-11-21 Thread Richard Biener
On Fri, Nov 21, 2014 at 12:29 PM, Patrick Palka wrote: > When adjusting the value range of an induction variable using SCEV, VRP > calls scev_probably_wraps_p() with use_overflow_semantics=true. This > parameter set to true makes scev_probably_wraps_p() assume that signed > induction variables ne

Re: [PATCH 3/4][x86] Add clwb,pcommit,avx512avbmi,avx512ifma.

2014-11-21 Thread Uros Bizjak
On Fri, Nov 21, 2014 at 12:45 PM, Ilya Tocar wrote: > On 20 Nov 09:43, Uros Bizjak wrote: >> On Wed, Nov 19, 2014 at 6:32 PM, Ilya Tocar wrote: >> > Hi, >> > >> > New revision of Intel ISA reference [1] has new instructions: >> > Clwb, pcommit and new flavors of AVX512. Patch bellow adds them. >>

FW: [Aarch64][BE][2/2] Fix vector load/stores to not use ld1/st1

2014-11-21 Thread Alan Hayward
On 20/11/2014 18:13, "Marcus Shawcroft" wrote: >On 14 November 2014 16:48, Alan Hayward wrote: >>This is a new version of my BE patch from a few weeks ago. >>This is part 2 and covers all the aarch64 changes. >> >>When combined with the first patch, It fixes up movoi/ci/xi for Big >>Endian, so

Re: [PATCH]Add myself to MAINTAINERS

2014-11-21 Thread Markus Trippelsdorf
On 2014.11.21 at 11:42 +, Richard Earnshaw wrote: > On 21/11/14 11:16, Renlin Li wrote: > > Hi, > > > > This patch is to add myself into section of > > MAINTAINERS file. > > > > Is it Okay to commit? > > OK There is no need to ask for permission in this case: http://gcc.gnu.org/svnwrite.h

Re: [PATCH 8/9] Negative numbers added for sreal class.

2014-11-21 Thread Richard Biener
On Fri, Nov 21, 2014 at 12:21 PM, Martin Liška wrote: > On 11/14/2014 11:48 AM, Richard Biener wrote: >> >> On Thu, Nov 13, 2014 at 1:35 PM, mliska wrote: >>> >>> gcc/ChangeLog: >>> >>> 2014-11-13 Martin Liska >>> >>> * predict.c (propagate_freq): More elegant sreal API is used. >>>

Re: [PATCH] PR ipa/63909 ICE: SIGSEGV in ipa_icf_gimple::func_checker::compare_bb()

2014-11-21 Thread Martin Liška
On 11/20/2014 05:41 PM, Richard Biener wrote: On Thu, Nov 20, 2014 at 5:30 PM, Martin Liška wrote: Hello. Following patch fixes ICE in IPA ICF. Problem was that number of non-debug statements in a BB can change (for instance by IPA split), so that the number is recomputed. Huh, so can it get

Re: [ia64 PATCH] Fix up ia64 attribute handling (PR target/61137)

2014-11-21 Thread Richard Biener
On Fri, Nov 21, 2014 at 12:01 PM, Andreas Schwab wrote: > Jakub Jelinek writes: > >> The following untested patch fixes that (tested on small-addr-1.c with >> a cross-compiler), I don't have ia64 hw nor spare cycles to test this >> though, so I'm just offering the patch as is if anyone wants to t

  1   2   >