Re: [PATCH] libierty: change _ANSIDECL_H to ANSIDECL_H

I’m sorry, I misdiagnosed the problem, so you don’t want to merge this patch. What was really happening was that I had a different version of binutils installed in /usr/local/ and that had an ansidecl.h of its own, which was being used by the gcc build instead of its own ansidecl.h So there is

Re: [PATCH] libierty: change _ANSIDECL_H to ANSIDECL_H

On Sun, Jul 19, 2015 at 9:22 PM, Lawrence D'Anna wrote: > > ansidecl.h is using _ANSIDECL_H as it's guard #ifdef. But names that start > with > underscore are reserved, and the latest version of Xcode seems to be > exercising > its prerogative to define _ANSIDECL_H. > > This patch changes _ANSI

Re: [wwwdocs] Broken links

On 04/09/2015 05:47 PM, Gerald Pfeifer wrote: > Thank you Phillipe! I just committed the readings.html portion > of your patch which fixes the long standing broken link. > > Apologies again for nobody picking up your fine patches > originally -- please do not hesitate to send more, I promise > w

Re: [RFC, PR66873] Use graphite for parloops

On 20/07/15 20:31, Sebastian Pop wrote: Tom de Vries wrote: So I wondered, why not always use the graphite dependency analysis in parloops. (Of course you could use -floop-parallelize-all, but that also changes the heuristic). So I wrote a patch for parloops to use graphite dependency analysis b

[PING 5] Re: [PATCH] warn for unsafe calls to __builtin_return_addres

Ping #5. On 07/13/2015 09:33 PM, Martin Sebor wrote: Still looking for a review of this small patch to help detect potentially unsafe calls to __builtin_{frame,return}_address (with an argument > 2) that tend to either return bogus values or lead to crashes at runtime. The problem the function

Re: [RFC, PR66873] Use graphite for parloops

On 20/07/15 20:22, Sebastian Pop wrote: Tom de Vries wrote: graphite dependence analysis is too slow to be enabled unconditionally. (read: hours in some simple cases - see bugzilla) Haha, "cool"! ;-) Maybe it is still reasonable to use graphite to analyze the code inside OpenACC kernels regi

[rl78] extend set1/clr1 operands

Minor tweak to allow more addressing modes. Committed. * config/rl78/rl78-real.md (andqi3_real): Expand operands for clr1. (iorqi3_real): Likewise for set1. Index: config/rl78/rl78-real.md === --- config/rl78/rl78-r

fix gomp offload routine unloading

I've posted versions of this patch before, but muddled up with other changes. Hopefully this version is sufficiently self-contained to be understandable. When a dynamic object is unregistered, we unload the offloaded routines it registed. We walk the splay tree mapping host-side functions and

Re: [Patch, fortran] PR 37131, inline matmul

Hi, I'm back from holiday, so I can finally reply. Am 13.07.2015 um 21:54 schrieb Thomas Schwinge: --- gcc/fortran/iresolve.c +++ gcc/fortran/iresolve.c @@ -2207,6 +2207,9 @@ gfc_resolve_fe_runtime_error (gfc_code *c) a->name = "%VAL"; c->resolved_sym = gfc_get_intrinsic_sub_symbol

[v3 PATCH] Implement N4279, Improved insertion interface for unique-key maps.

