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

2014-10-13 Thread Richard Biener
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 "-Ofast" on Silver

[PATCH] Fix typo in comment for IRA

2014-10-13 Thread Kito Cheng
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: Ditto. * ira-int.h: Ditto. * ir

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

2014-10-13 Thread Richard Biener
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 expression > > I like this. > > Unlike the other suggestions, it d

[Ada] Fix PR ada/63225

2014-10-13 Thread Eric Botcazou
As reported by Alan, the Ada compiler doesn't build anymore if you compile it with -fno-inline because cuintp.c references a private function of uintp.adb. Tested on x86_64-suse-linux, applied on all active branches. 2014-10-13 Eric Botcazou Alan Modra PR ada/63225

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

2014-10-13 Thread Mark Wielaard
On Wed, 2014-10-08 at 12:13 +0200, Mark Wielaard wrote: > This patch was written a long time ago by Jakub and has been in Fedora > gcc for some time. All I did was rebase it to current gcc trunk > and add a testcase. Back when it was originally proposed the issue was > that because DWARF was genera

Re: [PATCH] cleanups in line-map

2014-10-13 Thread Dodji Seketeli
Manuel López-Ibáñez writes: > A few cleanups in line-map code. Bootstrapped and regression tested on > x86_64-linux-gnu. Thanks for doing this. > OK? Yes, barring this little nit: [...] > Index: libcpp/line-map.c > === > --- lib

Re: [v3, patch] Move std::list::_M_size closer to iterators

2014-10-13 Thread Jonathan Wakely
On 12/10/14 21:41 +0200, Marc Glisse wrote: Hello, this patch moves _M_size in std::list to a place where it should be easier (no offsetof) to access from an iterator (it doesn't matter when you access it from the main std::list object). It does not take advantage of it yet, that can be done

Re: [PATCH] cleanups in line-map

2014-10-13 Thread Manuel López-Ibáñez
On 13 October 2014 10:52, Dodji Seketeli wrote: > Manuel López-Ibáñez writes: > >> Index: libcpp/line-map.c >> === >> --- libcpp/line-map.c (revision 216098) >> +++ libcpp/line-map.c (working copy) >> @@ -29,12 +29,10 @@ along with t

Re: [PATCH,1/2] Extended if-conversion for loops marked with pragma omp simd.

2014-10-13 Thread Yuri Rumyantsev
Richard, Here is updated patch (part1) for extended if conversion. Second part of patch will be sent later. Changelog. 2014-10-13 Yuri Rumyantsev * tree-if-conv.c (cgraph.h): Add include file to detect function clone. (flag_force_vectorize): New variable. (edge_predicate): New function. (se

[PATCH, 2/2] Extended if-conversion for loops marked with pragma omp simd.

2014-10-13 Thread Yuri Rumyantsev
Richard, Here is second part of patch which includes 1. One particular phi node recognition - if phi function has more than 2 arguments but it has only two really different arguments and one argument has the only occurrence. This is important for conditional scalar reduction conversion, e.g. for

[AArch64] [BE] [2/2] Make large opaque integer modes endianness-safe.

2014-10-13 Thread David Sherwood
Hi, This is the second patch of the work to fix: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59810 and removes the CANNOT_CHANGE_MODE_CLASS macro, which now permits subregs of vector registers to work correctly on aarch64_be. NOTE: This patch depends upon the following: [AArch64] [BE] [1/2] Ma

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

2014-10-13 Thread Jonathan Wakely
On 16/04/14 17:06 +0200, Rüdiger Sonderfeld wrote: C++11 [ptr.align]. This should probably not be inline. But for now this avoids any ABI changes. * libstdc++-v3/testsuite/20_util/align/1.cc: New file. * libstdc++-v3/include/std/memory (align): New function. Fixed (the new function should be

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

2014-10-13 Thread Jakub Jelinek
On Sat, Oct 11, 2014 at 06:49:00PM +0400, Ilya Verbin wrote: > This is the last common infrastructure patch in the series. > (Next patches will contain tests for libgomp testsuite and MIC specific > things) > > It introduces 2 new options: > 1. -foffload== >By default, GCC will build offload

[PATCHv5][PING] Vimrc config with GNU formatting

2014-10-13 Thread Yury Gribov
On 10/02/2014 09:14 PM, Yury Gribov wrote: On 09/17/2014 09:08 PM, Yury Gribov wrote: > On 09/16/2014 08:38 PM, Yury Gribov wrote: >> Hi all, >> >> This is the third version of the patch. A list of changes since last >> version: >> * move config to contrib so that it's _not_ enabled by defa

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

2014-10-13 Thread Ilya Verbin
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 something streamed already in the LTO bytec

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

2014-10-13 Thread Rüdiger Sonderfeld
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 look at the put_time patch. Unlike get_time it do

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

2014-10-13 Thread Ilya Enkovich
On 10 Oct 13:14, Jeff Law wrote: > On 10/08/14 12:55, Ilya Enkovich wrote: > >Hi, > > > >This patch introduces two IPA passes used by Pointer Bounds Checker. One > >pass creates clones for instrumentation. The other one transforms unneeded > >functions into thunks. > > > >Thanks, > >Ilya > >--

Re: Fix libgomp crash without TLS (PR42616)

2014-10-13 Thread Varvara Rainchik
> Now, I wonder on which OS and why does config/tls.m4 CHECK_GCC_TLS > actually fail? Can you figure that out? > On Android check passes with --disable-tls (standard while building gcc for Android as TLS is not supported in bionic) and fails with --enable-tls (i686-linux-android/libgomp/conftest.

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [6/n] Instrument calls and returns

2014-10-13 Thread Ilya Enkovich
On 10 Oct 12:50, Jeff Law wrote: > On 10/08/14 13:04, Ilya Enkovich wrote: > >Hi, > > > >This patch adds intrumentation of calls and returns into instrumentation > >pass. > > > >Thanks, > >Ilya > >-- > >2014-10-08 Ilya Enkovich > > > > * tree-chkp.c (chkp_add_bounds_to_ret_stmt): New. > >

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [7/n] Instrument params

2014-10-13 Thread Ilya Enkovich
On 10 Oct 12:10, Jeff Law wrote: > On 10/08/14 13:06, Ilya Enkovich wrote: > >Hi, > > > >This patch adds bounds initialization for address taken input arguments. > > > >Thanks, > >Ilya > >-- > >2014-10-08 Ilya Enkovich > > > > * tree-chkp.c (chkp_instrument_function): Store bounds for > >

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

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 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, 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 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 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: [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 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: [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. > > (

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 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

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: [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 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: [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

[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

[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

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

[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 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][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

[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

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

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

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 [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

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 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 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))

[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/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

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] 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: 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 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: [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 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 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] 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 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, 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 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 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: 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: 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: [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: [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, 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 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 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, 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, 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 [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

[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

[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

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

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

[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

[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 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

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

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: [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 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: [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] 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] 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] 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

[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: [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] 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: [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. >

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: 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: [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: [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

  1   2   >