Re: [PATCH] Fix typo in comment for IRA

2014-10-13 Thread Kito Cheng
Hi Marc: >> - -1 if it is not a cost classe. */ >> + -1 if it is not a cost classes. */ > > "a cost class", no plural here. Thank you for correcting me :) Hi Jeff: Thanks, and updated patch in attachment, However I don't have commit right yet, can you help me to commit it? thanks. Fr

[SH][committed] Fix PR 63260

2014-10-13 Thread Oleg Endo
og (revision 216172) +++ gcc/ChangeLog (working copy) @@ -1,3 +1,14 @@ +2014-10-14 Oleg Endo + + PR target/63260 + * config/sh/sh.md (negsf2, negsf2_i, negdf2, negdf2_i, abssf2, + abssf2_i, absdf2, absdf2_i): Remove fp_mode attribute. Remove use + of FPSCR. + (negsf2_i): Rename to *negsf2_i. + (abssf

Re: [C++ Patch] Move constexpr-related code to a new constexpr.c

2014-10-13 Thread Jason Merrill
OK. Jason

Re: RFA: fix mode confusion in caller-save.c:replace_reg_with_saved_mem

2014-10-13 Thread Joern Rennecke
On 13 October 2014 20:43, Jeff Law wrote: ... > I think you want "smode" in the mode_for_size call rather than "mode", right > (both instances)? No, nregs is the number of hard registers of regno in "mode". Hence we must use the size of "mode". How to choose the mode class is not so clear-cut.

Re: [PATCH 2/2] xtensa: use pre- and postincrement FP load/store when available

2014-10-13 Thread Max Filippov
On Mon, Oct 13, 2014 at 8:04 PM, augustine.sterl...@gmail.com wrote: > On Sun, Oct 12, 2014 at 3:46 PM, Max Filippov wrote: >> 2014-10-10 Max Filippov >> >> gcc/ >> * config/xtensa/xtensa.h (TARGET_HARD_FLOAT_POSTINC): new macro. >> * config/xtensa/xtensa.md (*lsiu, *ssiu): add depende

Re: [PATCH 4/n] OpenMP 4.0 offloading infrastructure: lto-wrapper

2014-10-13 Thread Ilya Verbin
On 10 Oct 19:00, Jakub Jelinek wrote: > On Fri, Oct 10, 2014 at 09:51:02AM -0700, Cary Coutant wrote: > > The linker already has a --strip-lto-sections option, and it's on by > > default. I'll approve a patch that modifies gold to recognize > > .gnu.offload_lto.* sections as part of --strip-lto-sec

Re: [PATCH 0/2] xtensa: fix floating-point parts of machine description

2014-10-13 Thread Max Filippov
On Mon, Oct 13, 2014 at 8:03 PM, augustine.sterl...@gmail.com wrote: > On Sun, Oct 12, 2014 at 3:46 PM, Max Filippov wrote: >> Hi Sterling, >> >> this series fixes two bugs in xtensa.md: > > HI Max, thanks for this. I don't see a patch though. It's a cover letter with a summary of changes. Patch

Re: [PATCH 2/2] Fix ILP32 ld.so.

2014-10-13 Thread Andrew Pinski
On Fri, Aug 8, 2014 at 8:51 PM, Andrew Pinski wrote: > This patch fixes the original problem that HJL was having with x32 in > PR 47727, it is more constraint than HJL's patch only care about what > is happening inside a CONST; if we allow it for other cases, > the RTL and generated code is incor

Re: [PATCH 1/2] Revert PR49721's patch

2014-10-13 Thread Andrew Pinski
On Fri, Aug 8, 2014 at 8:51 PM, Andrew Pinski wrote: > OK? When the second patch is approved? Ping? > > Thanks, > Andrew Pinski > > ChangeLog: > Revert: > 2011-08-19 H.J. Lu > > PR middle-end/49721 > * explow.c (convert_memory_address_addr_space): Also permute

ping x 7: [PATCH] [libgomp] make it possible to use OMP on both sides of a fork

2014-10-13 Thread Nathaniel Smith
Hi all, Got total silence the last 4 times I posted this, and users have been bugging me about it offline, so trying again. This patch fixes a showstopper problem preventing the transparent use of OpenMP in scientific libraries, esp. with Python. Specifically, it is currently not possible to use

Re: [PATCH] Merger of the "dmalcolm/jit" branch

2014-10-13 Thread Jeff Law
On 09/23/14 12:45, David Malcolm wrote: As mentioned before, I'm using Sphinx for documentation (and also checking in autogenerated .texinfo for those who want docs without a sphinx toolchain installed). I'm interested in how Sphinx is working for this project as well as the Ada folks who have

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [3/n] Helper functions

2014-10-13 Thread Ilya Enkovich
2014-10-14 1:05 GMT+04:00 Jeff Law : > On 10/08/14 12:57, Ilya Enkovich wrote: >> >> Hi, >> >> This patch introduces helper functions used by instrumentation. These >> helpers are used to access local structures, generate typical expressions, >> vars and statements. I also included local data ini

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [1/n] Expand interfaces

2014-10-13 Thread Jeff Law
On 10/08/14 12:50, Ilya Enkovich wrote: Hi, This patch starts a series which is a result of split of Pointer Bounds Checker patch #14 (Pointer Bounds Checker passes). This patch introduces compiler flags used by checker and helper functions mostly used by expand pass. Thanks, Ilya -- 2014-10

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [3/n] Helper functions

2014-10-13 Thread Jeff Law
On 10/08/14 12:57, Ilya Enkovich wrote: Hi, This patch introduces helper functions used by instrumentation. These helpers are used to access local structures, generate typical expressions, vars and statements. I also included local data initializer into this patch. Thanks, Ilya -- 2014-10-0

Re: [PATCH] cleanup in c-parser

2014-10-13 Thread Manuel López-Ibáñez
On 13 October 2014 20:52, Jeff Law wrote: > On 10/13/14 05:11, Anthony Brandon wrote: >> >> I updated the patch with a comment. Actually, Manuel handed me this >> patch just to help me get familiar with the process of submitting and >> testing. >> Generating this one with git diff looks different

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [13/n] Optimize bounds intersections

2014-10-13 Thread Ilya Enkovich
On 09 Oct 12:05, Jeff Law wrote: > On 10/08/14 13:19, Ilya Enkovich wrote: > >Hi, > > > >This patch adds removal of unnecessary intersections into checker > >optimizations. > > > >Thanks, > >Ilya > >-- > >2014-10-08 Ilya Enkovich > > > > * tree-chkp.c (chkp_release_check_info): New. > >

Re: __intN patch 3/5: main __int128 -> __intN conversion.

2014-10-13 Thread DJ Delorie
This is what I ended up with for the test case. It was a bit tricky since it only works with msp430x (not msp430) and requires the gnu extensions. Is this OK? If so, is there anything else, or can I check the whole mess in yet? // { dg-do compile { target msp430*-*-* } } // { dg-options "-std

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [4/n] Memory accesses instrumentation

2014-10-13 Thread Jeff Law
On 10/08/14 13:01, Ilya Enkovich wrote: Hi, This is the main chunk of instrumentation codes. This patch introduces instrumentation pass which instruments memory accesses. Thanks, Ilya -- 2014-10-08 Ilya Enkovich * tree-chkp.c (chkp_may_complete_phi_bounds): New. (chkp_may_fi

[PATCH] Handle CFA adjust notes in combine stack adjustments

2014-10-13 Thread Richard Henderson
ch but not mainline, but I'm applying it here anyway. Tested on i686 and x86_64. r~ 2014-10-13 Richard Henderson * combine-stack-adj.c (no_unhandled_cfa): New. (maybe_merge_cfa_adjust): New. (combine_stack_adjustments_for_block): Use them. * g++.dg/torture/

Re: [PATCH IRA] update_equiv_regs fails to set EQUIV reg-note for pseudo with more than one definition

2014-10-13 Thread Jeff Law
On 10/11/14 06:44, Felix Yang wrote: Hello Jeff, I see that you have improved the RTL typesafety issue for ira.c, so I rebased this patch on the latest trunk and change to use the new list walking interface. Bootstrapped on x86_64-SUSE-Linux and make check regression tested.

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [2/n] IPA passes

2014-10-13 Thread Jeff Law
On 10/13/14 04:48, Ilya Enkovich wrote: -- 2014-10-13 Ilya Enkovich * ipa-chkp.c: New. * ipa-chkp.h: New. * Makefile.in (OBJS): Add ipa-chkp.o. * c-family/c-common.c (c_common_attribute_table): Fix documentation. (c_common_format_attribute_table): Likew

Re: [C++] Handle && || ! for simd vectors

2014-10-13 Thread Marc Glisse
Ping https://gcc.gnu.org/ml/gcc-patches/2014-10/msg00361.html (sorry that my message looked like I had committed as obvious) On Sat, 4 Oct 2014, Marc Glisse wrote: On Thu, 2 Oct 2014, Jason Merrill wrote: OK. Thanks. While committing, I noticed that I restricted ! to integer vectors, where

Re: RFA: fix mode confusion in caller-save.c:replace_reg_with_saved_mem

2014-10-13 Thread Jeff Law
On 10/11/14 03:32, Joern Rennecke wrote: On 10 October 2014 21:13, Jeff Law wrote: ... ISTM it would be better to find the mode of the same class that corresponds to GET_MODE_SIZE (mode) / nregs. In your case that's obviously QImode :-) Like this? Or did you mean to remove the save_mode[regn

Re: [Patch] Fix PR61889 for the w64-mingw32 case

2014-10-13 Thread Xinliang David Li
Honza, not sure if this patch is idea, but this will unblock mingw build problems. Can this one get in? thanks, David On Wed, Sep 24, 2014 at 8:22 AM, Rainer Emrich wrote: > The following patch fixes PR61889 for x86_64-w64-mingw32. Details can be found > on https://gcc.gnu.org/bugzilla/show_bug

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [14/n] Optimize constant checks

2014-10-13 Thread Ilya Enkovich
On 09 Oct 11:13, Jeff Law wrote: > On 10/08/14 13:21, Ilya Enkovich wrote: > >Hi, > > > >This patch adds a removal of checks known to always pass into checker > >optimization. > > > >Thanks, > >Ilya > >-- > >2014-10-08 Ilya Enkovich > > > > * tree-chkp.c (chkp_remove_check_if_pass): New. >

[c++-concepts] introduction syntax regression

2014-10-13 Thread Andrew Sutton
The original patch for concept introductions was not popping a deferred access check. This fixes that problem, although I'm not sure if we need to defer access checks at all. 2014-10-13 Andrew Sutton Fix regression related to concept introductions. * gcc/cp/constraint.cc (cp_pa

Re: [c++-concepts] cleanup expressions

2014-10-13 Thread Andrew Sutton
And here's the patch: Andrew Sutton On Mon, Oct 13, 2014 at 3:33 PM, Andrew Sutton wrote: > Sometimes, cleanup_point_exprs are being added to concept definitions. > This patch allows that to happen, but removes the cleanup point during > normalization. > > 2014-10-13 Andrew Sutton > >

[c++-concepts] cleanup expressions

2014-10-13 Thread Andrew Sutton
Sometimes, cleanup_point_exprs are being added to concept definitions. This patch allows that to happen, but removes the cleanup point during normalization. 2014-10-13 Andrew Sutton Fix bug related to cleanup expressions in concept definitions. * gcc/cp/constraint.cc (check_fun

Re: [PATCH] move many gc hashtab to hash_table

2014-10-13 Thread Trevor Saunders
On Mon, Oct 13, 2014 at 09:02:44AM -0700, H.J. Lu wrote: > On Mon, Oct 13, 2014 at 8:33 AM, H.J. Lu wrote: > > On Fri, Oct 10, 2014 at 1:44 PM, Trevor Saunders > > wrote: > >> On Fri, Oct 10, 2014 at 02:26:00PM -0600, Jeff Law wrote: > >>> On 10/06/14 21:24, tsaund...@mozilla.com wrote: > >>> >F

Re: [PATCH] Fix typo in comment for IRA

2014-10-13 Thread Marc Glisse
On Mon, 13 Oct 2014, Kito Cheng wrote: - -1 if it is not a cost classe. */ + -1 if it is not a cost classes. */ "a cost class", no plural here. -- Marc Glisse

Re: [PATCH] Fix typo in comment for IRA

2014-10-13 Thread Jeff Law
On 10/13/14 02:09, Kito Cheng wrote: Hi all: This patch contain lots typo fix for IRA module by aspell :) ChangeLog 2014-10-13 Kito Cheng * ira.c: Fix typo in comment. * ira.h: Ditto. * ira-build.c: Ditto. * ira-color.c: Ditto. * ira-emit.c: Dit

Re: [PATCH] cleanup in c-parser

2014-10-13 Thread Jeff Law
On 10/13/14 05:11, Anthony Brandon wrote: I updated the patch with a comment. Actually, Manuel handed me this patch just to help me get familiar with the process of submitting and testing. Generating this one with git diff looks different so I'm not sure if that's a problem or not. This version

Re: [PATCH 1/X, i386, PR54232] Enable EBX for x86 in 32bits PIC code

2014-10-13 Thread H.J. Lu
On Mon, Oct 13, 2014 at 9:32 AM, Evgeny Stupachenko wrote: > Reattached. > > On Mon, Oct 13, 2014 at 8:22 PM, Uros Bizjak wrote: >> On Mon, Oct 13, 2014 at 4:53 PM, Evgeny Stupachenko >> wrote: >> >>> ChangeLog for testsuite: >>> >>> 2014-10-13 Evgeny Stupachenko >>> >>> PR target/83

Re: [gofrontend-dev] Re: [PATCH 03/13] HACK! Allow the static chain to be set from C

2014-10-13 Thread Peter Collingbourne
On Mon, Oct 13, 2014 at 1:10 AM, Richard Biener wrote: > On Sat, Oct 11, 2014 at 6:23 AM, Richard Henderson wrote: >> On 10/10/2014 06:42 PM, Peter Collingbourne wrote: >>> A colleague has suggested a perhaps nicer syntax: >>> >>> __builtin_call_chain(pointer, call) where call must be a call expr

Re: [PATCH 2/3] PR other/61321 - demangler crash on casts in template parameters

2014-10-13 Thread Cary Coutant
Ping. Jason, do you still think the special-case for conversion ops is inappropriate? -cary On Fri, Jul 25, 2014 at 2:16 AM, Pedro Alves wrote: > On 07/24/2014 11:35 PM, Cary Coutant wrote: >>> It seems that the problem here is more general; a template argument list is >>> not in scope within t

Re: PING: Re: [patch] tag ../include/*

2014-10-13 Thread Jeff Law
On 10/13/14 11:29, Aldy Hernandez wrote: On 10/13/14 09:27, Jeff Law wrote: On 10/13/14 08:53, Aldy Hernandez wrote: PING On 10/07/14 09:37, Aldy Hernandez wrote: Is there a reason we don't create etags for toplevel include files? If not, could I please apply this patch? ENOPATCH jeff So

[PATCH 4/5] State cleanups

2014-10-13 Thread David Malcolm
This patch contains various cleanups needed by the jit, so that we can rerun the compiler code repeatedly within the same process: * Introduce per-file finalization routines to clean up file-static variables. * Make various initializations idempotent (init_ggc), or cleanup their previous stat

[PATCH 3/5] timevar.h: Add an auto_timevar class

2014-10-13 Thread David Malcolm
This is used in a couple of places in jit/jit-playback.c to ensure that we pop the timevar on every exit path from a function. I could rewrite them if need be, but it does simplify things. Written by Tom Tromey. gcc/ChangeLog: * timevar.h (class auto_timevar): New class. --- gcc/timevar

[PATCH 2/5] gcc: configure and Makefile changes needed by jit

2014-10-13 Thread David Malcolm
gcc/ChangeLog: * configure.ac (gcc_version): Expose this value for use via AC_SUBST, since the jit code needs it within the new file libgccjit.pc.in. (doc_build_sys): New variable, set to "sphinx" if sphinx is installed, falling back to "texinfo" otherwise.

[PATCH 1/5] libiberty: Expose choose_tmpdir, and fix constness of return type

2014-10-13 Thread David Malcolm
The jit needs to create a temporary directory, so I need to expose libiberty's choose_tmpdir as a public entrypoint. include/ChangeLog: * libiberty.h (choose_tmpdir): New prototype. libiberty/ChangeLog: * choose-temp.c (choose_tmpdir): Remove now-redundant local copy of pr

[PATCH 0/5] Merger of jit branch (v2)

2014-10-13 Thread David Malcolm
I'd like to merge the JIT branch into trunk: https://gcc.gnu.org/wiki/JIT This is "v2" since it incorporates fixes for the various issues identified by Joseph in an earlier submission: https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02056.html I've split up the current diff between trunk and th

Re: [RFC: Patch, PR 60102] [4.9/4.10 Regression] powerpc fp-bit ices@dwf_regno

2014-10-13 Thread David Edelsohn
On Mon, Oct 13, 2014 at 9:20 AM, Ulrich Weigand wrote: > Maciej W. Rozycki wrote: >> On Thu, 9 Oct 2014, Maciej W. Rozycki wrote: >> >> > Seeing Rohit got good results it has struck me that perhaps one of the >> > patches I had previously reverted, to be able to compile GCC in the first >> > plac

Re: PING: Re: [patch] tag ../include/*

2014-10-13 Thread Aldy Hernandez
On 10/13/14 09:27, Jeff Law wrote: On 10/13/14 08:53, Aldy Hernandez wrote: PING On 10/07/14 09:37, Aldy Hernandez wrote: Is there a reason we don't create etags for toplevel include files? If not, could I please apply this patch? ENOPATCH jeff Sorry, patch was in original message. Attac

[committed] HP-UX 11 reentrant functions

2014-10-13 Thread John David Anglin
Most systems don't guard the C reentrant functions but on HP-UX 11 they are not declared unless _REENTRANT is defined. This causes problems for configure, etc. The attached change adds the _REENTRANT define when _HPUX_SOURCE is defined. A similar change was applied to HP-UX 10 three years a

[jit] Fix a comment in configure.ac

2014-10-13 Thread David Malcolm
Committed to branch dmalcolm/jit: The relevant code in jit/internal-api.c was moved to jit/jit-playback.c in 3071567787aef4a8ada8b38c890d01c19b4b998f. gcc/ChangeLog.jit: * configure.ac: Update a reference to jit/internal-api.c to jit/jit-playback.c. * configure: Regenerate

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [10/n] Stores handler

2014-10-13 Thread Jeff Law
On 10/13/14 05:23, Ilya Enkovich wrote: 2014-10-09 22:51 GMT+04:00 Jeff Law : On 10/08/14 13:12, Ilya Enkovich wrote: Hi, This patch adds an assignment processing function which is used by lnliner for newly generated stores. Thanks, Ilya -- 2014-10-08 Ilya Enkovich * tree-chkp.c

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [9/n] Static constructors

2014-10-13 Thread Jeff Law
On 10/13/14 06:41, Ilya Enkovich wrote: 2014-10-13 Ilya Enkovich * tree-chkp.c (MAX_STMTS_IN_STATIC_CHKP_CTOR): New. (chkp_ctor_stmt_list): New. (chkp_register_var_initializer): New. (chkp_add_modification_to_stmt_list): New. (chkp_output_static_bounds)

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [8/n] Remove useless builtin calls

2014-10-13 Thread Jeff Law
On 10/13/14 08:07, Ilya Enkovich wrote: Tests instrumentation are still needed. With some basic tests and instrumentation this will be OK. I hate to be harping tests, but few developers are going to be familiar with the MPX and related infrastructure and those tests are critical to helping them

Re: [PATCH 1/X, i386, PR54232] Enable EBX for x86 in 32bits PIC code

2014-10-13 Thread Uros Bizjak
On Mon, Oct 13, 2014 at 6:32 PM, Evgeny Stupachenko wrote: > Reattached. > > On Mon, Oct 13, 2014 at 8:22 PM, Uros Bizjak wrote: >> On Mon, Oct 13, 2014 at 4:53 PM, Evgeny Stupachenko >> wrote: >> >>> ChangeLog for testsuite: >>> >>> 2014-10-13 Evgeny Stupachenko >>> >>> PR target/83

Re: [PATCH 1/X, i386, PR54232] Enable EBX for x86 in 32bits PIC code

2014-10-13 Thread Evgeny Stupachenko
Reattached. On Mon, Oct 13, 2014 at 8:22 PM, Uros Bizjak wrote: > On Mon, Oct 13, 2014 at 4:53 PM, Evgeny Stupachenko > wrote: > >> ChangeLog for testsuite: >> >> 2014-10-13 Evgeny Stupachenko >> >> PR target/8340 >> PR middle-end/47602 >> PR rtl-optimization/55458 >>

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [12/n] Optimize string functions

2014-10-13 Thread Jeff Law
On 10/13/14 08:51, Ilya Enkovich wrote: I had assumed the gimple_call_set_fndecl was a nop if we didn't change the fndecl. Is that not the case? Right. But (!fndecl_nochk) doesn't mean we didn't change fndecl because there is another set to fndecl above. Thanks for clarifying. This is OK w

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

2014-10-13 Thread Felix Yang
Thanks for the comments. The patch checked the usage of teh trip count register, making sure that it is not used in the loop body other than the doloop_end or lives past the doloop_end instruction, as the following code snippet shows: + /* Scan all the blocks to make sure they don't use iter_reg

Re: [PATCH 1/X, i386, PR54232] Enable EBX for x86 in 32bits PIC code

2014-10-13 Thread Jeff Law
On 10/13/14 08:53, Evgeny Stupachenko wrote: ChangeLog for testsuite: 2014-10-13 Evgeny Stupachenko PR target/8340 PR middle-end/47602 PR rtl-optimization/55458 * gcc.target/i386/pic-1.c: Remove dg-error as test should pass now. * gcc.target/i386/

Re: PING: Re: [patch] tag ../include/*

2014-10-13 Thread Jeff Law
On 10/13/14 08:53, Aldy Hernandez wrote: PING On 10/07/14 09:37, Aldy Hernandez wrote: Is there a reason we don't create etags for toplevel include files? If not, could I please apply this patch? ENOPATCH jeff

RE: New rematerialization sub-pass in LRA

2014-10-13 Thread Wilco Dijkstra
> Here is a new rematerialization sub-pass of LRA. > > I've tested and benchmarked the sub-pass on x86-64 and ARM. The > sub-pass permits to generate a smaller code in average on both > architecture (although improvement no-significant), adds < 0.4% > additional compilation time in -O2 mode o

Re: [PATCH 1/X, i386, PR54232] Enable EBX for x86 in 32bits PIC code

2014-10-13 Thread Uros Bizjak
On Mon, Oct 13, 2014 at 4:53 PM, Evgeny Stupachenko wrote: > ChangeLog for testsuite: > > 2014-10-13 Evgeny Stupachenko > > PR target/8340 > PR middle-end/47602 > PR rtl-optimization/55458 > * gcc.target/i386/pic-1.c: Remove dg-error as test should pass now. >

Re: [PATCH 3/X, i386, PR54232] Enable EBX for x86 in 32bits PIC code

2014-10-13 Thread Uros Bizjak
On Mon, Oct 13, 2014 at 5:17 PM, Evgeny Stupachenko wrote: > Patch updated with the comment: > > diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c > index 2a64d2d..5fd6a82 100644 > --- a/gcc/config/i386/i386.c > +++ b/gcc/config/i386/i386.c > @@ -12455,9 +12455,18 @@ ix86_address_cost (

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [15/n] Optimize redundant checks

2014-10-13 Thread Jeff Law
On 10/13/14 08:58, Ilya Enkovich wrote: I'll look into DOM and a possibility to use it for checks removal. But I give higher priority to builtins instrumentation and therefore prefer to delay this one and return to it after builtins instrumentation work or in case there is some spare time for it

Re: [PATCH 2/X, i386, PR54232] Enable EBX for x86 in 32bits PIC code

2014-10-13 Thread Uros Bizjak
On Mon, Oct 13, 2014 at 5:01 PM, Evgeny Stupachenko wrote: >>-#define PIC_OFFSET_TABLE_REGNUM \ >>- ((TARGET_64BIT && (ix86_cmodel == CM_SMALL_PIC \ >>- || TARGET_PECOFF)) \ >>- || !flag_pic ? INVALID_REGNUM \ >>- : reload_completed ? REGNO (pic_offset_table_rtx) \ >>+#def

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

2014-10-13 Thread augustine.sterl...@gmail.com
On Fri, Oct 10, 2014 at 6:59 AM, Felix Yang wrote: > Hi Sterling, > > I made some improvement to the patch. Two changes: > 1. TARGET_LOOPS is now used as a condition of the doloop related > patterns, which is more elegant. Fine. > 2. As the trip count register of the zero-cost loop m

Re: [PATCH 1/2] xtensa: drop unimplemented floating point operations

2014-10-13 Thread augustine.sterl...@gmail.com
On Sun, Oct 12, 2014 at 3:46 PM, Max Filippov wrote: > xtensa ISA never implemented FP division, reciprocal, square root and > inverse square root as single opcode. Remove patterns that can emit > them. > > 2014-10-09 Max Filippov > > gcc/ > * config/xtensa/xtensa.md (divsf3, *recipsf2, sqr

Re: [PATCH 2/2] xtensa: use pre- and postincrement FP load/store when available

2014-10-13 Thread augustine.sterl...@gmail.com
On Sun, Oct 12, 2014 at 3:46 PM, Max Filippov wrote: > 2014-10-10 Max Filippov > > gcc/ > * config/xtensa/xtensa.h (TARGET_HARD_FLOAT_POSTINC): new macro. > * config/xtensa/xtensa.md (*lsiu, *ssiu): add dependency on > !TARGET_HARD_FLOAT_POSTINC. > (*lsip, *ssip): new instructio

Re: [PATCH 0/2] xtensa: fix floating-point parts of machine description

2014-10-13 Thread augustine.sterl...@gmail.com
On Sun, Oct 12, 2014 at 3:46 PM, Max Filippov wrote: > Hi Sterling, > > this series fixes two bugs in xtensa.md: HI Max, thanks for this. I don't see a patch though.

Re: [PATCH] move many gc hashtab to hash_table

2014-10-13 Thread H.J. Lu
On Mon, Oct 13, 2014 at 8:33 AM, H.J. Lu wrote: > On Fri, Oct 10, 2014 at 1:44 PM, Trevor Saunders > wrote: >> On Fri, Oct 10, 2014 at 02:26:00PM -0600, Jeff Law wrote: >>> On 10/06/14 21:24, tsaund...@mozilla.com wrote: >>> >From: Trevor Saunders >>> > >>> >Hi, >>> > >>> >This changes almost a

Re: New rematerialization sub-pass in LRA

2014-10-13 Thread Evgeny Stupachenko
I don't see significant performance changes from the patch (with and without patch enabling ebx) on x86 in 32bits mode. Thanks, Evgeny On Sat, Oct 11, 2014 at 2:31 AM, Sebastian Pop wrote: > Vladimir Makarov wrote: >> I've tested and benchmarked the sub-pass on x86-64 and ARM. The >> sub-pass

Re: [PATCH] move many gc hashtab to hash_table

2014-10-13 Thread H.J. Lu
On Fri, Oct 10, 2014 at 1:44 PM, Trevor Saunders wrote: > On Fri, Oct 10, 2014 at 02:26:00PM -0600, Jeff Law wrote: >> On 10/06/14 21:24, tsaund...@mozilla.com wrote: >> >From: Trevor Saunders >> > >> >Hi, >> > >> >This changes almost all of the ggc htab that don't use the if_marked option >> >t

Re: [PATCH 2/3] libstdc++: Add put_time support.

2014-10-13 Thread Jonathan Wakely
On 13/10/14 13:08 +0100, Jonathan Wakely wrote: On 15/04/14 23:20 +0200, Rüdiger Sonderfeld wrote: Described in [ext.manip]. * libstdc++-v3/include/std/iomanip (_Put_time): New struct. (put_time): New manipulator. (operator<<): New overloaded function. * libstdc++-v3/testsuite/27_io/manipulator

Re: [PATCH 3/X, i386, PR54232] Enable EBX for x86 in 32bits PIC code

2014-10-13 Thread Evgeny Stupachenko
Patch updated with the comment: diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c index 2a64d2d..5fd6a82 100644 --- a/gcc/config/i386/i386.c +++ b/gcc/config/i386/i386.c @@ -12455,9 +12455,18 @@ ix86_address_cost (rtx x, enum machine_mode, addr_space_t, bool) || REGNO (par

[jit] Work around a bug in dejagnu.exp's host_execute

2014-10-13 Thread David Malcolm
Committed to branch dmalcolm/jit: dejagnu.exp's host_execute has a bug, which (for me) reliably breaks the jit testsuite when running "make check" (though not when running "make check-jit" by itself, and appears to be unpleasantly timing-related). See the discussion at: http://lists.gnu.org/arc

Re: [PATCH 3/5] IPA ICF pass

2014-10-13 Thread Martin Liška
On 10/11/2014 02:05 AM, Martin Liška wrote: On 09/26/2014 09:46 PM, Jan Hubicka wrote: Hi, this is on ipa-icf-gimple.c @@ -2827,11 +2829,19 @@ cgraph_node::verify_node (void) { if (verify_edge_corresponds_to_fndecl (e, decl))

Re: [PATCH 6/n] OpenMP 4.0 offloading infrastructure: option handling

2014-10-13 Thread Bernd Schmidt
On 10/13/2014 12:33 PM, Ilya Verbin wrote: On 13 Oct 12:19, Jakub Jelinek wrote: But I'd like to understand why is this one needed. Why should the compilers care? Aggregates layout and alignment of integral/floating types must match between host and offload compilers, sure, but isn't that somet

Re: [PATCH 2/X, i386, PR54232] Enable EBX for x86 in 32bits PIC code

2014-10-13 Thread Evgeny Stupachenko
>-#define PIC_OFFSET_TABLE_REGNUM \ >- ((TARGET_64BIT && (ix86_cmodel == CM_SMALL_PIC \ >- || TARGET_PECOFF)) \ >- || !flag_pic ? INVALID_REGNUM \ >- : reload_completed ? REGNO (pic_offset_table_rtx) \ >+#define PIC_OFFSET_TABLE_REGNUM \ >+ ((TARGET_64BIT && (ix86_cmodel =

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [15/n] Optimize redundant checks

2014-10-13 Thread Ilya Enkovich
2014-10-10 20:56 GMT+04:00 Jeff Law : > On 10/10/14 09:50, Ilya Enkovich wrote: >> >> Checks and and intersection removal code was added as a simple pass >> catching trivial cases. I'm sure there are optimizations having >> common elements with what checker optimizer does. But initially we >> did

PING: Re: [patch] tag ../include/*

2014-10-13 Thread Aldy Hernandez
PING On 10/07/14 09:37, Aldy Hernandez wrote: Is there a reason we don't create etags for toplevel include files? If not, could I please apply this patch? Thanks. Aldy

Re: [PATCH 1/X, i386, PR54232] Enable EBX for x86 in 32bits PIC code

2014-10-13 Thread Evgeny Stupachenko
On Fri, Oct 10, 2014 at 8:03 PM, Jeff Law wrote: > On 10/10/14 01:42, Evgeny Stupachenko wrote: >> >> Hi, >> >> The patch enables EBX in RA for x86 32bits PIC mode. >> It was discussed here: >> https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02513.html >> Now there is working version with good perfo

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [12/n] Optimize string functions

2014-10-13 Thread Ilya Enkovich
On 10 Oct 10:25, Jeff Law wrote: > On 10/10/14 08:19, Ilya Enkovich wrote: > >>So is the purpose here to expose the checks that would normally be > >>done in the mem* routines to their caller in the hopes that doing > >>so will expose redundant checks? Or is there some other reason? > > > >There a

PR 62127 (VLA arrays and remap_type)

2014-10-13 Thread Jan Hubicka
Hi, this patch fixes bug I introduced in 2014-06-28 Jan Hubicka * tree-inline.c (remap_type_1): Do not duplicate fields that are shared in between type and its main variant. when adding the conditionals I somehow dropped else path of the test copying TREE_TYPE of array. Boots

[jit] Avoid overwhelming expect's buffer

2014-10-13 Thread David Malcolm
Committed to git branch dmalcolm/jit: "expect" has a default buffer size of 2000 bytes ("match_max"). If the testsuite programs send large amounts of data to stdout/stderr, they could overwhelm this buffer. Avoid this. gcc/testsuite/ChangeLog.jit: * jit.dg/harness.h [MAKE_DEJAGNU_H_THRE

[PATCH][AARCH64]Remove unused variable and marco

2014-10-13 Thread Renlin Li
Hi all, This is a simple patch to remove unused variables and marco. Is it Okay to commit? gcc/ChangeLog: 2014-10-13 Renlin Li * config/aarch64/aarch64.h (ARM_DEFAULT_PCS, arm_pcs_variant): Delete. Regards, Renlin LiFrom 88b6dc7fccf70c140070fbc5472d9bc1a8590754 Mon Sep 17 00:00:00 2001

Re: [PATCH 1/2] libstdc++: Add std::align.

2014-10-13 Thread Jonathan Wakely
On 13/10/14 14:04 +0100, Jonathan Wakely wrote: On 13/10/14 13:10 +0100, Jonathan Wakely wrote: On 13/10/14 12:35 +0200, Rüdiger Sonderfeld wrote: On Monday 13 October 2014 11:03:51 Jonathan Wakely wrote: Fixed (the new function should be in namespace std!) and applied to trunk. Thanks for the

[PATCH][ARM]Add ACLE 2.0 predefined marco __ARM_FEATURE_IDIV

2014-10-13 Thread Renlin Li
Hi all, This is a simple patch to add missing __ARM_FEATURE_IDIV__ predefined marco(ACLE 2.0) into TARGET_CPU_CPP_BUILTINS. Is it Okay to commit? gcc/ChangeLog: 2014-10-13 Renlin Li * config/arm/arm.h (TARGET_CPU_CPP_BUILTINS): Add ACLE 2.0 predefined marco __ARM_FEATURE_IDIV__. d

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [8/n] Remove useless builtin calls

2014-10-13 Thread Ilya Enkovich
On 10 Oct 10:11, Jeff Law wrote: > On 10/10/14 08:52, Ilya Enkovich wrote: > >> > >>THanks, Jeff > > > >With this code we remove user builtins calls coming from source code. > >E.g.: > > > >p2 = (int *)__bnd_init_ptr_bounds (p1); *p2 = 0; > > > >which means p2 has value of p1 but has default bounds

[committed] Fix comment about RUNTESTFLAGS and parallelization

2014-10-13 Thread Christophe Lyon
Hi, After Jakub's pre-approval https://gcc.gnu.org/ml/gcc/2014-10/msg00084.html I have committed the small attached patch as r216147. 2014-10-13 Christophe Lyon * Makefile.in: (check-%): Update comment, as RUNTESTFLAGS no longer impact parallelization. Christophe. Index: gcc

[C++ patch] PR tree-optimization/62053 (VLA arrays)

2014-10-13 Thread Jan Hubicka
Hi, this patch triggers assert in tree-inline that verify that array type and its main variant have same size (as pointer). In this case we have two types that do have same size, but the expression is different. It is created in both cases by layout_type becuase C++ FE calls layout_type before

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [11/n] Optimization helpers

2014-10-13 Thread Ilya Enkovich
On 10 Oct 10:14, Jeff Law wrote: > On 10/10/14 08:24, Ilya Enkovich wrote: > >On 09 Oct 12:09, Jeff Law wrote: > >>On 10/08/14 13:16, Ilya Enkovich wrote: > >>>Hi, > >>> > >>>This patch introduces structures and manipulation functions used by simple > >>>checker optimizations. Structures are used

Re: [PATCH 3/5] IPA ICF pass

2014-10-13 Thread Jan Hubicka
> > > >35% means that 35% of all function bodies are compared with something else? > >That feels pretty high. > >but overall numbers are not so terrible. > > Currently, the pass is able to merge 32K functions. As you know, we group > functions to so called classes. > According to stats, average

Re: [RFC: Patch, PR 60102] [4.9/4.10 Regression] powerpc fp-bit ices@dwf_regno

2014-10-13 Thread Ulrich Weigand
Maciej W. Rozycki wrote: > On Thu, 9 Oct 2014, Maciej W. Rozycki wrote: > > > Seeing Rohit got good results it has struck me that perhaps one of the > > patches I had previously reverted, to be able to compile GCC in the first > > place, interfered with this fix -- I backed out all the subseque

Re: [PATCH 3/5] IPA ICF pass

2014-10-13 Thread Martin Liška
On 10/11/2014 10:19 AM, Jan Hubicka wrote: After few days of measurement and tuning, I was able to get numbers to the following shape: Execution times (seconds) phase setup : 0.00 ( 0%) usr 0.00 ( 0%) sys 0.00 ( 0%) wall 1412 kB ( 0%) ggc phase opt and generate : 27.

Re: [PATCH 1/2] libstdc++: Add std::align.

2014-10-13 Thread Jonathan Wakely
On 13/10/14 13:10 +0100, Jonathan Wakely wrote: On 13/10/14 12:35 +0200, Rüdiger Sonderfeld wrote: On Monday 13 October 2014 11:03:51 Jonathan Wakely wrote: Fixed (the new function should be in namespace std!) and applied to trunk. Thanks for the patch. Thanks for merging and improving my pat

Fix bootstrap/63496 (pasto in ipa-polymorphic-call-context.c)

2014-10-13 Thread Jan Hubicka
Hi, this patch fixes pasto in ipa-polymorphic-call.c. Of course I was not intending to compare offset+64 with offset ;) Honza PR bootstrap/63496 * ipa-polymorphic-call.c (extr_type_from_vtbl_ptr_store): Fix pasto. Index: ipa-polymorphic-call.c

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [9/n] Static constructors

2014-10-13 Thread Ilya Enkovich
On 09 Oct 13:04, Jeff Law wrote: > On 10/08/14 13:10, Ilya Enkovich wrote: > >Hi, > > > >This patch introduces functions to handle static pointers and static bounds. > > > >Thanks, > >Ilya > >-- > >2014-10-08 Ilya Enkovich > > > > * tree-chkp.c (MAX_STMTS_IN_STATIC_CHKP_CTOR): New. > > (

Re: [PATCH x86] Increase PARAM_MAX_COMPLETELY_PEELED_INSNS when branch is costly

2014-10-13 Thread Jan Hubicka
> On Fri, Oct 10, 2014 at 5:40 PM, Evgeny Stupachenko > wrote: > > Hi, > > > > The patch increase PARAM_MAX_COMPLETELY_PEELED_INSNS for CPUs with > > high branch cost. > > Bootstrap and make check are in progress. > > The patch boosts (up to 2,5 times improve) several benchmarks compiled > > with

Re: [libstdc++] Add xmethods for array, deque, forward_list, list and vector.

2014-10-13 Thread Siva Chandra
On Mon, Oct 13, 2014 at 4:23 AM, Jonathan Wakely wrote: > On 12/10/14 06:49 -0700, Siva Chandra wrote: >> >> Hello, >> >> Attached is a patch which adds xmethods for std::array, std::deque, >> std::forward_list, std::list and std::vector. There were already >> couple of xmethods existing for std:

Re: [PATCH 1/2] libstdc++: Add std::align.

2014-10-13 Thread Jonathan Wakely
On 13/10/14 12:35 +0200, Rüdiger Sonderfeld wrote: On Monday 13 October 2014 11:03:51 Jonathan Wakely wrote: Fixed (the new function should be in namespace std!) and applied to trunk. Thanks for the patch. Thanks for merging and improving my patches! If you have the time maybe you could also

Re: [PATCH 2/3] libstdc++: Add put_time support.

2014-10-13 Thread Jonathan Wakely
On 15/04/14 23:20 +0200, Rüdiger Sonderfeld wrote: Described in [ext.manip]. * libstdc++-v3/include/std/iomanip (_Put_time): New struct. (put_time): New manipulator. (operator<<): New overloaded function. * libstdc++-v3/testsuite/27_io/manipulators/extended/put_time/char/1.cc: * libstdc++-v3/t

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [10/n] Stores handler

2014-10-13 Thread Ilya Enkovich
2014-10-09 22:51 GMT+04:00 Jeff Law : > On 10/08/14 13:12, Ilya Enkovich wrote: >> >> Hi, >> >> This patch adds an assignment processing function which is used by lnliner >> for newly generated stores. >> >> Thanks, >> Ilya >> -- >> 2014-10-08 Ilya Enkovich >> >> * tree-chkp.c (chkp_copy

Re: [libstdc++] Add xmethods for array, deque, forward_list, list and vector.

2014-10-13 Thread Jonathan Wakely
On 12/10/14 06:49 -0700, Siva Chandra wrote: Hello, Attached is a patch which adds xmethods for std::array, std::deque, std::forward_list, std::list and std::vector. There were already couple of xmethods existing for std::vector, but this patch adds more over them. Committed - thanks!

Re: [PATCH 1/2] libstdc++: Add std::align.

2014-10-13 Thread Jonathan Wakely
On 13/10/14 12:35 +0200, Rüdiger Sonderfeld wrote: On Monday 13 October 2014 11:03:51 Jonathan Wakely wrote: Fixed (the new function should be in namespace std!) and applied to trunk. Thanks for the patch. Thanks for merging and improving my patches! Sorry it's taken so long, I needed to get

Re: [PATCH] cleanup in c-parser

2014-10-13 Thread Anthony Brandon
I updated the patch with a comment. Actually, Manuel handed me this patch just to help me get familiar with the process of submitting and testing. Generating this one with git diff looks different so I'm not sure if that's a problem or not. Thanks, Anthony On Sun, Oct 12, 2014 at 10:09 PM, wrot

Re: [PATCH x86] Increase PARAM_MAX_COMPLETELY_PEELED_INSNS when branch is costly

2014-10-13 Thread Evgeny Stupachenko
I need to collect data from Haswell, but the patch should not help it's performance much, just increase code size. On Mon, Oct 13, 2014 at 12:01 PM, Richard Biener wrote: > On Fri, Oct 10, 2014 at 5:40 PM, Evgeny Stupachenko > wrote: >> Hi, >> >> The patch increase PARAM_MAX_COMPLETELY_PEELED_I

  1   2   >