[PATCH] Account for prologue spills in reg_pressure scheduling

2014-10-20 Thread Maxim Kuvyrkov
Hi, This patch improves register pressure scheduling (both SCHED_PRESSURE_WEIGHTED and SCHED_PRESSURE_MODEL) to better estimate number of available registers. At the moment the scheduler does not account for spills in the prologues and restores in the epilogue, which occur from use of

[PATCH] Improve scheduler dumps of ready list

2014-10-20 Thread Maxim Kuvyrkov
Hi, Following previous improvement to scheduler dumps that provided insight into which heuristics in rank_for_schedule make most decisions, this patch adds print outs that show the deciding reason for an instruction in the ready list to be at its particular place. This patch allowed me to

Re: [PATCH] Fix for PR63569

2014-10-20 Thread Richard Biener
On Fri, Oct 17, 2014 at 3:36 PM, Martin Liška mli...@suse.cz wrote: Hello. Following patch fixes PR63569. Bootstrap executed on ppc64-linux and no regression seen on x86_64-pc-linux. Ready for trunk? Um. As suggested in the bugreport I replied to please work on splitting out general

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

2014-10-20 Thread Richard Biener
On Fri, Oct 17, 2014 at 4:09 PM, Yuri Rumyantsev ysrum...@gmail.com wrote: Richard, I reworked the patch as you proposed, but I didn't understand what did you mean by: So please rework the patch so critical edges are always handled correctly. In current patch flag_force_vectorize is used

Re: [patch] Create cfgrtl.h

2014-10-20 Thread Richard Biener
On Fri, Oct 17, 2014 at 6:44 PM, Andrew MacLeod amacl...@redhat.com wrote: Rather than trying to flatten basic-block.h and do all the work associated in one big patch, I'll try to do it in smaller steps :-) This patch creates cfgrtl.h to maintain the prototypes for functions exported from

Re: [PATCH] PR preprocessor/42014

2014-10-20 Thread Krzesimir Nowak
2014-10-18 23:07 GMT+02:00 Krzesimir Nowak qdl...@gmail.com: Hello. This is my first patch for GCC. I already started a paperwork for copyright assignment (sent an email to fsf-records at gnu org) - waiting for response. So, about this patch - it basically removes column printing from In

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

2014-10-20 Thread Andreas Schwab
DJ Delorie d...@redhat.com writes: FAIL: g++.dg/init/enum1.C -std=gnu++11 (test for errors, line 12) FAIL: g++.dg/init/enum1.C -std=gnu++1y (test for errors, line 12) FAIL: g++.dg/init/enum1.C -std=gnu++98 (test for errors, line 12) That used to complain about enum1.C:12:1: error: no

Re: [PATCH PR63530] Fix the pointer alignment in vectorization

2014-10-20 Thread Richard Biener
On Fri, Oct 17, 2014 at 7:58 PM, Carrot Wei car...@google.com wrote: Hi In current vectorization pass, when a new vector pointer is created, its alignment is not set correctly. We should use DR_MISALIGNMENT (dr) since only this alignment is adjusted when loop peeling or multi version is

Re: [PATCH, rtl-optimization]: Remove const_alias_set

2014-10-20 Thread Richard Biener
On Sun, Oct 19, 2014 at 8:13 PM, Uros Bizjak ubiz...@gmail.com wrote: Hello! The fix that fixed scheduler issues with AND addresses (the fix prevented early exit for MEM_READONLY_P addresses when AND alignment addresses were involved) caused some fall-out for libgo testsuite. These tests

Re: [GOOGLE] Increase max-early-inliner-iterations to 2 for profile-gen and use

2014-10-20 Thread Richard Biener
On Mon, Oct 20, 2014 at 12:02 AM, Xinliang David Li davi...@google.com wrote: On Sat, Oct 18, 2014 at 4:19 PM, Xinliang David Li davi...@google.com wrote: On Sat, Oct 18, 2014 at 3:27 PM, Jan Hubicka hubi...@ucw.cz wrote: The difference in instrumentation runtime is huge -- as topn profiler is

Re: [AArch64] Add --enable-fix-cortex-a53-835769 configure-time option

2014-10-20 Thread Kyrill Tkachov
On 19/10/14 21:31, Gerald Pfeifer wrote: On Friday 2014-10-10 11:53, Kyrill Tkachov wrote: This adds a new configure-time option --enable-fix-cortex-a53-835769 that will enable the Cortex-A53 erratum fix by default so you don't have to specify -mfix-cortex-a53-835769 every time. Documentation

Re: [gomp4] Use GOMP_PLUGIN_ not gomp_plugin_ for libgomp plugin API

2014-10-20 Thread Thomas Schwinge
Hi Julian! On Fri, 17 Oct 2014 16:48:26 +0100, Julian Brown jul...@codesourcery.com wrote: As the title says, this patch makes the libgomp plugin API use the GOMP_PLUGIN_ prefix rather than gomp_plugin_. This is purely a mechanical change. OK for the gomp4 branch? Yes, thanks.

Re: [gomp4] Fix include path configury for gomp-constants.h

