Re: [PATCH][RTL-ifcvt] Improve conditional select ops on immediates

2015-07-30 Thread Kyrill Tkachov
Hi Jeff, On 29/07/15 23:38, Jeff Law wrote: On 07/29/2015 07:49 AM, Kyrill Tkachov wrote: Hi all, This patch improves RTL if-conversion on sequences that perform a conditional select on integer constants. Most of the smart logic to do that already exists in the noce_try_store_flag_constants

libgomp: plugin for non-shared memory host execution

2015-07-30 Thread Thomas Schwinge
Hi! While I still think that a generic libgomp plugin for non-shared memory host execution is a good idea conceptually, for the following reasons: On Wed, 19 Feb 2014 17:10:15 +0100, I wrote: On Thu, 12 Dec 2013 12:31:40 +0100, I wrote: On Fri, 8 Nov 2013 16:40:00 +0100, Jakub Jelinek

[AArch64] Remove unused VRL2/3/4 iterator values (was: Re: [PATCH 8/15][AArch64] Add support for float14x{4,8}_t vectors/builtins)

2015-07-30 Thread Alan Lawrence
James Greenhalgh wrote: On Tue, Jul 28, 2015 at 12:25:40PM +0100, Alan Lawrence wrote: I'd have preferred the unrelated changes here as separate patches. If you pull them out, they are OK to commit independent of this patch. Done (r226352 and r226353). Ah ok, I see what is going on here...

Re: [PATCH 0/9] start converting POINTER_SIZE to a hook

2015-07-30 Thread David Edelsohn
On Wed, Jul 29, 2015 at 11:16:40AM +0100, Richard Earnshaw wrote: I'm getting a bit worried about the potential performance impact from all these indirect function call hooks. This is a good example of when it's probably somewhat unnecessary. I doubt that the compiler could function

Re: [PATCH] PR fortran/66942 -- avoid referencing a NULL C++ thing

2015-07-30 Thread Mikael Morin
Le 29/07/2015 18:45, Steve Kargl a écrit : On Wed, Jul 29, 2015 at 02:04:12PM +0200, Richard Biener wrote: On Wed, Jul 29, 2015 at 1:59 PM, Mikael Morin mikael.mo...@sfr.fr wrote: Le 29/07/2015 13:22, Richard Biener a écrit : On Wed, Jul 29, 2015 at 11:34 AM, Mikael Morin mikael.mo...@sfr.fr

Re: [PATCH] Add warnings about GENERIC code-gen deficiencies in genmatch

2015-07-30 Thread Michael Matz
Hi, On Thu, 30 Jul 2015, Richard Biener wrote: @@ -4174,11 +4267,13 @@ main (int argc, char **argv) else if (strcmp (argv[i], --generic) == 0) gimple = false; else if (strcmp (argv[i], -v) == 0) - verbose = true; + verbose = 1; If you don't want to start using

Re: [AArch64] Remove unused VRL2/3/4 iterator values (was: Re: [PATCH 8/15][AArch64] Add support for float14x{4,8}_t vectors/builtins)

2015-07-30 Thread James Greenhalgh
On Thu, Jul 30, 2015 at 12:47:20PM +0100, Alan Lawrence wrote: James Greenhalgh wrote: On Tue, Jul 28, 2015 at 12:25:40PM +0100, Alan Lawrence wrote: I'd have preferred the unrelated changes here as separate patches. If you pull them out, they are OK to commit independent of this patch.

[patch] Move symbol_visibility to coretypes.h

2015-07-30 Thread Andrew MacLeod
Pretty simple. Both flag-types.h and tree-core.h check to see if the other header has already defined this enumerated type, and defined it if not. Its an unnecessary conditional compilation and dual maintenance of this small enum. This patch puts it in coretypes.h and makes it always

Re: [PATCH] 2015-07-29 Benedikt Huber benedikt.hu...@theobroma-systems.com Philipp Tomsich philipp.toms...@theobroma-systems.com

2015-07-30 Thread Kyrill Tkachov
Hi Benedikt, On 29/07/15 18:47, Benedikt Huber wrote: * config/aarch64/aarch64-builtins.c: Builtins for rsqrt and rsqrtf. * config/aarch64/aarch64-opts.h: -mrecip has a default value depending on the core. * config/aarch64/aarch64-protos.h: Declare.

