Re: [MIPS, committed] Revert some Octeon BADDU patches

2014-01-09 Thread Richard Sandiford
Eric Botcazou writes: >> This patch just reverts some changes I'd made to the BADDU patterns >> for the infamous (truncate:QI (plus:SI ...)) -> (plus:QI ...) >> simplification. That simplification was limited to CISCy targets for PR >> 58295. >> >> Tested on mips64-linux-gnu and applied. It fixe

[MIPS, committed] Backport bswap patches to 4.8

2014-01-09 Thread Richard Sandiford
I got an off-list request to backport the bswap patterns to 4.8. They've been in trunk for a while without any problems being reported and they should be relatively safe. Here's what I applied after testing on mips64-linux-gnu (with --with-arch=mips64r2). The only difference is the use of: [(s

Re: [PATCH] Ignore DECL_ALIGN of SSA_NAME underlying decls for dynamic stack realignment (PR middle-end/47735)

2014-01-09 Thread Richard Biener
Jakub Jelinek wrote: >Hi! > >As discussed in the PR, if a var isn't addressable and has gimple reg >type, >I don't see any point to honor it's DECL_ALIGN, we only refer to the >var through SSA_NAME_VAR of SSA_NAMEs, nothing is allocated on the >stack >immediately and the SSA_NAMEs are turned into

Re: [Patch] Remove references to non-existent tree-flow.h file

2014-01-09 Thread Richard Biener
Steve Ellcey wrote: >While looking at PR 59335 (plugin doesn't build) I saw the comments >about >tree-flow.h and tree-flow-inline.h not existing anymore. While these >files have been removed there are still some references to them in >Makefile.in, doc/tree-ssa.texi, and a couple of source files.

Question about gimplify.c:gimplify_adjust_omp_clauses_1, GOVD_MAP_TO_ONLY

2014-01-09 Thread Thomas Schwinge
Hi! In gimplify.c:gimplify_adjust_omp_clauses_1, does the case for GOVD_MAP_TO_ONLY have a real current use case (I couldn't spot any), or is it "just for completeness"? diff --git gcc/gimplify.c gcc/gimplify.c index 3738589..870550c 100644 --- gcc/gimplify.c +++ gcc/gimplify.c @@ -6136,9 +6136,9

Re: [PATCH][IRA] Analysis of register usage of functions for usage by IRA.

2014-01-09 Thread Tom de Vries
On 09-01-14 15:41, Richard Earnshaw wrote: On 30/03/13 16:10, Tom de Vries wrote: On 29/03/13 13:54, Tom de Vries wrote: I split the patch up into 10 patches, to facilitate further review: ... 0001-Add-command-line-option.patch 0002-Add-new-reg-note-REG_CALL_DECL.patch 0003-Add-implicit-paramet

Re: Question about gimplify.c:gimplify_adjust_omp_clauses_1, GOVD_MAP_TO_ONLY

2014-01-09 Thread Jakub Jelinek
On Thu, Jan 09, 2014 at 09:38:25PM +0100, Thomas Schwinge wrote: > In gimplify.c:gimplify_adjust_omp_clauses_1, does the case for > GOVD_MAP_TO_ONLY have a real current use case (I couldn't spot any), or > is it "just for completeness"? It is typically for any artificial vars that are known not to

Re: [PATCH][IRA] Analysis of register usage of functions for usage by IRA.

2014-01-09 Thread Andi Kleen
Tom de Vries writes: > > Is this patch OK for stage1 (after proper retesting)? Could you perhaps post the latest series first? I don't think it made it to the mailing list. -Andi

PR 59712 patch

2014-01-09 Thread François Dumont
Hi Here is a patch for this small problem with clang. It is not a blocking issue for the 4.9 release but at the same time it is a rather safe fix so just tell me if I can commit it. All unordered_* tests run under Linux x86_64. I haven't clang installed at the moment so a clang feedb

Re: Rb tree node recycling patch

2014-01-09 Thread François Dumont
On 01/08/2014 02:34 PM, Paolo Carlini wrote: Hi, On 12/27/2013 07:30 PM, François Dumont wrote: Note that this patch contains also a cleanup of a useless template parameter _Is_pod_comparator on _Rb_tree_impl. The useless parameter is a remnant of an attempt at exploiting the EBO for _Rb_tree_

[Patch] Fix PR plugin/59335, plugins not compiling

2014-01-09 Thread Steve Ellcey
Some header files needed by plugins are not present in the installed plugin include directory. This patch adds those headers. I added all the headers I needed for my plugin plus the ones mentioned in the patch. I did not add the x86 specific *.def files but this should fix all the generic platfo

Re: Question about gimplify.c:gimplify_adjust_omp_clauses_1, GOVD_MAP_TO_ONLY

2014-01-09 Thread Thomas Schwinge
Hi! On Thu, 9 Jan 2014 21:43:35 +0100, Jakub Jelinek wrote: > On Thu, Jan 09, 2014 at 09:38:25PM +0100, Thomas Schwinge wrote: > > In gimplify.c:gimplify_adjust_omp_clauses_1, does the case for > > GOVD_MAP_TO_ONLY have a real current use case (I couldn't spot any), or > > is it "just for complet

Re: Question about gimplify.c:gimplify_adjust_omp_clauses_1, GOVD_MAP_TO_ONLY

2014-01-09 Thread Thomas Schwinge
Hi! On Thu, 09 Jan 2014 23:21:26 +0100, I wrote: > On Thu, 9 Jan 2014 21:43:35 +0100, Jakub Jelinek wrote: > > On Thu, Jan 09, 2014 at 09:38:25PM +0100, Thomas Schwinge wrote: > > > In gimplify.c:gimplify_adjust_omp_clauses_1, does the case for > > > GOVD_MAP_TO_ONLY have a real current use case

Re: Rb tree node recycling patch

2014-01-09 Thread Paolo Carlini
Hi > Could you point me to the bugzilla entry you are mentioning ? libstdc++/29988 Thanks, Paolo

Re: PR 59712 patch

2014-01-09 Thread Jonathan Wakely
On 9 January 2014 21:55, François Dumont wrote: > > All unordered_* tests run under Linux x86_64. Please make sure you run the entire testsuite, not just the parts that seem relevant.

libgo patch committed: Fix 32-bit memory allocation

2014-01-09 Thread Ian Lance Taylor
This patch to libgo fixes memory allocation on 32-bit systems when a lot of memory has been allocated. The problem is described in this patch to the master repository: https://codereview.appspot.com/49460043 . runtime: fix 32-bit malloc for pointers >= 0x8000 The spans array is alloc

Go patch committed: Add flattening pass

2014-01-09 Thread Ian Lance Taylor
This patch from Chris Manghane adds a flattening pass to the Go frontend. This is a step toward moving more types of expressions into the backend interface. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 447683c37ddf go/expressions.h --- a/go/

Re: [PATCH] Allocate all target globals using GC for SWITCHABLE_TARGETs

2014-01-09 Thread Jakub Jelinek
On Thu, Jan 09, 2014 at 08:42:03AM -0800, Richard Henderson wrote: > On 01/09/2014 08:35 AM, Jakub Jelinek wrote: > > That would be fine for 1), but would mean 2). It is also fine to GC > > allocate each structure individually, but some (like bb_reorder) are say > > just 4 bytes long, so it might

[patch] fix libstdc++/59680

2014-01-09 Thread Jonathan Wakely
PR libstdc++/59680 * src/c++11/thread.cc (__sleep_for): Fix call to ::sleep. Tested x86_64-linux, and tested again with a hacked c++config.h to use ::sleep(), committed to trunk. commit 3de5274663e0214b8c5335e99ef85f325f709840 Author: Jonathan Wakely Date: Thu Jan 9 14:33:20 201

[committed] Fix vect_analyze_data_refs (PR middle-end/59670)

2014-01-09 Thread Jakub Jelinek
Hi! I forgot to check is_gimple_call before checking gimple_call_internal_p. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, committed as obvious. 2014-01-09 Jakub Jelinek PR middle-end/59670 * tree-vect-data-refs.c (vect_analyze_data_refs): Check

Re: [PING^2][PATCH] -fuse-caller-save - Implement TARGET_FN_OTHER_HARD_REG_USAGE hook for MIPS

2014-01-09 Thread Tom de Vries
On 09-01-14 16:31, Richard Sandiford wrote: Tom de Vries writes: On 25/12/13 14:02, Tom de Vries wrote: On 07-12-13 16:07, Tom de Vries wrote: Richard, This patch implements the target hook TARGET_FN_OTHER_HARD_REG_USAGE (posted here: http://gcc.gnu.org/ml/gcc-patches/2013-03/msg01318.html)

Re: [PATCH] Add zero-overhead looping for xtensa backend

2014-01-09 Thread Felix Yang
Hi Sterling, Please note that version 2 of the patch is for gcc trunk, not for gcc-4.8 branch. Since the doloop_end pattern format has changed, this patch need small adaptation in order for it to work on gcc-4.8. Although I test it on gcc-4.8, I think the testing result still holds fo

Re: [PATCH][IRA] Analysis of register usage of functions for usage by IRA.

2014-01-09 Thread Tom de Vries
On 09-01-14 22:10, Andi Kleen wrote: Tom de Vries writes: Is this patch OK for stage1 (after proper retesting)? Could you perhaps post the latest series first? I don't think it made it to the mailing list. Andi, the current status is: - toplevel of patch series: http://gcc.gnu.org/ml/

Re: [PATCH] Tiny predcom improvement (PR tree-optimization/59643)

2014-01-09 Thread H.J. Lu
On Tue, Dec 31, 2013 at 11:04 AM, Jakub Jelinek wrote: > Hi! > > As written in the PR, I've been looking why is llvm 3.[34] so much faster > on Scimark2 SOR benchmark and the reason is that it's predictive commoning > or whatever it uses doesn't give up on the inner loop, while our predcom > unnec

Re: [gofrontend-dev] libgo patch committed: Fix 32-bit memory allocation

2014-01-09 Thread Michael Hudson-Doyle
Ian Lance Taylor writes: > This patch to libgo fixes memory allocation on 32-bit systems when a lot > of memory has been allocated. The problem is described in this patch to > the master repository: https://codereview.appspot.com/49460043 . Here's a patch for the 4.8 branch if you are interest

RE: [PATCH] Add zero-overhead looping for xtensa backend

2014-01-09 Thread Yangfei (Felix)
And here is the xtensa configuration tested (include/xtensa-config.h): #define XCHAL_HAVE_BE 0 #define XCHAL_HAVE_LOOPS1 > > Hi Sterling, > > Please note that version 2 of the patch is for gcc trunk, not for > gcc-4.8 branch. > Since the doloop_end pattern fo

Re: [PATCH] libsanitizer demangling using cp-demangle.c

2014-01-09 Thread Konstantin Serebryany
On Thu, Jan 9, 2014 at 5:57 PM, Jakub Jelinek wrote: > On Thu, Jan 09, 2014 at 05:51:05PM +0400, Konstantin Serebryany wrote: >> On Tue, Dec 10, 2013 at 3:38 PM, Jakub Jelinek wrote: >> > On Fri, Dec 06, 2013 at 06:40:52AM -0800, Ian Lance Taylor wrote: >> >> There was a recent buggy patch to the

Re: Improving mklog [was: Re: RFC Asan instrumentation control]

2014-01-09 Thread Yury Gribov
> I hacked a simple addition to mklog which skips unchanged functions > in diff-log while adding function names to the final ChangeLog. > > New mklog results were verified by testsuite which compares reference > ChangeLogs of patches from gcc trunk with logs generated by mklog. > > Patched mklog c

Re: [gofrontend-dev] libgo patch committed: Fix 32-bit memory allocation

2014-01-09 Thread Ian Lance Taylor
On Thu, Jan 9, 2014 at 6:34 PM, Michael Hudson-Doyle wrote: > > Ian Lance Taylor writes: > >> This patch to libgo fixes memory allocation on 32-bit systems when a lot >> of memory has been allocated. The problem is described in this patch to >> the master repository: https://codereview.appspot.c

[PATCH] MIPS: improve Loongson-2E/2F/3A detection for -march=native

2014-01-09 Thread Huacai Chen
Hi all, For human-readability, I submit a patch to Linux kernel to display Loongson-2E/2F/3A in /proc/cpuinfo. But that break -march=native, this patch fix that. Regards, Huacai 2014-01-10 Huacai Chen * config/mips/driver-native.c: improve Loongson-2E/2F/3A detection --- gcc/confi

Go patch committed: Use backend interface for slice info

2014-01-09 Thread Ian Lance Taylor
This patch from Chris Manghane changes gccgo to use the backend interface for slice info. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 44fc257ad7f2 go/expressions.cc --- a/go/expressions.cc Thu Jan 09 15:25:44 2014 -0800 +++ b/go/expressions.

Re: RFC Asan instrumentation control

2014-01-09 Thread Maxim Ostapenko
Hi! >>> * c-c++-common/asan/no-asan-stack.c (this triggers read overflow >>> because we haven't found a cross-platform way to grep for stack >>> redzones instrumentation) >> >> I'd prefer no test in that case, or just some semi-platform specific test >> (scan that the 0x41b58ab3 constant doesn't

Re: RFC Asan instrumentation control

2014-01-09 Thread Jakub Jelinek
On Fri, Jan 10, 2014 at 11:49:43AM +0400, Maxim Ostapenko wrote: > 2014-01-10 Max Ostapenko > > * c-c++-common/asan/no-asan-stack.c: New test. > --- /dev/null > +++ b/gcc/testsuite/c-c++-common/asan/no-asan-stack.c > @@ -0,0 +1,17 @@ > +/* { dg-do assemble { target { x86_64-unknown-linux

<    1   2