2014-10-20 Thread Thomas Schwinge
Hi Julian! On Fri, 17 Oct 2014 16:51:17 +0100, Julian Brown jul...@codesourcery.com wrote: This patch tweaks the include path configury used by libgomp to find the gomp-constants.h header, as suggested by Jakub. OK for the gomp4 branch? Thanks, yes, with the following changed:

Re: [gomp4] Asynchronous data unmapping wait fixes for OpenACC

2014-10-20 Thread Thomas Schwinge
Hi Julian! On Fri, 17 Oct 2014 17:05:40 +0100, Julian Brown jul...@codesourcery.com wrote: This patch introduces a new plugin hook in libgomp to register a callback function to clean up host-side bookkeeping data after an asynchronous operation has completed (replacing the previous ad-hoc

Re: [gomp4] OpenACC / C++

2014-10-20 Thread Thomas Schwinge
Hi! On Thu, 16 Oct 2014 19:38:23 +0200, I wrote: On Wed, 15 Oct 2014 11:21:05 -0500, James Norris jnor...@codesourcery.com wrote: This patch adds OpenACC support to C++ in the gomp4 branch. We understand that there will be further patches required on top of this, but we shall work on

[PATCH] Backport misalign tests to 4.9

2014-10-20 Thread Yury Gribov
On Fri, Oct 17, 2014 at 06:15:11PM +0400, Yury Gribov wrote: On 10/17/2014 05:49 PM, Jakub Jelinek wrote: So, what about this? Just checked that with make -k check-g{cc,++} RUNTESTFLAGS='--target_board=unix\{-m32,-m64\} asan.exp tsan.exp ubsan.exp' so far. Plus if you add misalign

Re: [PATCH] -fsanitize-recover=list

2014-10-20 Thread Yury Gribov
On 10/17/2014 08:13 PM, Jakub Jelinek wrote: On Mon, Oct 13, 2014 at 02:47:07PM +0400, Yury Gribov wrote: On 09/30/2014 09:39 PM, Jakub Jelinek wrote: LGTM, will hack it up soon in GCC then. Do you plan to work on this in near future? Here is only very lightly tested patch, didn't get to

Re: [PATCH] Backport misalign tests to 4.9

2014-10-20 Thread Jakub Jelinek
On Mon, Oct 20, 2014 at 02:37:21PM +0400, Yury Gribov wrote: On Fri, Oct 17, 2014 at 06:15:11PM +0400, Yury Gribov wrote: On 10/17/2014 05:49 PM, Jakub Jelinek wrote: So, what about this? Just checked that with make -k check-g{cc,++} RUNTESTFLAGS='--target_board=unix\{-m32,-m64\} asan.exp

Re: PR debug/60655, debug loc expressions

2014-10-20 Thread Alan Modra
On Thu, Oct 16, 2014 at 09:07:58AM +0200, Jakub Jelinek wrote: So, please find the spot where we forget to simplify stuff, and put the simplification there. You were correct to be suspicious that we weren't simplifying as we should. After more time in the debugger than I care to admit, I found

Re: PR debug/60655, debug loc expressions

2014-10-20 Thread Jakub Jelinek
On Mon, Oct 20, 2014 at 09:16:57PM +1030, Alan Modra wrote: PR debug/60655 * simplify-rtx.c (simplify_plus_minus): Delete unused input_ops. Increase ops array size. Correct array size tests. Init n_constants in loop. Break out of innermost loop when finding a

Re: [PATCH 2/n] OpenMP 4.0 offloading infrastructure: LTO streaming

2014-10-20 Thread Ilya Verbin
On 15 Oct 16:23, Richard Biener wrote: +static bool +initialize_offload (void) +{ + bool have_offload = false; + struct cgraph_node *node; + struct varpool_node *vnode; + + FOR_EACH_DEFINED_FUNCTION (node) +if (lookup_attribute (omp declare target, DECL_ATTRIBUTES

Re: [PATCH][1/n] Merge from match-and-simplify, public API

2014-10-20 Thread Richard Biener
On Fri, 17 Oct 2014, Jakub Jelinek wrote: On Wed, Oct 15, 2014 at 01:40:07PM +0200, Richard Biener wrote: 2014-10-15 Richard Biener rguent...@suse.de * gimple-fold.h (gimple_build): Declare various overloads. (gimple_simplify): Likewise. (gimple_convert): Re-implement in

Re: [PATCH 2/n] OpenMP 4.0 offloading infrastructure: LTO streaming

2014-10-20 Thread Jakub Jelinek
On Mon, Oct 20, 2014 at 03:19:35PM +0400, Ilya Verbin wrote: + /* If '#pragma omp critical' is inside target region, the symbol must + have an 'omp declare target' attribute. */ + omp_context *octx; + for (octx = ctx-outer; octx; octx = octx-outer) + if

Re: [PATCH][1/n] Merge from match-and-simplify, infrastructure

2014-10-20 Thread Richard Biener
On Fri, 17 Oct 2014, Jakub Jelinek wrote: On Wed, Oct 15, 2014 at 01:39:33PM +0200, Richard Biener wrote: 2014-10-15 Richard Biener rguent...@suse.de Shouldn't Prathamesh be listed as co-author of the patch? Yes, of course. + fprintf (f, case SSA_NAME:\n); + fprintf (f,

Re: [wwwdocs] Add recent C++ changes to gcc-5/changes.html

2014-10-20 Thread Jonathan Wakely
On 19/10/14 22:19 +0200, Gerald Pfeifer wrote: On Friday 2014-10-17 13:34, Jonathan Wakely wrote: Index: htdocs/gcc-5/changes.html === @@ -128,12 +164,13 @@ ul li Class codestd::experimental::any/code; /li li

Re: [PATCH][3/n] Merge from match-and-simplify, first patterns and questions

2014-10-20 Thread Richard Biener
On Fri, 17 Oct 2014, Jakub Jelinek wrote: On Wed, Oct 15, 2014 at 01:40:49PM +0200, Richard Biener wrote: This adds a bunch of simplifications with constant operands or ones that simplify to constants, such as a + 0, x * 1. It's a patch mainly to get a few questions answered for

Re: [PATCH][0/n] Merge from match-and-simplify

2014-10-20 Thread Richard Biener
On Fri, 17 Oct 2014, Sebastian Pop wrote: Sebastian Pop wrote: Richard Biener wrote: looks like RTL issues and/or IVOPTs issues? I should have posted the first diff between the compilers with -fdump-tree-all: that would expose the problem at its root. Looks like this is

Re: [PATCH][3/n] Merge from match-and-simplify, first patterns and questions

2014-10-20 Thread Richard Biener
On Sun, 19 Oct 2014, Marc Glisse wrote: Hello, looking though the patterns on the branch (not specifically the ones attached here), I am surprised to see so few calls to has_single_use. In RTL-land, we don't even valueize if there are several uses, so the question doesn't occur. In

[PATCH] More aggressively try swapping operands / SLP nodes

2014-10-20 Thread Richard Biener
This fixes the vectorizer testsuite fallout from folding all stmts which can swap tree operands where the SLP vectorizer currently cannot deal with that. It fixes it by making the SLP vectorizer deal with swapped operands in more cases, mostly SLP tree leafs and operations with one dt_external

[AARCH64, NEON] Any regression testcase for AARCH64 NEON intrinsics in GCC testsuite?

2014-10-20 Thread Yangfei (Felix)
Hi, I am trying to improve the AARCH64 NEON intrinsics. It seems that we don't enough testcases for this part in GCC testsuite. How do you guys test your patch on this part? Any suggestions? Thanks.

Re: [patch] LWG 2019 - std::isblankC(C, const std::locale)

2014-10-20 Thread Jonathan Wakely
On 17/10/14 10:41 +0100, Jonathan Wakely wrote: http://cplusplus.github.io/LWG/lwg-defects.html#2019 I've checked the relevant _ISblank/_ISBLANK/_CTYPE_B constant on all targets except VxWorks where I chose something that looks reasonable. Not all targets reserve a bit for isblank, but this way

Re: [PATCH i386 AVX512] [56/n] Add plus/minus/abs/neg/andnot insn patterns.

2014-10-20 Thread Jakub Jelinek
On Tue, Oct 14, 2014 at 11:18:28AM +0400, Kirill Yukhin wrote: * config/i386/sse.md (define_mode_iterator VI_AVX2): Extend to support AVX-512BW. (define_mode_iterator VI124_AVX2_48_AVX512F): Remove. (define_expand plusminus_insnmode3): Remove masking support.

Re: [libstdc++ PATCH] More Fundamentals v1 variable templates

2014-10-20 Thread Jonathan Wakely
On 18/10/14 23:22 +0300, Ville Voutilainen wrote: On 18 October 2014 23:18, Ville Voutilainen ville.voutilai...@gmail.com wrote: Tested on Linux-x64. 2014-10-18 Ville Voutilainen ville.voutilai...@gmail.com Implement more Library Fundamentals v1 variable templates for type traits.

[PATCH] Adjust testcases to be robust against operand order changes

2014-10-20 Thread Richard Biener
When folding all stmts we can end up canonicalizing operand order correctly which breaks at least the following two testcases. Fixed by making their expected outcome more robust. Tested on x86_64-unknown-linux-gnu, applied. Richard. 2014-10-20 Richard Biener rguent...@suse.de *

Re: [PATCH i386 AVX512] [56/n] Add plus/minus/abs/neg/andnot insn patterns.

2014-10-20 Thread Kirill Yukhin
Hello, On 20 Oct 14:36, Jakub Jelinek wrote: On Tue, Oct 14, 2014 at 11:18:28AM +0400, Kirill Yukhin wrote: * config/i386/sse.md (define_mode_iterator VI_AVX2): Extend to support AVX-512BW. (define_mode_iterator VI124_AVX2_48_AVX512F): Remove. (define_expand

Re: [PATCH i386 AVX512] [56/n] Add plus/minus/abs/neg/andnot insn patterns.

2014-10-20 Thread Jakub Jelinek
On Mon, Oct 20, 2014 at 05:30:36PM +0400, Kirill Yukhin wrote: Unfortunately this caused PR63600. The problem is that VI_AVX2 mode iterator includes V2DI and for AVX2 also V4DI, but for pre-ssse3 ix86_expand_sse2_abs doesn't handle V2DI (and can't easily, we don't have PSRAQ instruction),

Re: [PATCH i386 AVX512] [81/n] Add new built-ins.

2014-10-20 Thread Jakub Jelinek
On Mon, Oct 20, 2014 at 05:41:25PM +0400, Kirill Yukhin wrote: Hello, This patch adds (almost) all built-ins needed by AVX-512VL,BW,DQ intrinsics. Main questionable hunk is: diff --git a/gcc/tree-core.h b/gcc/tree-core.h index b69312b..a639487 100644 --- a/gcc/tree-core.h +++

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

2014-10-20 Thread Yuri Rumyantsev
Richard, Thanks for your answer! In current implementation phi node conversion assume that one of incoming edge to bb containing given phi has at least one non-critical edge and choose it to insert predicated code. But if we choose critical edge we need to determine insert point and insertion

[Ada] Spurious output on optimized default-initialized limited aggregate

2014-10-20 Thread Arnaud Charlet
When expanding a limited aggregate into individual assignments, we create a transient scope if the type of a component requires it. This must not be done if the context is an initialization procedure, because the target of the assignment must be visible outside of the block, and stack cleanup will

[Ada] Aspect specifications and incomplete views

2014-10-20 Thread Arnaud Charlet
Typically an indexing aspect is specified on the private view of a tagged type. In the unusual case where there is an incomplete view and the aspect specification appears on the full view, the aspect specification must be analyzed on the full view rather than the incomplete one, to prevent

Re: [PATCH] PR preprocessor/42014

2014-10-20 Thread Manuel López-Ibáñez
2014-10-18 23:07 GMT+02:00 Krzesimir Nowak qdl...@gmail.com: Hello. This is my first patch for GCC. I already started a paperwork for copyright assignment (sent an email to fsf-records at gnu org) - waiting for response. So, about this patch - it basically removes column printing from In

The nvptx port [0/11+]

2014-10-20 Thread Bernd Schmidt
This is a patch kit that adds the nvptx port to gcc. It contains preliminary patches to add needed functionality, the target files, and one somewhat optional patch with additional target tools. There'll be more patch series, one for the testsuite, and one to make the offload functionality work

The nvptx port [2/11+] No register allocation

2014-10-20 Thread Bernd Schmidt
Since it's a virtual target, I've chosen not to run register allocation. This is one of the patches necessary to make that work, it primarily adds a target hook to disable it and fixes some of the fallout. Bernd

The nvptx port [1/11+] indirect jumps

2014-10-20 Thread Bernd Schmidt
ptx doesn't have indirect jumps, so CODE_FOR_indirect_jump may not be defined. Add a sorry. Bernd gcc/ * optabs.c (emit_indirect_jump): Test HAVE_indirect_jump and emit a sorry if necessary. Index: gcc/optabs.c

[Ada] Lift limitation on inter-unit inlining of instantiated subprograms

2014-10-20 Thread Arnaud Charlet
This change makes it so that instantiations of generic subprograms marked as inline are considered for inter-unit inlining. This was not previously the case because of a technical limitation that was too broadly enforced (unlike the associated comment which was more accurate) and excluded

Re: The nvptx port [3/11+] Struct returns

2014-10-20 Thread Bernd Schmidt
Even when returning a structure by passing an invisible reference, gcc still likes to set the return register to the address of the struct. This is undesirable on ptx where things like the return register have to be declared, and the function really returns void at ptx level. I've added a

[Ada] Implement pragma/aspect No_Tagged_Streams

2014-10-20 Thread Arnaud Charlet
The No_Tagged_Streams pragma (and aspect) provides a method for selectively inhibiting the generation of stream routines for tagged types. It can be used either in a form naming a specific tagged type, or in a sequence of declarations to apply to all subsequent declarations. The following tests

The nvptx port [4/11+] Post-RA pipeline

2014-10-20 Thread Bernd Schmidt
This stops most of the post-regalloc passes to be run if the target doesn't want register allocation. I'd previously moved them all out of postreload to the toplevel, but Jakub (I think) pointed out that the idea is not to run them to avoid crashes if reload fails e.g. for an invalid asm. So

The nvptx port [5/11+] Variable declarations

2014-10-20 Thread Bernd Schmidt
ptx assembly follows rather different rules than what's typical elsewhere. We need a new hook to add a }; string when we are finished outputting a variable with an initializer. Bernd gcc/ * target.def (decl_end): New hook. * varasm.c (assemble_variable_contents,

[Ada] Improve error recovery for bad comma/semicolon in expression

2014-10-20 Thread Arnaud Charlet
This patch improves the error recovery for an errant comma or semicolon after one condition in an expression when more conditions follow, as shown in this example: 1. procedure BadANDTHEN (X : Integer) is 2. begin 3.if X 10 4. and then X mod 4 = 2;

[Ada] Improve recognition of misspelled aspects

2014-10-20 Thread Arnaud Charlet
As shown by this example, the recognition of misspelled aspects is improved: 1. package UnrecogAs with Prelaborate is | Prelaborate is not a valid aspect identifier possible misspelling of Preelaborate 2.type R is tagged null record;

The nvptx port [6/11+] Pseudo call args

2014-10-20 Thread Bernd Schmidt
On ptx, we'll be using pseudos to pass function args as well, and there's one assert that needs to be toned town to make that work. Bernd gcc/ * expr.c (use_reg_mode): Just return for pseudo registers. Index:

The nvptx port [7/11+] Inform the port about call arguments

2014-10-20 Thread Bernd Schmidt
In ptx assembly we need to decorate call insns with the arguments that are being passed. We also need to know the exact function type. This is kind of hard to do with the existing infrastructure since things like function_arg are called at other times rather than just when emitting a call, so

The nvptx port [8/11+] Write undefined decls.

2014-10-20 Thread Bernd Schmidt
ptx assembly requires that declarations are written for undefined variables. This adds that functionality. Bernd gcc/ * target.def (assemble_undefined_decl): New hooks. * hooks.c (hook_void_FILEptr_constcharptr_const_tree): New function. * hooks.h

[Ada] Crash on unconstrained unchecked union declaration

2014-10-20 Thread Arnaud Charlet
When an object declaration as an indefinite type, the actual subtype of the object is constructed from the expression itself. If the type is an unchecked union such a subtype cannot be constructed because discriminants cannot be retrieved from the expression. In this case, rewrite declaration as a

The nvptx port [9/11+] Epilogues

2014-10-20 Thread Bernd Schmidt
We skip the late compilation passes on ptx, but there's one piece we do need - fixing up the function so that we get return insns in the right places. This patch just makes thread_prologue_and_epilogue_insns callable from the reorg pass. Bernd gcc/ * function.c

The nvptx port [10/11+] Target files

2014-10-20 Thread Bernd Schmidt
These are the main target files for the ptx port. t-nvptx is empty for now but will grow some content with follow up patches. Bernd * configure.ac: Allow configuring lto for nvptx. * configure: Regenerate. gcc/ * config/nvptx/nvptx.c: New file. * config/nvptx/nvptx.h: New file. *

[Ada] Slices of parameterless calls

2014-10-20 Thread Arnaud Charlet
This patch handles correctly constructs of the forms F (T) where F denotes a possibly overloaded function that can be invoked without actual parameters, and T denotes a discrete type. The construct is parsed as an indexed component but must be rewritten and analyzed as a slice of a parameterless

The nvptx port [11/11] More tools.

2014-10-20 Thread Bernd Schmidt
This is a bonus optional patch which adds ar, ranlib, as and ld to the ptx port. This is not proper binutils; ar and ranlib are just linked to the host versions, and the other two tools have the following functions: * nvptx-as is required to convert the compiler output to actual valid ptx

Re: [PATCH i386 AVX512] [56/n] Add plus/minus/abs/neg/andnot insn patterns.

2014-10-20 Thread Uros Bizjak
On Mon, Oct 20, 2014 at 3:41 PM, Jakub Jelinek ja...@redhat.com wrote: On Mon, Oct 20, 2014 at 05:30:36PM +0400, Kirill Yukhin wrote: Unfortunately this caused PR63600. The problem is that VI_AVX2 mode iterator includes V2DI and for AVX2 also V4DI, but for pre-ssse3 ix86_expand_sse2_abs

Re: [AARCH64, NEON] Any regression testcase for AARCH64 NEON intrinsics in GCC testsuite?

2014-10-20 Thread Christophe Lyon
On 20 October 2014 14:01, Yangfei (Felix) felix.y...@huawei.com wrote: Hi, I am trying to improve the AARCH64 NEON intrinsics. It seems that we don't enough testcases for this part in GCC testsuite. How do you guys test your patch on this part? Any suggestions? Thanks. Hello, I have

Re: [PATCH i386 AVX512] [63.1/n] Add vpshufb, perm autogen (except for v64qi).

2014-10-20 Thread Ilya Tocar
The patch is OK with the above improvement. Will commit version below, if no objections in 24 hours. Sorry, I've missed palignr, which should also have v64qi version, and lost return in expand_vec_perm_palignr case (this caused avx512f-vec-unpack test failures). Patch below fixes

RE: [Patch, MIPS] Add Octeon3 support

2014-10-20 Thread Matthew Fortune
2014-10-08 Andrew Pinski apin...@cavium.com * config/mips/mips-cpus.def (octeon3): New cpu. * config/mips/mips.c (mips_rtx_cost_data): Add octeon3. (mips_print_operand case 'T', case 't'): Fix a bug as the mode of the comparison no longer matches mode of the operands.

[c++-concepts]

2014-10-20 Thread Andrew Sutton
Fixing issues reported by users. 2014-10-20 Andrew Sutton andrew.n.sut...@gmail.com Fixing user-reported issues and regressions * gcc/cp/parser.c (cp_parser_template_declaration_after_exp): Only pop access checks on failed parsing. * gcc/cp/pt.cpp

Re: [PATCH] Don't call fatal_error before error reporting has been initialized.

2014-10-20 Thread Ilya Tocar
Same in collect2. On 09 Oct 15:40, Ilya Tocar wrote: Ping. On 29 Sep 18:02, Ilya Tocar wrote: Hi, Currently if call to atexit (lto_wrapper_cleanup) fails we won't report error as we haven't initialized error-reporting infrastructure. This patch moves this call after

Re: [PING][PATCH] Warn about unclosed pragma omp declare target.

2014-10-20 Thread Ilya Tocar
Ping. On 02 Oct 17:38, Ilya Tocar wrote: Ping. On 15 Aug 16:26, Ilya Tocar wrote: Ping. On 29 Jul 18:45, Ilya Tocar wrote: Hi, As discussed here in https://gcc.gnu.org/ml/gcc/2014-01/msg00189.html Gcc should complain about pragma omp declare target without corresponding

Re: [GOOGLE] Increase max-early-inliner-iterations to 2 for profile-gen and use

2014-10-20 Thread Xinliang David Li
On Mon, Oct 20, 2014 at 1:32 AM, Richard Biener richard.guent...@gmail.com wrote: On Mon, Oct 20, 2014 at 12:02 AM, Xinliang David Li davi...@google.com wrote: On Sat, Oct 18, 2014 at 4:19 PM, Xinliang David Li davi...@google.com wrote: On Sat, Oct 18, 2014 at 3:27 PM, Jan Hubicka

Re: [PATCH i386 AVX512] [63.1/n] Add vpshufb, perm autogen (except for v64qi).

2014-10-20 Thread Uros Bizjak
On Mon, Oct 20, 2014 at 5:19 PM, Ilya Tocar tocarip.in...@gmail.com wrote: The patch is OK with the above improvement. Will commit version below, if no objections in 24 hours. Sorry, I've missed palignr, which should also have v64qi version, and lost return in

[PING][PATCH] GCC/test: Set timeout factor for c11-atomic-exec-5.c

2014-10-20 Thread Maciej W. Rozycki
Hi, I thought http://gcc.gnu.org/ml/gcc-patches/2014-09/msg00242.html would be folded into PowerPC TARGET_ATOMIC_ASSIGN_EXPAND_FENV support, but I see r216437 went without it. In that case would someone please review my proposal as a separate change? Thanks, Maciej

Re: -fuse-caller-save - Collect register usage information

2014-10-20 Thread Eric Botcazou
But, given the preference of a number of others for fipa-ra, could you live with that? Yes, IMO that's too vague a name but still better than the existing one. :-) -- Eric Botcazou

[Patch, libstdc++/63497] Avoid dereferencing invalid iterator in regex_executor

2014-10-20 Thread Tim Shen
Bootstrapped and tested. Thanks! -- Regards, Tim Shen commit 95c73ab6280c1f8182d018ee29a44230965dd4ef Author: timshen tims...@google.com Date: Sun Oct 19 15:14:55 2014 -0700 PR libstdc++/63497 include/bits/regex_executor.h (_Executor::_M_word_boundary): Remove const

C++ PATCH for c++/63601 (lambda, 'this' outside class)

2014-10-20 Thread Jason Merrill
finish_this_expr needs to be prepared for lambda_expr_this_capture to return NULL_TREE. Tested x86_64-pc-linux-gnu, applying to trunk. commit c631290819f1ab3754041c46d351745953fb8319 Author: Jason Merrill ja...@redhat.com Date: Mon Oct 20 09:56:35 2014 -0400 PR c++/63601 * lambda.c

[PATCH] Add top-level config support for gold mips target

2014-10-20 Thread Cary Coutant
This patch adds support for the mips target in gold. OK to commit? -cary 2014-10-20 Cary Coutant ccout...@google.com * configure (--enable-gold): Add mips*-*-*. * configure.ac: Regenerate. Index: configure ===

[Google/gcc-4_9][PATCH][target/x86_64] PR 63538

2014-10-20 Thread Sriraman Tallam
Hi, This patch is under review for trunk GCC : https://gcc.gnu.org/ml/gcc-patches/2014-10/msg01638.html. In the mean time, is this ok for google/gcc-4_9 branch? Without this, -mcmodel=medium is unusable if .lrodata goes beyond the 2G boundary. Thanks Sri Index:

Re: [Google/gcc-4_9][PATCH][target/x86_64] PR 63538

2014-10-20 Thread Xinliang David Li
Why removing the tree_code check? David On Mon, Oct 20, 2014 at 10:35 AM, Sriraman Tallam tmsri...@google.com wrote: Hi, This patch is under review for trunk GCC : https://gcc.gnu.org/ml/gcc-patches/2014-10/msg01638.html. In the mean time, is this ok for google/gcc-4_9 branch?

Re: [Google/gcc-4_9][PATCH][target/x86_64] PR 63538

2014-10-20 Thread Sriraman Tallam
On Mon, Oct 20, 2014 at 10:42 AM, Xinliang David Li davi...@google.com wrote: Why removing the tree_code check? The actual problem happens because STRING_CSTs (end up in .lrodata) are not set a far address as they dont match the VAR_DECL check here. Futher, ix86_in_large_data_p call has the

Re: [Google/gcc-4_9][PATCH][target/x86_64] PR 63538

2014-10-20 Thread Andrew Pinski
On Mon, Oct 20, 2014 at 10:46 AM, Sriraman Tallam tmsri...@google.com wrote: On Mon, Oct 20, 2014 at 10:42 AM, Xinliang David Li davi...@google.com wrote: Why removing the tree_code check? The actual problem happens because STRING_CSTs (end up in .lrodata) are not set a far address as they

Re: [Google/gcc-4_9][PATCH][target/x86_64] PR 63538

2014-10-20 Thread Xinliang David Li
Perhaps explicitly allowing STRING_CST to go through the large data check, instead of removing the var-decl check? Do you see other opcodes that need to be handled too? David On Mon, Oct 20, 2014 at 10:46 AM, Sriraman Tallam tmsri...@google.com wrote: On Mon, Oct 20, 2014 at 10:42 AM, Xinliang

libgo patch committed: Allocate correct types in refect for interface conversions

2014-10-20 Thread Ian Taylor
This patch to libgo is a copy of a patch I recently made to the master Go library. This changes the reflect package to allocate memory using the correct types for interface conversions. The code was incorrectly allocating an empty interface type to hold a non-empty interface value. This was

[patch] Second basic-block.h restructuring patch.

2014-10-20 Thread Andrew MacLeod
creates cfg.h, cfganal.h, lcm.h, and loop-unroll.h to house the prototypes for those .c files. cfganal.h also gets struct edge_list and class control_dependences definitions since that is where all the routines and manipulators are declared. loop-unroll.h only exports 2 routines, so

[jit] Error-handling within gcc::jit::dump

2014-10-20 Thread David Malcolm
On Fri, 2014-10-17 at 21:52 +, Joseph S. Myers wrote: [...snip static linkage discussion...] The dump file handling appears to have no I/O error checking (no checking for error on fopen, nothing obvious to prevent fwrite to a NULL m_file if fopen did have an error, no checking for error

Re: [Google/gcc-4_9][PATCH][target/x86_64] PR 63538

2014-10-20 Thread Sriraman Tallam
On Mon, Oct 20, 2014 at 10:59 AM, Xinliang David Li davi...@google.com wrote: Perhaps explicitly allowing STRING_CST to go through the large data check, instead of removing the var-decl check? Do you see other opcodes that need to be handled too? I do not see any other opcodes explicitly but

Re: [Google/gcc-4_9][PATCH][target/x86_64] PR 63538

2014-10-20 Thread Sriraman Tallam
On Mon, Oct 20, 2014 at 10:51 AM, Andrew Pinski pins...@gmail.com wrote: On Mon, Oct 20, 2014 at 10:46 AM, Sriraman Tallam tmsri...@google.com wrote: On Mon, Oct 20, 2014 at 10:42 AM, Xinliang David Li davi...@google.com wrote: Why removing the tree_code check? The actual problem happens

Re: [PATCH] Improve scheduler dumps of ready list

2014-10-20 Thread Sebastian Pop
Maxim Kuvyrkov wrote: Hi, Following previous improvement to scheduler dumps that provided insight into which heuristics in rank_for_schedule make most decisions, this patch adds print outs that show the deciding reason for an instruction in the ready list to be at its particular place.

Re: [PATCH] Account for prologue spills in reg_pressure scheduling

2014-10-20 Thread Sebastian Pop
Maxim Kuvyrkov wrote: Hi, This patch improves register pressure scheduling (both SCHED_PRESSURE_WEIGHTED and SCHED_PRESSURE_MODEL) to better estimate number of available registers. At the moment the scheduler does not account for spills in the prologues and restores in the epilogue,

Go patch committed: Pass type information to heap allocations

2014-10-20 Thread Ian Taylor
This patch by Chris Manghane passes type information to compiler-generated heap allocations in gccgo. This gives us precise type information for much of the gccgo heap, and means that garbage collection is much more precise and less prone to errors due to mistaking integer or float values as

Re: [PATCH] Account for prologue spills in reg_pressure scheduling

2014-10-20 Thread Maxim Kuvyrkov
On Oct 21, 2014, at 8:11 AM, Sebastian Pop seb...@gmail.com wrote: Maxim Kuvyrkov wrote: Hi, This patch improves register pressure scheduling (both SCHED_PRESSURE_WEIGHTED and SCHED_PRESSURE_MODEL) to better estimate number of available registers. At the moment the scheduler does not

[jit] Add Sphinx to install.texi

2014-10-20 Thread David Malcolm
On Fri, 2014-10-17 at 21:25 +, Joseph S. Myers wrote: Although Sphinx isn't a build dependency, as a dependency for regenerating checked-in files I think it should be documented in install.texi (like autoconf, gettext, etc.). Does this look OK? (Committed to branch dmalcolm/jit for now)

Re: [Google/gcc-4_9][PATCH][target/x86_64] PR 63538

2014-10-20 Thread Xinliang David Li
On Mon, Oct 20, 2014 at 11:59 AM, Sriraman Tallam tmsri...@google.com wrote: On Mon, Oct 20, 2014 at 10:51 AM, Andrew Pinski pins...@gmail.com wrote: On Mon, Oct 20, 2014 at 10:46 AM, Sriraman Tallam tmsri...@google.com wrote: On Mon, Oct 20, 2014 at 10:42 AM, Xinliang David Li

Re: [PATCH PR63530] Fix the pointer alignment in vectorization

2014-10-20 Thread Carrot Wei
Hi Richard An arm testcase that can reproduce this bug is attached. 2014-10-20 Guozhi Wei car...@google.com PR tree-optimization/63530 gcc.target/arm/pr63530.c: New testcase. Index: pr63530.c === --- pr63530.c

Re: [jit] Drop libgccjit.pc

2014-10-20 Thread Basile Starynkevitch
On Mon, 2014-10-20 at 13:54 -0400, David Malcolm wrote: Committed to branch dmalcolm/jit: pkg-config appears to be controversial, so don't provide a .pc file. I would put it under contrib/; it is controversial, but some would like to have it. Cheers. -- Basile STARYNKEVITCH

[gomp4] c++ delete clause

2014-10-20 Thread Cesar Philippidis
The OpenACC delete clause isn't detected in the c++ front end because the lexer classifies it as a keyword, which it is. This patch makes the openacc pragma parser aware of that. I've committed this patch to gomp-4_0-branch. A test case will be included in a follow up patch along with support for

[gomp4] acc update bug

2014-10-20 Thread Cesar Philippidis
The OpenACC update directive would cause an ICE if there was an error parsing one of its clauses in the c front end. E.g. #pragma acc update copy(a(1:10)). This patch fixes that. Also, it declare GOACC_update inside libgomp_g.h I've committed this patch to gomp-4_0-branch. A test case will be

Re: [gomp4] c++ delete clause

2014-10-20 Thread Jakub Jelinek
On Mon, Oct 20, 2014 at 01:12:08PM -0700, Cesar Philippidis wrote: The OpenACC delete clause isn't detected in the c++ front end because the lexer classifies it as a keyword, which it is. This patch makes the openacc pragma parser aware of that. I've committed this patch to gomp-4_0-branch.

[Patch, Fortran] Add CO_REDUCE

2014-10-20 Thread Tobias Burnus
This patch adds some more checks and the actual implementation (compiler side and libcaf_single) for CO_REDUCE. It also rejects coindexed variables elsewhere, in line with the recent J3 changes. I also updated the API documentation (adding doc for collectives.) Still unsupported as elsewhere

Re: [gomp4] c++ delete clause

2014-10-20 Thread Cesar Philippidis
On 10/20/2014 01:18 PM, Jakub Jelinek wrote: On Mon, Oct 20, 2014 at 01:12:08PM -0700, Cesar Philippidis wrote: The OpenACC delete clause isn't detected in the c++ front end because the lexer classifies it as a keyword, which it is. This patch makes the openacc pragma parser aware of that.

Re: [jit] Drop libgccjit.pc

2014-10-20 Thread Matthias Klose
Am 20.10.2014 um 22:11 schrieb Basile Starynkevitch: On Mon, 2014-10-20 at 13:54 -0400, David Malcolm wrote: Committed to branch dmalcolm/jit: pkg-config appears to be controversial, so don't provide a .pc file. I would put it under contrib/; it is controversial, but some would like to

[gomp4] acc dealloc map

2014-10-20 Thread Cesar Philippidis
All of the various OpenACC memory maps are now fully supported in GCC. This patch removes an obsolete sorry message complaining about DEALLOCATE maps not being implemented. I've committed this to gomp-4_0-branch. Cesar 2014-10-20 Cesar Philippidis ce...@codesourcery.com gcc/ * gimplify.c

Re: [jit] Drop libgccjit.pc

2014-10-20 Thread David Malcolm
On Mon, 2014-10-20 at 22:11 +0200, Basile Starynkevitch wrote: On Mon, 2014-10-20 at 13:54 -0400, David Malcolm wrote: Committed to branch dmalcolm/jit: pkg-config appears to be controversial, so don't provide a .pc file. I would put it under contrib/; it is controversial, but some

Re: [PATCH] Account for prologue spills in reg_pressure scheduling

2014-10-20 Thread Sebastian Pop
Hi Maxim, Maxim Kuvyrkov wrote: Thanks, benchmarking results are welcome! AArch64 doesn't use reg_pressure scheduling by default. Use -fsched-pressure --param=sched-pressure-algorithm=2 to enable same thing as on ARM. I would imagine C++ and Fortran floating-point code to be most

  1   2   >