Re: libgomp: plugin for non-shared memory host execution

2015-07-30 Thread Jakub Jelinek
On Thu, Jul 30, 2015 at 01:47:37PM +0200, Thomas Schwinge wrote: Here is such a libgomp plugin plus the infrastructure for initial support of non-shared memory host execution. [...] ... the libgomp plugin as it is currently implemented fails to adequately provide such functionality:

Re: C++ delayed folding branch review

2015-07-30 Thread Kai Tietz
2015-07-30 0:56 GMT+02:00 Kai Tietz ktiet...@googlemail.com: 2015-07-29 19:48 GMT+02:00 Jason Merrill ja...@redhat.com: On 07/28/2015 04:10 PM, Kai Tietz wrote: The change to adjust_temp_type seems to be no more necessary (just doing tests on it). Yes, committed it. @@ -3391,8 +3431,23 @@

[PATCH][2/2] Remove GENERIC comparison folding from fold_stmt

2015-07-30 Thread Richard Biener
This removes the assign RHS part. Likewise tested with an assert. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Just spotted case GIMPLE_TERNARY_RHS: /* Try to fold a conditional expression. */ if (gimple_assign_rhs_code (stmt) == COND_EXPR) { so

[v3 patch] Fix FAIL in experimental/filesystem/operations/current_path.cc

2015-07-30 Thread Jonathan Wakely
This test is failing on Darwin, it turns out to be because TMPDIR is set to a symlink, so after changing to that directory current_path() returns a different path. We should canonicalize the paths before comparing them. Tested x86_64-linux, committed to trunk. commit

Remove support for atomic_clear pattern

2015-07-30 Thread Richard Sandiford
expand_builtin_atomic_clear has code to handle an atomic_clear instruction. However, no target defines such a pattern, and the code wouldn't compile on any target that tried to define it, since the generator is passed the memory model enum directly rather than via an rtx wrapper. It seems better

Re: Remove support for atomic_clear pattern

2015-07-30 Thread Richard Biener
On Thu, Jul 30, 2015 at 3:02 PM, Richard Sandiford richard.sandif...@arm.com wrote: expand_builtin_atomic_clear has code to handle an atomic_clear instruction. However, no target defines such a pattern, and the code wouldn't compile on any target that tried to define it, since the generator is

Re: [PATCH] Fix PR67052

2015-07-30 Thread Richard Biener
On Thu, 30 Jul 2015, Richard Biener wrote: The following patch avoids folding NaN 0 to false if -ftrapping-math is in effect. Bootstrap regtest pending on x86_64-unknown-linux-gnu. FAIL: gcc.dg/fold-abs-4.c scan-tree-dump-times gimple ABS 0 FAIL: gcc.dg/pr15784-1.c scan-tree-dump-times

Re: [gomp4] Redesign oacc_parallel launch API

2015-07-30 Thread Nathan Sidwell
On 07/29/15 17:45, Nathan Sidwell wrote: On 07/29/15 08:24, Nathan Sidwell wrote: On 07/29/15 05:22, Thomas Schwinge wrote: Likewise for the other torture testing flags. Investigating ... (I've seen those failures be intermittent) Interestingly the fails go away with an unoptimized

[PATCH][ARM][libgcc] Gate crtfastmath.c contents on __ARM_32BIT_STATE

2015-07-30 Thread Kyrill Tkachov
Hi all, I tried bootstrapping on Thumb1 recently and hit an error while building libgcc and in particular config/arm/crtfastmath.c. This file generates vmrs instructions which are not allowed in Thumb1, causing an assembler error. The fix for that is to gate the file not only on !__SOFTFP__

Re: [AArch64] Remove unused VRL2/3/4 iterator values

2015-07-30 Thread Alan Lawrence
James Greenhalgh wrote: (define_mode_attr VRL2 [(V8QI V32QI) (V4HI V16HI) (V2SI V8SI) (V2SF V8SF) - (DI V4DI) (DF V4DF) - (V16QI V32QI) (V8HI V16HI) - (V4SI V8SI) (V4SF V8SF) -

Re: [PATCH 1/2] Allow REG_EQUAL for ZERO_EXTRACT

2015-07-30 Thread Andreas Schwab
Kugan kugan.vivekanandara...@linaro.org writes: * cse.c (cse_insn): Restoring old behaviour for src_eqv when dest and value in the REG_EQUAL are same and dest is STRICT_LOW_PART. This fixes the regression and doesn't introduce any new one. Andreas. -- Andreas Schwab,

PR c/c++/diagnostics/66098 Take -Werror into account when deciding what was the command-line status

2015-07-30 Thread Manuel López-Ibáñez
When I fixed PR59304, I forgot that a command-line warning can be also an error if -Werror was enabled. This introduced a regression since anything enabled in the command-line together with -Werror would get initially classified as a warning when reaching the first #pragma GCC diagnostic, and this

Re: [gomp4.1] Support #pragma omp target {enter,exit} data

2015-07-30 Thread Jakub Jelinek
On Thu, Jul 30, 2015 at 05:40:15PM +0300, Ilya Verbin wrote: Sure, but it's not possible to fully test data mapping without non-shared address space. I've created new check_effective_target, ok for gomp-4_1-branch? * testsuite/lib/libgomp.exp

Re: [PATCH] fold builtin_tolower, builtin_toupper

2015-07-30 Thread Joseph Myers
On Thu, 9 Jul 2015, Richard Biener wrote: On Thu, 9 Jul 2015, Bernhard Reutner-Fischer wrote: gcc/ChangeLog 2015-07-09 Bernhard Reutner-Fischer al...@gcc.gnu.org * builtins.c (fold_builtin_tolower, fold_builtin_toupper): New static functions. (fold_builtin_1):

Re: [PATCH][ARM][libgcc] Gate crtfastmath.c contents on __ARM_32BIT_STATE

2015-07-30 Thread Richard Earnshaw
On 30/07/15 15:45, Kyrill Tkachov wrote: Hi all, I tried bootstrapping on Thumb1 recently and hit an error while building libgcc and in particular config/arm/crtfastmath.c. This file generates vmrs instructions which are not allowed in Thumb1, causing an assembler error. The fix for

Re: [AArch64] Remove unused VRL2/3/4 iterator values

2015-07-30 Thread James Greenhalgh
On Thu, Jul 30, 2015 at 04:06:22PM +0100, Alan Lawrence wrote: James Greenhalgh wrote: (define_mode_attr VRL2 [(V8QI V32QI) (V4HI V16HI) (V2SI V8SI) (V2SF V8SF) - (DI V4DI) (DF V4DF) - (V16QI V32QI) (V8HI V16HI) -

Re: [gomp4.1] Support #pragma omp target {enter,exit} data

2015-07-30 Thread Ilya Verbin
On Thu, Jul 30, 2015 at 10:12:59 +0200, Jakub Jelinek wrote: This test will fail on HSA, you don't assume just that it doesn't fallback to host, but also non-shared address space. I think it would be better to start with some check for non-shared address space, like: /* This test relies on

Re: [gomp4] Redesign oacc_parallel launch API

2015-07-30 Thread Thomas Schwinge
Hi Nathan! On Wed, 29 Jul 2015 17:45:32 -0400, Nathan Sidwell nat...@acm.org wrote: On 07/29/15 08:24, Nathan Sidwell wrote: On 07/29/15 05:22, Thomas Schwinge wrote: Likewise for the other torture testing flags. Investigating ... (I've seen those failures be intermittent) (For me,

Re: C++ delayed folding branch review

2015-07-30 Thread Kai Tietz
2015-07-30 18:52 GMT+02:00 Jason Merrill ja...@redhat.com: On 07/29/2015 06:56 PM, Kai Tietz wrote: @@ -1430,6 +1438,8 @@ cxx_eval_call_expression (const constexpr_ctx *ctx, tree t, bool reduced_constant_expression_p (tree t) { + /* Make sure we remove useless initial

Re: [PATCH 0/9] start converting POINTER_SIZE to a hook

2015-07-30 Thread Richard Sandiford
David Edelsohn dje@gmail.com writes: On Wed, Jul 29, 2015 at 11:16:40AM +0100, Richard Earnshaw wrote: I'm getting a bit worried about the potential performance impact from all these indirect function call hooks. This is a good example of when it's probably somewhat unnecessary. I doubt

Re: [PATCH], PowerPC IEEE 128-bit patch #4c (revised)

2015-07-30 Thread Michael Meissner
Seger pointed out that the altivec_high_bit insn was not safe if it was called when it could not allocate pseudos. I have removed that insn in this patch. I will revise the patch. I had meant to tie it in with easy_altivec_constant originally, and I will do a patch with that change. Is this

Re: [PATCH 0/9] start converting POINTER_SIZE to a hook

2015-07-30 Thread Richard Sandiford
Trevor Saunders tbsau...@tbsaunde.org writes: On Wed, Jul 29, 2015 at 09:11:21AM +0100, Richard Sandiford wrote: Trevor Saunders tbsau...@tbsaunde.org writes: On Tue, Jul 28, 2015 at 09:24:17PM +0100, Richard Sandiford wrote: Trevor Saunders tbsau...@tbsaunde.org writes: On Mon, Jul 27,

[gomp] more ptx builtins

2015-07-30 Thread Nathan Sidwell
I've committed this to gomp4. It adds spinlock builtins, which will be necessary for complete reduction implementation. There is a single global and a single .shared lock variable (of course the latter is per-cta). We lazily emit these declarations, if locks are taken during compilation.

Re: [gomp4.1] Support #pragma omp target {enter,exit} data

2015-07-30 Thread Ilya Verbin
On Thu, Jul 30, 2015 at 10:12:59 +0200, Jakub Jelinek wrote: On Wed, Jul 29, 2015 at 10:06:52PM +0300, Ilya Verbin wrote: @@ -1241,6 +1245,62 @@ GOMP_target_update (int device, const void *unused, size_t mapnum, gomp_update (devicep, mapnum, hostaddrs, sizes, kinds, false); }

Re: Elimitate duplication of get_catalogs in different abi

2015-07-30 Thread François Dumont
It seems that this patch results in unresolved symbols. I am quite sure that the code is right but build system should be adapted. I noticed that *_cow.cc files are built with -fimplicit-templates. I try to apply the same with the old abi but I still experiment unresolved symbols. Any help is

Re: [gomp4.1] Support #pragma omp target {enter,exit} data

2015-07-30 Thread Jakub Jelinek
On Thu, Jul 30, 2015 at 10:44:33PM +0300, Ilya Verbin wrote: libgomp/ * libgomp.h (enum gomp_map_vars_kind): New. (gomp_map_vars): Change type of the argument from bool to enum gomp_map_vars_kind. * oacc-mem.c (acc_map_data, present_create_copy,

Re: [ARM] implement division using vrecpe/vrecps with -funsafe-math-optimizations

2015-07-30 Thread Prathamesh Kulkarni
On 29 July 2015 at 16:03, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Hi Prathamesh, This is probably not appropriate for -Os optimisation. And for speed optimisation I imagine it can vary a lot on the target the code is run. Do you have any benchmark results for this patch? Hi Kyrill,

[PATCH] Fix PR66870 ppc64le, ppc64 split stack

2015-07-30 Thread Lynn A. Boger
The attached patch fixes a problem identified in PR66870 with split stack on ppc64 and ppc64le. If flag_split_stack is true, but the function being processed has the no_split_stack attribute, then the split stack prologue should not be generated. Bootstrapped on ppc64le-linux and ppc64-linux,

Re: [Patch 0/4] PowerPC64 Linux split stack support

2015-07-30 Thread Ian Lance Taylor
On Thu, Jul 30, 2015 at 10:46 AM, Lynn A. Boger labo...@linux.vnet.ibm.com wrote: Can this patch be submitted to fix the ppc 32 bootstrap error? On 06/19/2015 07:58 AM, Andreas Schwab wrote: * go-lang.c (go_langhook_init_options_struct): Don't set x_flag_split_stack.

Re: C++ delayed folding branch review

2015-07-30 Thread Jason Merrill
On 07/30/2015 05:00 PM, Kai Tietz wrote: 2015-07-30 18:52 GMT+02:00 Jason Merrill ja...@redhat.com: On 07/29/2015 06:56 PM, Kai Tietz wrote: @@ -1430,6 +1438,8 @@ cxx_eval_call_expression (const constexpr_ctx *ctx, tree t, bool reduced_constant_expression_p (tree t) { + /*

Re: [Patch, MIPS] MIPS specific optimization for o32 ABI

2015-07-30 Thread Joseph Myers
New command-line options need documenting in invoke.texi. -- Joseph S. Myers jos...@codesourcery.com

[RFC, patch] New attribute to create target clones

2015-07-30 Thread Evgeny Stupachenko
Hi All, The patch enables new attribute 'ctarget', The attribute force compiler to create clones of a function with the attribute. For example: __attribute__((ctarget(avx,arch=slm,arch=core-avx2,default))) int foo () will create 3 clones of foo() optimized for corresponding targets and replace

Re: [RFC, patch] New attribute to create target clones

2015-07-30 Thread Evgeny Stupachenko
You can't apply both 'ctarget' and 'target' attributes to a function. The patch uses same target specific methods to create dispatcher and target clones. On Fri, Jul 31, 2015 at 1:30 AM, Andrew Pinski pins...@gmail.com wrote: On Thu, Jul 30, 2015 at 3:19 PM, Evgeny Stupachenko evstu...@gmail.com

Re: [RFC, patch] New attribute to create target clones

2015-07-30 Thread Joseph Myers
On Fri, 31 Jul 2015, Evgeny Stupachenko wrote: The patch enables new attribute 'ctarget', Attributes should be documented in extend.texi. -- Joseph S. Myers jos...@codesourcery.com

Re: [RFC, patch] New attribute to create target clones

2015-07-30 Thread Andrew Pinski
On Thu, Jul 30, 2015 at 3:19 PM, Evgeny Stupachenko evstu...@gmail.com wrote: Hi All, The patch enables new attribute 'ctarget', The attribute force compiler to create clones of a function with the attribute. For example: __attribute__((ctarget(avx,arch=slm,arch=core-avx2,default))) int

Re: [PATCH 3/4] remove unused TARGET_DEFERRED_OUTPUT_DEFS

2015-07-30 Thread Joseph Myers
On Fri, 24 Jul 2015, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders tbsaunde+...@tbsaunde.org gcc/ChangeLog: 2015-07-24 Trevor Saunders tbsaunde+...@tbsaunde.org * config/rs6000/aix43.h (TARGET_DEFERRED_OUTPUT_DEFS): Remove. * defaults.h

Re: C++ delayed folding branch review

2015-07-30 Thread Jeff Law
On 07/30/2015 05:02 PM, Jason Merrill wrote: Actually ME deals with none-cannonical form too. It just asserts on it at this place. After delayed-folding work I will continue work (Jeff pushed first parts of this work already to ML) on eliminating use of shorten_compare completely, and move

Re: [RFC, patch] New attribute to create target clones

2015-07-30 Thread Jeff Law
On 07/30/2015 04:19 PM, Evgeny Stupachenko wrote: Hi All, The patch enables new attribute 'ctarget', The attribute force compiler to create clones of a function with the attribute. For example: __attribute__((ctarget(avx,arch=slm,arch=core-avx2,default))) int foo () will create 3 clones of

Re: [gomp4.1] fold ordered depend(sink) clauses

2015-07-30 Thread Aldy Hernandez
On 07/30/2015 03:01 AM, Jakub Jelinek wrote: On Wed, Jul 29, 2015 at 04:48:23PM -0700, Aldy Hernandez wrote: @@ -7490,8 +7503,12 @@ gimplify_omp_for (tree *expr_p, gimple_seq *pre_p) == TREE_VEC_LENGTH (OMP_FOR_COND (for_stmt))); gcc_assert (TREE_VEC_LENGTH (OMP_FOR_INIT

Re: [PATCH] Fix PR66870 ppc64le, ppc64 split stack

2015-07-30 Thread Alan Modra
On Thu, Jul 30, 2015 at 03:30:12PM -0500, Lynn A. Boger wrote: PR66870 * gcc/config/rs6000/rs6000.c: Add check for no_split_stack function attribute along with flag_split_stack check to determine when to generate split stack

Re: lto wrapper verboseness

2015-07-30 Thread Richard Biener
On Thu, Jul 30, 2015 at 1:05 AM, Nathan Sidwell nat...@acm.org wrote: Jakub, this patch augments the lto wrapper to print out the arguments to spawned commands when verbose. I found this useful in debugging recent development. ok for trunk? Err - fork_execute through collect_execute already

Re: [gomp4.1] Support #pragma omp target {enter,exit} data

2015-07-30 Thread Jakub Jelinek
On Wed, Jul 29, 2015 at 10:06:52PM +0300, Ilya Verbin wrote: @@ -1241,6 +1245,62 @@ GOMP_target_update (int device, const void *unused, size_t mapnum, gomp_update (devicep, mapnum, hostaddrs, sizes, kinds, false); } +static void +gomp_exit_data (struct gomp_device_descr *devicep,

Re: lto wrapper verboseness

2015-07-30 Thread Richard Biener
On Thu, Jul 30, 2015 at 10:15 AM, Jakub Jelinek ja...@redhat.com wrote: On Thu, Jul 30, 2015 at 10:09:05AM +0200, Richard Biener wrote: On Thu, Jul 30, 2015 at 1:05 AM, Nathan Sidwell nat...@acm.org wrote: Jakub, this patch augments the lto wrapper to print out the arguments to spawned

Re: [PATCH] -Wtautological-compare should be quiet on floats

2015-07-30 Thread Marek Polacek
On Wed, Jul 29, 2015 at 04:44:46PM -0600, Jeff Law wrote: On 07/29/2015 08:08 AM, Marek Polacek wrote: As discussed elsewhere, -Wtautological-compare shouldn't warn about floating-point types because of the way NaN behave. I've been meaning to commit this one as obvious, but I'm not sure

[PATCH] Add warnings about GENERIC code-gen deficiencies in genmatch

2015-07-30 Thread Richard Biener
This adds warnings when a pattern requires inputs to have no side-effects to match for GENERIC code-gen. It also improves on some obvious cases like escapes through predicates or a few special cases such as TREE_REAL_CST. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.

Re: lto wrapper verboseness

2015-07-30 Thread Jakub Jelinek
On Thu, Jul 30, 2015 at 10:09:05AM +0200, Richard Biener wrote: On Thu, Jul 30, 2015 at 1:05 AM, Nathan Sidwell nat...@acm.org wrote: Jakub, this patch augments the lto wrapper to print out the arguments to spawned commands when verbose. I found this useful in debugging recent development.

New Swedish PO file for 'cpplib' (version 5.2.0)

2015-07-30 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'cpplib' has been submitted by the Swedish team of translators. The file is available at: http://translationproject.org/latest/cpplib/sv.po (This file,

Contents of PO file 'cpplib-5.2.0.sv.po'

2015-07-30 Thread Translation Project Robot
cpplib-5.2.0.sv.po.gz Description: Binary data The Translation Project robot, in the name of your translation coordinator. coordina...@translationproject.org

New Swedish PO file for 'gcc' (version 5.2.0)

2015-07-30 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the Swedish team of translators. The file is available at: http://translationproject.org/latest/gcc/sv.po (This file, 'gcc-5.2.0.sv.po', has

Re: [PATCH][AARCH64]Add backend combine_bfi pattern.

2015-07-30 Thread Kyrill Tkachov
Hi Renlin, On 30/07/15 16:50, Renlin Li wrote: Hi all, This insn should match the following similar rtx pattern and remove the redundant zero_extend operation if the width of zero_extract and inner-size of zero_extend totally match. (set (zero_extract:SI (reg/i:SI 0 x0)

Re: [PATCH] Honour DriverOnly for enum values in error messages.

2015-07-30 Thread Gunther Nikl
Dominik Vogt schrieb: The attached patch fixes a glicht in the error message generated for invalid values of enum options. IMHO, the patch is incomplete. diff --git a/gcc/opts-common.c b/gcc/opts-common.c index 8e51974..3bcbaf1 100644 --- a/gcc/opts-common.c +++ b/gcc/opts-common.c @@

[PATCH] Add __builtin_stack_top to x86 backend

2015-07-30 Thread H.J. Lu
On Tue, Jul 21, 2015 at 02:45:39PM -0700, H.J. Lu wrote: When __builtin_frame_address is used to retrieve the address of the function stack frame, the frame pointer is always kept, which wastes one register and 2 instructions. For x86-32, one less register means significant negative impact on

Re: [PATCH 0/9] start converting POINTER_SIZE to a hook

2015-07-30 Thread Trevor Saunders
On Wed, Jul 29, 2015 at 09:11:21AM +0100, Richard Sandiford wrote: Trevor Saunders tbsau...@tbsaunde.org writes: On Tue, Jul 28, 2015 at 09:24:17PM +0100, Richard Sandiford wrote: Trevor Saunders tbsau...@tbsaunde.org writes: On Mon, Jul 27, 2015 at 09:05:08PM +0100, Richard Sandiford

Re: [PATCH, i386]: Fix PR 66648, incorrect memcpy expansion with unrolled_loop strategy at -O2

2015-07-30 Thread Renlin Li
Hi Uros, You seems committing the test case in the wrong directory. I found it in testsuite/gcc.target/pr66648.c. Regards, Renlin On Sat, Jul 25, 2015 at 10:23 AM, Uros Bizjak ubiz...@gmail.com wrote: We have to emit check for main loop execution UNLESS we guarantee that min_size is at least

[PATCH][AARCH64]Add backend combine_bfi pattern.

2015-07-30 Thread Renlin Li
Hi all, This insn should match the following similar rtx pattern and remove the redundant zero_extend operation if the width of zero_extract and inner-size of zero_extend totally match. (set (zero_extract:SI (reg/i:SI 0 x0) (const_int 8 [0x8])

Re: [PATCH, i386]: Fix PR 66648, incorrect memcpy expansion with unrolled_loop strategy at -O2

2015-07-30 Thread Uros Bizjak
On Thu, Jul 30, 2015 at 6:01 PM, Renlin Li renlin@gmail.com wrote: Hi Uros, You seems committing the test case in the wrong directory. I found it in testsuite/gcc.target/pr66648.c. Thanks, I have moved the test to the correct place. Uros.

Re: C++ delayed folding branch review

2015-07-30 Thread Jason Merrill
On 07/29/2015 06:56 PM, Kai Tietz wrote: @@ -1430,6 +1438,8 @@ cxx_eval_call_expression (const constexpr_ctx *ctx, tree t, bool reduced_constant_expression_p (tree t) { + /* Make sure we remove useless initial NOP_EXPRs. */ + STRIP_NOPS (t); Checked, and removing those

Re: [PR64164] drop copyrename, integrate into expand

2015-07-30 Thread H.J. Lu
On Wed, Jul 29, 2015 at 1:23 PM, H.J. Lu hjl.to...@gmail.com wrote: On Wed, Jul 29, 2015 at 1:13 PM, Alexandre Oliva aol...@redhat.com wrote: On Jul 23, 2015, H.J. Lu hjl.to...@gmail.com wrote: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66978

Re: [RFC, patch] New attribute to create target clones

2015-07-30 Thread Evgeny Stupachenko
Forget C++ tests. On Fri, Jul 31, 2015 at 1:19 AM, Evgeny Stupachenko evstu...@gmail.com wrote: Hi All, The patch enables new attribute 'ctarget', The attribute force compiler to create clones of a function with the attribute. For example:

Re: [PATCH], PowerPC IEEE 128-bit patch #4

2015-07-30 Thread Joseph Myers
On Wed, 29 Jul 2015, Michael Meissner wrote: #6 Add support for using different names for the 64/128-bit integer conversion to IBM extended double, to allow a future version to switch the default for what long double is. It is not expected that GCC 6.x will make this

Re: [Patch 0/4] PowerPC64 Linux split stack support

2015-07-30 Thread Lynn A. Boger
Can this patch be submitted to fix the ppc 32 bootstrap error? On 06/19/2015 07:58 AM, Andreas Schwab wrote: * go-lang.c (go_langhook_init_options_struct): Don't set x_flag_split_stack. (go_langhook_post_options): Set it here instead. --- gcc/go/go-lang.c | 9 +

Re: C++ delayed folding branch review

2015-07-30 Thread Jeff Law
On 07/30/2015 10:52 AM, Jason Merrill wrote: This hunk is necessary as we don't use canonical-form produced by shorten_compare anymore. Therefore special operand can occur on right-hand side too. That seems like a problem, if the middle end is expecting the canonical form. What is your plan

c-common PATCH to pretty-printing of INDIRECT_REF of reference

2015-07-30 Thread Jason Merrill
I noticed that on the concepts branch we were printing an extra '*' when dumping an expression involving references, but haven't been able to come up with a testcase for the trunk. Tested x86_64-pc-linux-gnu, applying to trunk. commit c81115210bdc4dbc5614a1fea4dca59fa9f42af9 Author: Jason

Re: Add irreflexive comparison debug check

2015-07-30 Thread Jonathan Wakely
On 29/07/15 22:08 +0200, François Dumont wrote: Hi Here is a patch to add irreflexive debug check. Awesome! You can add PR libstdc++/60519 to the changelog. Standard algos signatures are such that there is no guaranty that the operator or predicate to compare the iterator value

[PATCH] Merge decision_tree::gen_{gimple,generic}

2015-07-30 Thread Richard Biener
This refactors things to remove duplicated code and make it easier to add more of it. Bootstrap on x86_64-unknown-linux-gnu running, no changes in generated code. Richard. 2015-07-30 Richard Biener rguent...@suse.de * genmatch.c (decision_tree::gen_gimple): Merge with ...

Re: [gomp4.1] fold ordered depend(sink) clauses

2015-07-30 Thread Jakub Jelinek
On Wed, Jul 29, 2015 at 04:48:23PM -0700, Aldy Hernandez wrote: @@ -7490,8 +7503,12 @@ gimplify_omp_for (tree *expr_p, gimple_seq *pre_p) == TREE_VEC_LENGTH (OMP_FOR_COND (for_stmt))); gcc_assert (TREE_VEC_LENGTH (OMP_FOR_INIT (for_stmt)) == TREE_VEC_LENGTH

[v3 patch] Fix self-assignment in std::experimental::any

2015-07-30 Thread Jonathan Wakely
When I fixed any's copying of non-trivial types I broke the self-assignment case. This adds an explicit check for it. Thanks to Axel Menzel for noticing the bug. Tested ppc64le-linux, committed to trunk. commit 8a9db2d69b88c72ca0fd794c1058918069787f83 Author: Jonathan Wakely jwak...@redhat.com

Re: [Patch] Small refactor on regex _State

2015-07-30 Thread Jonathan Wakely
On 29/07/15 21:55 -0700, Tim Shen wrote: - typedef _Matchertypename _TraitsT::char_type _MatcherT; + typedef _Matcher_Char_type _MatcherT; + static_assert(sizeof(_MatcherT) == sizeof(_Matcherchar), + The aussmption std::functionbool(char) has +

Re: [PATCH] -Wtautological-compare should be quiet on floats

2015-07-30 Thread Richard Biener
On Thu, Jul 30, 2015 at 10:29 AM, Marek Polacek pola...@redhat.com wrote: On Wed, Jul 29, 2015 at 04:44:46PM -0600, Jeff Law wrote: On 07/29/2015 08:08 AM, Marek Polacek wrote: As discussed elsewhere, -Wtautological-compare shouldn't warn about floating-point types because of the way NaN

[C++ Patch, preapproved] Prefer DECL_SOURCE_LOCATION to +D and +#D (3/n)

2015-07-30 Thread Paolo Carlini
Hi, for the time being this is the last installment. As anticipated, in some cases (4 overall) I'm using location_of instead of the simple DECL_SOURCE_LOCATION: in the case of note_name_declared_in_class the testsuite would pass anyway, but I prefer to be safe. Tested x86_64-linux. Thanks,

Re: lto wrapper verboseness

2015-07-30 Thread Nathan Sidwell
On 07/30/15 04:28, Richard Biener wrote: On Thu, Jul 30, 2015 at 10:15 AM, Jakub Jelinek ja...@redhat.com wrote: On Thu, Jul 30, 2015 at 10:09:05AM +0200, Richard Biener wrote: On Thu, Jul 30, 2015 at 1:05 AM, Nathan Sidwell nat...@acm.org wrote: Jakub, this patch augments the lto wrapper to

[PATCH] Fix PR67052

2015-07-30 Thread Richard Biener
The following patch avoids folding NaN 0 to false if -ftrapping-math is in effect. Bootstrap regtest pending on x86_64-unknown-linux-gnu. Richard. 2015-07-30 Richard Biener rguent...@suse.de PR middle-end/67052 * fold-const.c (fold_binary_loc): Do not fold NaN 0 to false

[gomp4] ptx builtins

2015-07-30 Thread Nathan Sidwell
I've committed this to gomp4 branch. The vector neutering code already has machinery to generate DI/DF shuffles from the SI underlying instruction. This generalizes that machinery and changes the shuffle-down machinery to use it. Less code duplication - yay! Also added a DF mode shuffle