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 p

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 clone

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 its

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

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

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

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

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 > > gcc/ChangeLog: > > 2015-07-24 Trevor Saunders > > * config/rs6000/aix43.h (TARGET_DEFERRED_OUTPUT_DEFS): Remove. > * defaults.h (TARGET_DEFERRED_OUTPUT_DEFS): Likewise. > * doc/tm.texi: Regen

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

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 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. >>> (go_langhoo

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 wrote: > On Thu, Jul 30, 2015 at 3:19 PM, Evgeny Stupachenko > wrote: >> Hi All, >> >> The pat

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 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: [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 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","d

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

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

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

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

2015-07-30 Thread Richard Sandiford
David Edelsohn 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 that the co

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 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, Thanks for the review

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

Re: C++ delayed folding branch review

2015-07-30 Thread Kai Tietz
2015-07-30 18:52 GMT+02:00 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) >>

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

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 we

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

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

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 revis

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 Merri

[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

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

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 STRIP_NOPS

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 wrote: > On Wed, Jul 29, 2015 at 1:13 PM, Alexandre Oliva wrote: >> On Jul 23, 2015, "H.J. Lu" wrote: >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66978 >> >>> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66983 >> >> Thanks, both of these are also

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 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: [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, 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 wrote: > We have to emit check for main loop execution UNLESS we guarantee that > min_size is at least equal to size_ne

[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]) (const_in

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

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

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 > > > > * builtins.c (fold_builtin_tolower, fold_builtin_toupper): New > > static functions. > > (fold_builtin_1): Handl

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

2015-07-30 Thread Andreas Schwab
Kugan 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, sch...@linux-m68k.org GPG Key fin

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

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

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

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 libg

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 o

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

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 wrote: Le 29/07/2015 13:22, Richard Biener a écrit : On Wed, Jul 29, 2015 at 11:34 AM, Mikael Morin wrote: Le 29/07/2015 10:26, Richard

Re: C++ delayed folding branch review

2015-07-30 Thread Kai Tietz
2015-07-30 0:56 GMT+02:00 Kai Tietz : > 2015-07-29 19:48 GMT+02:00 Jason Merrill : >> 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 @@ cxx_eval_constant_expres

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 fu

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

Re: Remove support for atomic_clear pattern

2015-07-30 Thread Richard Biener
On Thu, Jul 30, 2015 at 3:02 PM, Richard Sandiford 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 passed the > memory mod

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 corre

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 t

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

Re: [PATCH] 2015-07-29 Benedikt Huber Philipp Tomsich

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

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

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:

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 wrot

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

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

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

[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 PR middle-end/67052 * fold-const.c (fold_binary_loc): Do not fold NaN < 0 to false if FP m

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 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 wrote: Jakub, this patch augments the lto wrapper to print out the arguments to spaw

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 typ

[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 * genmatch.c (decision_tree::gen_gimple): Merge with ... (decision_tree::gen

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

Re: [Patch] Small refactor on _State<>

2015-07-30 Thread Jonathan Wakely
On 29/07/15 21:55 -0700, Tim Shen wrote: - typedef _Matcher _MatcherT; + typedef _Matcher<_Char_type> _MatcherT; + static_assert(sizeof(_MatcherT) == sizeof(_Matcher), + "The aussmption std::function has " + "the same size as std::function is vio

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

[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 Date: Thu Jul 30

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

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 just

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

Re: lto wrapper verboseness

2015-07-30 Thread Richard Biener
On Thu, Jul 30, 2015 at 10:15 AM, Jakub Jelinek 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 wrote: >> > Jakub, >> > this patch augments the lto wrapper to print out the arguments to spawned >> > commands when verbose.

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.

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, 'cpplib-5.2.0.sv.po',

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

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

Re: lto wrapper verboseness

2015-07-30 Thread Richard Biener
On Thu, Jul 30, 2015 at 1:05 AM, Nathan Sidwell 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 does this

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