Tested on Linux-PPC64. 2015-07-21 Ville Voutilainen Implement N4279, Improved insertion interface for unique-key maps. * include/bits/stl_map.h (try_emplace, insert_or_assign): New. * include/bits/stl_tree.h (_M_get_insert_unique_pos, _M_get_insert_equal_pos, _M_get_insert_hint_

Re: [PATCH, i386, PR driver/66737] Don't pass '-z bndplt' to linker for 32bit target

On Mon, Jul 20, 2015 at 1:57 PM, H.J. Lu wrote: > On Mon, Jul 20, 2015 at 5:42 AM, Ilya Enkovich wrote: >> Hi, >> >> This patch adds a target filter for '-z bndplt' linker option. Bootstrapped >> and regtested for x86_64-unknown-linux-gnu. MPX tests at lto.exp are not >> marked as unsupported

RE: [PATCH, MIPS] Scheduling for M51xx core family

Robert Suchanek writes. > 2015-07-16 Prachi Godbole > > gcc/ > > * config/mips/m5100.md: New file. > * config/mips/mips-cpus.def (m5100, m5101): Define. > * config/mips/mips-tables.opt: Regenerate. > * config/mips/mips.c (mips_rtx_cost_data): Add costs for m5100. >

Re: [PATCH, i386, PR driver/66737] Don't pass '-z bndplt' to linker for 32bit target

On Mon, Jul 20, 2015 at 5:42 AM, Ilya Enkovich wrote: > Hi, > > This patch adds a target filter for '-z bndplt' linker option. Bootstrapped > and regtested for x86_64-unknown-linux-gnu. MPX tests at lto.exp are not > marked as unsupported for 32bit any more. Going to commit it to trunk in a

rx: remove some asserts

Nick, There is no need to assert these just to say "not supported" and gcc may rarely generate addresses from valid code which trigger these asserts. Ok? Index: gcc/config/rx/rx.c === --- gcc/config/rx/rx.c (revision 225533) +++ g

[PATCH] remove tree-browser

Regstrapped on amd64-linux. Ok for trunk? Thanks, Sebastian 2015-07-20 Sebastian Pop * Makefile.in: Remove use of TREEBROWSER. * config.in: Regenerated. * configure: Regenerated. * configure.ac: Remove definition of TREEBROWSER. * t

Re: [C/C++ PATCH] Implement -Wshift-overflow (PR c++/55095) (take 3)

This seems to have caused a number of new failures in the PPC testsuite for vmx/unpack. - David

RE: [PATCH] 2015-07-14 Benedikt Huber Philipp Tomsich

Or rather, have an additional tuning/selection flag that a core may choose to make -mrecip the default with -ffast-math. Cheers, -- Evandro Menezes Austin, TX > -Original Message- > From: Andrew Pinski [mailto:pins...@gmail.com] > Sent: Saturday, July 18,

Re: [PATCH][C++] Fix PR65091

Hi, On 07/15/2015 04:12 PM, Jason Merrill wrote: On 07/14/2015 07:29 PM, Paolo Carlini wrote: I see. At the time I had this second try: https://gcc.gnu.org/ml/gcc-patches/2015-06/msg01232.html which has the action in cp_parser_unqualified_id but unfortunately is more complex. Certainly w

Re: revised and updated new-if-converter patch… [PATCH] fix PR46029: reimplement if conversion of loads and stores

[Alan wrote:] > Would having a testsuite predicate for the target supporting gathered loads > let you run this test on those architectures? I'd expect one to be useful > in a few other places too, in time if it doesn't exist already... Thanks, but I don`t think that would fully fix all the regres

RE: [PATCH] 2015-07-14 Benedikt Huber Philipp Tomsich

Hi, Benedikt. Please, see below. -- Evandro Menezes Austin, TX > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On > Behalf Of Benedikt Huber > Sent: Friday, July 17, 2015 10:44 > To: gcc-patches@gcc.gnu.org

Re: Tests for libgomp based on OpenMP Examples 4.0.2.

On Tue, Jul 14, 2015 at 11:04 AM, Jakub Jelinek wrote: > On Tue, Jul 14, 2015 at 08:40:50PM +0300, Maxim Blumental wrote: >> The patch replaces all FP comparisons with inequalities and epsilons >> in those tests for libgomp. > libgomp.fortran/examples-4/simd-7.f90 takes a very long time to run o

[PATCH, i386]: Fix asm flag output to DImode for 32bit targets

2015-07-20 Uros Bizjak * config/i386/i386.c (ix86_md_asm_adjust): Handle DImode dest_mode for !TARGET_64BIT. testsuite/ChangeLog: 2015-07-20 Uros Bizjak * gcc.target/i386/asm-flag-5.c (f_ll): New. Bootstrapped and regression tested on x86_64-linux-gnu {,m32}. Committed to ma

Re: [gomp4.1] Initial support for some OpenMP 4.1 construct parsing

On Fri, Jul 17, 2015 at 18:43:06 +0200, Jakub Jelinek wrote: > On Fri, Jul 17, 2015 at 07:31:36PM +0300, Ilya Verbin wrote: > > One big question is who will maintain the list of scheduled job, its > > dependencies, etc. - libgomp or each target plugin? > > > > > > OpenACC has async queues: > > #p

Re: [RFC, PR66873] Use graphite for parloops

Tom de Vries wrote: > So I wondered, why not always use the graphite dependency analysis > in parloops. (Of course you could use -floop-parallelize-all, but > that also changes the heuristic). So I wrote a patch for parloops to > use graphite dependency analysis by default (so without > -floop-para

Re: [RFC, PR66873] Use graphite for parloops

Tom de Vries wrote: > >>>graphite dependence analysis is too slow to be enabled unconditionally. > >>>(read: hours in some simple cases - see bugzilla) > >> > >>Haha, "cool"! ;-) > >> > >>Maybe it is still reasonable to use graphite to analyze the code inside > >>OpenACC kernels regions -- maybe s

Re: [gomp4.1] Initial support for some OpenMP 4.1 construct parsing

Hi! And here is untested incremental libgomp side of the proposed GOMP_MAP_FIRSTPRIVATE_POINTER. I'll probably tweak it so that if GOMP_MAP_FIRSTPRIVATE is followed by one or more GOMP_MAP_FIRSTPRIVATE_POINTER where the pointers fall into the extents of the firstprivate object, it will make the wh

Re: Refactor openacc wait routine

On 07/19/15 08:37, Nathan Sidwell wrote: this trunk patch refactors libgomp's goacc_wait, which is used for two different purposes. 1) when openacc pragmas specify a (non-zero) waits. 2) when the wait pragma itself specifies a zero number of waits. this leads to #2 calling goacc_wait with num_

Re: revised and updated new-if-converter patch… [PATCH] fix PR46029: reimplement if conversion of loads and stores

Abe wrote: diff --git a/gcc/testsuite/gcc.dg/vect/if-cvt-stores-vect-ifcvt-18.c b/gcc/testsuite/gcc.dg/vect/if-cvt-stores-vect-ifcvt-18.c index 71f2db3..2b159d7 100644 --- a/gcc/testsuite/gcc.dg/vect/if-cvt-stores-vect-ifcvt-18.c +++ b/gcc/testsuite/gcc.dg/vect/if-cvt-stores-vect-ifcvt-18.c @@

Go patch committed: Don't simply discard blank labels

The Go frontend was simply discarding blank labels, which caused it to accept invalid programs, such as: func F() { for {} _: } This patch from Chris Manghane fixes the problem. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian Index: gcc/config/arm/

Re: [PATCH][AArch64][4/14] Create TARGET_FIX_ERR_A53_835769 and use that instead of aarch64_fix_a53_err835769

On Thu, Jul 16, 2015 at 04:48:43PM +0100, Kyrill Tkachov wrote: > Sorry, had sent out the wrong version. > This is the right patch. > > Thanks, > Kyrill > > On 16/07/15 16:20, Kyrill Tkachov wrote: > > Hi all, > > > > This patch transforms the Cortex-A53 erratum 835769 workaround checks into > >

Go patch committed: Remove unnecessary tests

This patch by Chris Manghane removes some unnecessary tests from the Go frontend. These checks are now handled by the GCC-specific code in gcc/go/go-gcc.cc. Bootstrapped and ran testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian Index: gcc/go/gofrontend/MERGE

Re: [PATCH][AArch64][3/14] Refactor option override code

On Thu, Jul 16, 2015 at 04:20:37PM +0100, Kyrill Tkachov wrote: > Hi all, > > This one is more meaty than the previous ones. It buffs up the parsing > functions for > the mcpu, march, mtune options, decouples them and makes them return an enum > describing > the errors that may occur. This will

Re: [PATCH] Refactor graphite-isl-ast-to-gimple.c

On 07/20/2015 06:24 PM, Aditya Kumar wrote: From: Aditya Kumar Refactor graphite-isl-ast-to-gimple.c: Refactor so that each function can access 'region'. This will help maintain a parameter rename_map within a region. No functional change intended. This patch will be followed by another set of

Re: [PATCH 3/3] lt0_disi

On Mon, Jul 20, 2015 at 12:04 PM, Segher Boessenkool wrote: > After Kyrill's patch (r225996) (changing combine to do simplification > before doing some transformations) a shortcoming of the rs6000 backend > becomes obvious: we have no patterns to handle > > (set (reg:DI) (lt:DI (reg:SI) (c

[PATCH] Refactor graphite-isl-ast-to-gimple.c

From: Aditya Kumar Refactor graphite-isl-ast-to-gimple.c: Refactor so that each function can access 'region'. This will help maintain a parameter rename_map within a region. No functional change intended. This patch will be followed by another set of patches where translate_isl_ast_to_gimple::reg

Re: Fix PTX loaded module data

On 07/19/15 17:04, Nathan Sidwell wrote: Hi, this patch fixes the ptx plugin's management of offloaded functions. Currently it has a single global list of functions. But it should track this information per ptx device instance. I've moved the linked list into the per-device data. We were als

Re: [PATCH 2/3] Fix shift amount (GPR->SI)

On Mon, Jul 20, 2015 at 12:04 PM, Segher Boessenkool wrote: > This changes the shift amount to always be SI (as it should be), not GPR. > It doesn't matter for constant shifts, but there are some variable shifts > as well, and consistency is good. > > No changelog, I'll fold it into the previous b

Re: [PATCH 1/3] Doc fixes for rot

On Mon, Jul 20, 2015 at 12:04 PM, Segher Boessenkool wrote: > This removes the removed constraints from the documentation. It also > removes "q" (for MQ) which hasn't existed for a while. > > > 2015-07-20 Segher Boessenkool > > * config/rs6000/constraints.md: Update "available letters"

Re: [PATCH] rs6000: Revamp rotate-and-mask and insert

On Sun, Jul 12, 2015 at 1:08 PM, Segher Boessenkool wrote: > This rewrites all the rotate-and-mask and insert patterns. > > The goals are to have simpler, shorter, less error-prone code (with much > fewer machine description patterns), as well as to get better optimised > machine code. > > All "ma

Re: [gomp4.1] Initial support for some OpenMP 4.1 construct parsing

On Fri, Jul 17, 2015 at 06:43:06PM +0200, Jakub Jelinek wrote: > > BTW, do you plan to remove GOMP_MAP_POINTER mappings from array sections? > > The enter/exit patch for libgomp depends on this change. > > My current plan (for Monday and onwards) is to first implement firstprivate > on target cons

[PATCH 3/3] lt0_disi

After Kyrill's patch (r225996) (changing combine to do simplification before doing some transformations) a shortcoming of the rs6000 backend becomes obvious: we have no patterns to handle (set (reg:DI) (lt:DI (reg:SI) (const_int 0))) although we can do that with a single rotate-and-mask i

[PATCH 2/3] Fix shift amount (GPR->SI)

This changes the shift amount to always be SI (as it should be), not GPR. It doesn't matter for constant shifts, but there are some variable shifts as well, and consistency is good. No changelog, I'll fold it into the previous big patch, if approved. Segher --- gcc/config/rs6000/rs6000.md | 24

[PATCH 1/3] Doc fixes for rot

This removes the removed constraints from the documentation. It also removes "q" (for MQ) which hasn't existed for a while. 2015-07-20 Segher Boessenkool * config/rs6000/constraints.md: Update "available letters" comment. * doc/md.texi (Machine Constraints): Remove q, S, T, t

Re: [PING][PATCH, 1/2] Merge rewrite_virtuals_into_loop_closed_ssa from gomp4 branch

On 09/07/15 13:04, Richard Biener wrote: On Thu, 9 Jul 2015, Tom de Vries wrote: On 07/07/15 17:58, Tom de Vries wrote: If you can handle one exit edge I also can't see the difficulty in handling all exit edges. Agreed, that doesn't look to complicated. I could call rewrite_virtuals_into_lo

[PATCH 0/3] rs6000: Some updates for rotate etc.

Hi all, Two updates for the rotate revamp, and a third patch that won't apply without it. I'll fold the first two together with the big patch, if approved. Everything bootstrapped and tested on powerpc64-linux, as usual; no regressions. Segher Segher Boessenkool (3): Doc fixes for rot Fi

Re: [PATCH 3/4] S390 -march=native related fixes

On Sat, Jul 18, 2015 at 01:09:26AM +0200, Ulrich Weigand wrote: > Dominik Vogt wrote: > > > + opt_esa_zarch = (has_highgprs) ? " -mzarch" : " -mesa"; > > This will force -mesa on old machines *even in -m64 mode*, > which is wrong and will cause compilation to fail. > > > > -/* Defaulting r

Re: [PATCH] Update decls in genemit.c

On Mon, Jul 20, 2015 at 04:10:33PM +0200, Marek Polacek wrote: > Since r225883, genemit.c's gen_insn/gen_expand/gen_split have a parameter > of type md_rtx_info *, not rtx, but the function prototypes weren't updated, > which results into 'declared but never defined' warnings. The following > triv

RE: [PATCH] [aarch64] Implemented reciprocal square root (rsqrt) estimation in -ffast-math

Hi, Venkat. Since x^1/2 = x * x^-1/2, the Newton series can also be used for the regular square root with an extra multiplication, as it is done in x86. That's what I was trying to estimate below. Cheers, -- Evandro Menezes Austin, TX > -Original Message

Re: [gomp] Move openacc vector& worker single handling to RTL

On 07/20/15 09:01, Nathan Sidwell wrote: On 07/18/15 11:37, Thomas Schwinge wrote: Hi Nathan! For OpenACC nvptx offloading, there must still be something wrong; here's a count of the (non-deterministic!) regressions of ten runs of the libgomp testsuite. As private-vars-loop-worker-5.c fails

Re: [PATCH] Simple optimization for MASK_STORE.

Hi Jeff! Thanks for your details comments. You asked: How are conditionals on vectors usually handled? You should try to mimick that and report, with detail, why that's not working. In current implementation of vectorization pass all comparisons are handled through COND_EXPR, i.e. vect-pattern

[PATCH, rtl-opt, i386]: Backport fix for PR 58066, __tls_get_addr is called with misaligned stack on x86-64

Attached patch backports fixes for PR 58066 to release branches. 2015-07-XX Uros Bizjak Backport from mainline: 2015-07-17 Uros Bizjak PR rtl-optimization/66891 * calls.c (expand_call): Wrap precompute_register_parameters with NO_DEFER_POP/OK_DEFER_POP to prevent deferr

Re: [gomp4.1] Pretty print '+' for sink positive offsets

On Mon, Jul 20, 2015 at 07:47:25AM -0700, Aldy Hernandez wrote: > Positive numbers get no '+' by default, so the dumps look like this: > > #pragma omp ordered depend(sink:i1) > > The attached patch fixes this oversight to print: > > #pragma omp ordered depend(sink

[gomp4.1] Pretty print '+' for sink positive offsets

Positive numbers get no '+' by default, so the dumps look like this: #pragma omp ordered depend(sink:i1) The attached patch fixes this oversight to print: #pragma omp ordered depend(sink:i+1) OK for branch? commit fc23dda3b860931ca72bf00236e3f929b48c751e Author:

[COMMITTED][AArch64] Restrict got_mem_hoist_1.c with small memory model

Test testcase fail on tiny model, it's because single "ldr" generated to address got entry of "bar", while the early RTL PRE pass has done the IV hoisting, so loop IV will do nothing, thus we should restrict this testcase under small model. For large model, -fpic is unsupported on AArch64, while

Re: [PING] Re: [PATCH] New configure option to default enable Smart Stack Protection

måndag 13 juli 2015 15.20.40 skrev Magnus Granberg: > söndag 05 juli 2015 23.59.32 skrev Magnus Granberg: > > Changlogs > > /gcc > > 2015-07-05 Magnus Granberg > > > > * common.opt (fstack-protector): Initialize to -1. > > (fstack-protector-all): Likewise. > > (fstack-

Re: Fix two more memory leaks in threader

On Mon, Jul 20, 2015 at 03:19:06PM +0100, James Greenhalgh wrote: > On Wed, May 20, 2015 at 05:36:25PM +0100, Jeff Law wrote: > > > > These fix the remaining leaks in the threader that I'm aware of. We > > failed to properly clean-up when we had to cancel certain jump threading > > opportunitie

[PATCH][match.pd] PR middle-end/66915 Restrict A - B -> A + (-B) to non-fixed-point types

Hi all, This patch fixes the PR in question which is a miscompilation of gcc.dg/fixed-point/unary.c on arm. It just restricts the A - B -> A + (-B) transformation when the type is fixed-point. This fixes the testcase for me. Is this the right approach? Bootstrap and test on arm and x86 runnin

Re: Fix two more memory leaks in threader

On Wed, May 20, 2015 at 05:36:25PM +0100, Jeff Law wrote: > > These fix the remaining leaks in the threader that I'm aware of. We > failed to properly clean-up when we had to cancel certain jump threading > opportunities. So thankfully this wasn't a big leak. Hi Jeff, I don't have a reduced

[PATCH] Update decls in genemit.c

Since r225883, genemit.c's gen_insn/gen_expand/gen_split have a parameter of type md_rtx_info *, not rtx, but the function prototypes weren't updated, which results into 'declared but never defined' warnings. The following trivial patch updates the declarations. Bootstrapped/regtested on x86_64-l

Re: [C/C++ PATCH] Implement -Wshift-overflow (PR c++/55095) (take 3)

On Fri, Jul 17, 2015 at 03:51:33PM -0600, Jeff Law wrote: > I'll approve the C++ parts given how simple they are :-) Thanks, I've committed the patch now after another regtest/bootstrap. Marek

Re: [gomp4] New nvptx pattern and internal builtin

On 07/20/2015 03:19 PM, Nathan Sidwell wrote: I was looking at adding another target builtin, and found this code rather convoluted. It seemed to have been cloned from somewhere more complicated -- for instance, nvptx_expand_binop_builtin's comment discusses a MACFLAG argument, which is nowhere

Re: [gomp4] New nvptx pattern and internal builtin

On 07/17/15 11:37, Bernd Schmidt wrote: I've made this change at the request of Cesar who says it's needed for his reductions work. It makes a new instruction to represent shfl.down, a thread communication instruction, and some builtin functions for internal use to access it. I was looking at a

Re: [PATCH, PR66846] Mark inner loop for fixup in parloops

On 16/07/15 12:15, Richard Biener wrote: On Thu, Jul 16, 2015 at 11:39 AM, Tom de Vries wrote: On 16/07/15 10:44, Richard Biener wrote: On Wed, Jul 15, 2015 at 9:36 PM, Tom de Vries wrote: Hi, I. In openmp expansion of loops, we do some effort to try to create matching loops in the loop

Re: [gomp] Move openacc vector& worker single handling to RTL

On 07/18/15 11:37, Thomas Schwinge wrote: Hi Nathan! For OpenACC nvptx offloading, there must still be something wrong; here's a count of the (non-deterministic!) regressions of ten runs of the libgomp testsuite. As private-vars-loop-worker-5.c fails most often, it probably makes sense to loo

Re: [PATCH][combine][1/2] Try to simplify before substituting

On 18/07/15 17:02, Segher Boessenkool wrote: On Fri, Jul 17, 2015 at 02:47:34PM -0600, Jeff Law wrote: I mean move the whole "if (BINARY_P ..." block to after the existing simplify calls, to just before the "First see if we can apply" comment, and not do a new simplify_rtx call at all. Does th

[PATCH] Track indirect calls for call site information in debug info.

Hello, On PowerPC targets with -mlongcall, most subprogram calls are turned into indirect calls: the call target is read from a register even though it is compile-time known. This makes it difficult for machine code static analysis engines to recover the callee information. The attached patch

[PATCH, i386, PR driver/66737] Don't pass '-z bndplt' to linker for 32bit target

Hi, This patch adds a target filter for '-z bndplt' linker option. Bootstrapped and regtested for x86_64-unknown-linux-gnu. MPX tests at lto.exp are not marked as unsupported for 32bit any more. Going to commit it to trunk in a few days if no obections appear. Thanks, Ilya -- gcc/ 2015-07-

Re: [gomp4] remove kernel-specific launch

On 07/20/15 05:54, Tom de Vries wrote: Turning it into an internal fn will make it harder to convert a GOACC_kernels_internal call into a GOACC_parallel call, which we're doing here I wondered ...

Re: [PATCH, PR ipa/66566] Fix ICE in early_inliner: internal compiler error: in operator[]

Ping 2015-07-13 11:47 GMT+03:00 Ilya Enkovich : > Ping > > 2015-06-18 12:54 GMT+03:00 Ilya Enkovich : >> Hi, >> >> In early_inliner we do recompute inline summaries for edges after >> optimize_inline_calls, but check this summary exists in case new edges >> appear. But then it calls inline_upda

Re: [CHKP, GCC 5] Port a set of stability chkp patches to gcc-5-branch

Ping 2015-06-19 17:10 GMT+03:00 Ilya Enkovich : > Hi, > > There was a set of stability fixes (mostly different ICEs) for Pointer Bounds > Checker done in GCC 6. But only few of them were approved to be ported to > GCC 5. Will it be OK to port other chkp specific stability fixes to GCC 5? > H

Re: [gomp4] remove kernel-specific launch

On 19/07/15 23:08, Nathan Sidwell wrote: On 07/19/15 16:30, Thomas Schwinge wrote: gcc/tree-parloops.c:/* Remove GOACC_kernels. */ libgomp/libgomp.map:GOACC_kernels; libgomp/libgomp_g.h:extern void GOACC_kernels (int, void (*) (void *), size_t, I fixed all byt the parl

Re: [PATCH][RTL-ifcvt] Make non-conditional execution if-conversion more aggressive

Ping. https://gcc.gnu.org/ml/gcc-patches/2015-07/msg01047.html The go testsuite passes for me on x86_64-unknown-linux-gnu for me. A third data point on testing would be appreciated... Thanks, Kyrill On 13/07/15 15:03, Kyrill Tkachov wrote: Hi Bernhard, On 13/07/15 10:45, Kyrill Tkachov wrote

Re: [GSoC] Patches for shared_ptr array and polymorphic_allocator

On 18/07/15 00:02 -0700, Tim Shen wrote: On Fri, Jul 17, 2015 at 7:16 PM, Fan You wrote: Hi, According to Here is my implementation of [8.2] Extend shared_ptr to support arrays Please don't resend the sha

Re: [PATCH][AArch64] Use cinc for if_then_else of plus-immediates

On 19/07/15 07:40, Oleg Endo wrote: On 19 Jul 2015, at 12:13, Andrew Pinski wrote: On Thu, Jul 16, 2015 at 8:33 AM, Kyrill Tkachov wrote: Hi all, This patch improves codegen for expressions of the form: (x ? y + c1 : y + c2) when |c1 - c2| == 1 It matches the if_then_else of the two plus-

RE: [PATCH] [aarch64] Implemented reciprocal square root (rsqrt) estimation in -ffast-math

Hi, I missed your email and noticed it this week. What does column 2 tests? Are you trying to implement square roots using reciprocal estimate and step? But reciprocal square root using reciprocal estimate and (2 for fp 3 for dp) step seems to be better that using fdiv and fsqrt in your

Re: [PATCH 1/3] tree-ssa-tail-merge: add IPA ICF infrastructure.

On 07/16/2015 01:03 PM, Martin Liška wrote: > On 07/09/2015 06:24 PM, Jeff Law wrote: >> On 07/09/2015 07:56 AM, mliska wrote: >>> gcc/ChangeLog: >>> >>> 2015-07-09 Martin Liska >>> >>> * dbgcnt.def: Add new debug counter. >>> * ipa-icf-gimple.c (func_checker::compare_ssa_name): Add flag

Re: [PATCH 1/3] tree-ssa-tail-merge: add IPA ICF infrastructure.

On 07/16/2015 01:03 PM, Martin Liška wrote: > On 07/09/2015 06:24 PM, Jeff Law wrote: >> On 07/09/2015 07:56 AM, mliska wrote: >>> gcc/ChangeLog: >>> >>> 2015-07-09 Martin Liska >>> >>> * dbgcnt.def: Add new debug counter. >>> * ipa-icf-gimple.c (func_checker::compare_ssa_name): Add flag

[PATCH] PR debug/53927: fix value for DW_AT_static_link

Hello, This patch fixes the static link description in DWARF to comply with the specification. In order to do so, it appends a field to all FRAME objects to hold the frame base address (DW_AT_frame_base) so that the nested subrograms can directly reference this field. See

Re: [PATCH] Fix partial template specialization syntax in wide-int.h

On 07/17/2015 07:46 PM, Mike Stump wrote: > On Jul 17, 2015, at 2:28 AM, Mikhail Maltsev wrote: >> The following code (reduced from wide-int.h) is rejected by Intel C++ >> Compiler (EDG-based): > > So, could you test this with the top of the tree compiler and file a bug > report against g++ for i