Re: [rs6000] Fix compare debug failure on AIX

2015-05-01 Thread Alan Modra
On Fri, May 01, 2015 at 09:48:51AM -0400, David Edelsohn wrote: > On Thu, Apr 30, 2015 at 7:28 AM, Eric Botcazou wrote: > >> We might want to check if doing -Og and not just -O0. > > > > You're right, thanks, amended patch attached, same ChangeLog. > > Why should GCC unnecessarily create stack fr

[doc, committed] more attribute documentation cleanup

2015-05-01 Thread Sandra Loosemore
I've checked in another installment of my ongoing cleanup of the attribute documentation in extend.texi. In this patch, I've tidied up the variable and type attribute sections. There's not much actual change to the document text here, mostly just moving things around and tweaking structure.

Re: [PATCH] fixup libobjc usage of PCC_BITFIELD_TYPE_MATTERS

2015-05-01 Thread pinskia
> On May 1, 2015, at 8:30 PM, tbsaunde+...@tbsaunde.org wrote: > > From: Trevor Saunders > > Hi, > > This adds a configure check to libobjc to find out if types of bitfields > effect > their layout, and uses it to replace the rather broken usage of > PCC_BITFIELD_TYPE_MATTERS. > > bootstr

[PATCH] fixup libobjc usage of PCC_BITFIELD_TYPE_MATTERS

2015-05-01 Thread tbsaunde+gcc
From: Trevor Saunders Hi, This adds a configure check to libobjc to find out if types of bitfields effect their layout, and uses it to replace the rather broken usage of PCC_BITFIELD_TYPE_MATTERS. bootstrapped + regtested x86_64-linux-gnu, bootstrapped on ppc64le-linux-gnu and ran check-objc th

Re: [Patch] pr65779 - [5/6 Regression] undefined local symbol on powerpc

2015-05-01 Thread Alan Modra
On Tue, Apr 21, 2015 at 02:30:15PM +0200, Jakub Jelinek wrote: > This isn't safe. There could be a debug_insn for the same decl anywhere in > between the dinsn and bb_note (bb) on the chosen live path, if there is, > this change will break stuff. Yeah, I see now that what I was trying to do can't

Re: [libstdc++ PATCH] Implement observer_ptr

2015-05-01 Thread Ed Smith-Rowland
On 05/01/2015 05:01 PM, Jonathan Wakely wrote: On 01/05/15 16:37 +0300, Ville Voutilainen wrote: Tested on Linux-x64. Implement observer_ptr. Thanks! Committed with some minor formatting changes. I've also committed this to add feature-test macros and update the docs. Tested powerpc64le-l

[PATCH 4/4] Replace line_map union with C++ class hierarchy

2015-05-01 Thread David Malcolm
This patch eliminates the union in struct line_map in favor of a simple class hierarchy, making struct line_map a base class, with line_map_ordinary and line_map_macro subclasses. The patch eliminates all usage of linemap_check_ordinary and linemap_check_macro from line-map.h, updating return type

[PATCH 3/4] libcpp/input.c: Add a way to visualize the linemaps

2015-05-01 Thread David Malcolm
As a relative newcomer to GCC, one of the issues I had was becoming comfortable with the linemap API and its internal representation. To familiarize myself with it, I wrote a dumping routine to try to visualize how the source_location space is carved up between line maps, and what each number can

[PATCH 2/4] libcpp: Replace macro usage with C++ constructs

2015-05-01 Thread David Malcolm
libcpp makes extensive use of the C preprocessor. Whilst this has a pleasingly self-referential quality, I find the code hard-to-read; implementing source location support in my JIT branch was much harder than I felt it should have been. In an attempt at making the code easier to follow, and to b

[PATCH 0/4] libcpp patches

2015-05-01 Thread David Malcolm
The following patches for libcpp (and gcc) are an attempt at making the libcpp API and internal representation easier to understand and to hack on, simplifying it (IMHO) with a modest amount of C++, without changing its behavior. See the individual patches for their descriptions. How do these loo

[PATCH 1/4] libcpp: Improvements to comments in line-map.h/c

2015-05-01 Thread David Malcolm
This patch updates and expands some comments in libcpp, adding a big table to try to clarify what an individual source_location value can mean. libcpp/ChangeLog: * include/line-map.h: Fix comment at the top of the file. (source_location): Rewrite and expand the comment for this

[RFA] More type narrowing in match.pd V2

2015-05-01 Thread Jeff Law
Here's an updated patch to add more type narrowing to match.pd. Changes since the last version: Slight refactoring of the condition by using types_match as suggested by Richi. I also applied the new types_match to 2 other patterns in match.pd where it seemed clearly appropriate. Additionall

Re: [PATCH/libiberty] fix build of gdb/binutils with clang.

2015-05-01 Thread Yunlian Jiang
The test case does not have #define _GNU_SOURCE, so it says error: ‘asprintf’ undeclared (first use in this function) On Fri, May 1, 2015 at 3:45 PM, Ian Lance Taylor wrote: > On Tue, Apr 28, 2015 at 2:59 PM, Yunlian Jiang wrote: >> I believe this is the same problem as >> https://gcc.gnu.org/ml

PR 64454: Improve VRP for %

2015-05-01 Thread Marc Glisse
Hello, this patch tries to tighten a bit the range estimate for x%y. slp-perm-7.c started failing by vectorizing more than expected, I assumed it was a good thing and updated the test. I am less conservative than Jakub with division by 0, but I still don't really understand how empty ranges ar

Re: [PATCH/libiberty] fix build of gdb/binutils with clang.

2015-05-01 Thread Ian Lance Taylor
On Tue, Apr 28, 2015 at 2:59 PM, Yunlian Jiang wrote: > I believe this is the same problem as > https://gcc.gnu.org/ml/gcc-patches/2008-07/msg00292.html > > The asprinf declaration is messed up when using clang to build gdb. > > diff --git a/include/libiberty.h b/include/libiberty.h > index b33dd

Re: [wwwdocs, committed] gcc-5/changes.html: Update Fortran section

2015-05-01 Thread Gerald Pfeifer
On Sun, 31 Aug 2014, Tobias Burnus wrote: > I have committed the attached patch. Comment and suggestions are welcome! I applied the small follow-up below. What I am struggling to understand, though, is the following piece: Full experimental support of Fortran 2008's coarrays with -fcoarray

Re: [libstdc++ PATCH] Implement observer_ptr

2015-05-01 Thread Jonathan Wakely
On 01/05/15 22:01 +0100, Jonathan Wakely wrote: On 01/05/15 16:37 +0300, Ville Voutilainen wrote: Tested on Linux-x64. Implement observer_ptr. Thanks! Committed with some minor formatting changes. I've also committed this to add feature-test macros and update the docs. Tested powerpc64le-l

Re: [Patch, Fortran] Simplify lbound

2015-05-01 Thread Mikael Morin
Hello, Le 30/04/2015 20:19, Mikael Morin a écrit : >>> As you may want to simplify in the limited scope of the matmul inlining, >>> I'm giving comments about the patch (otherwise you can ignore them): >>> - No need to check for allocatable or pointer, it should be excluded by >>> as->type == AS_A

Re: [libstdc++ PATCH] Implement observer_ptr

2015-05-01 Thread Jonathan Wakely
On 01/05/15 16:37 +0300, Ville Voutilainen wrote: Tested on Linux-x64. Implement observer_ptr. Thanks! Committed with some minor formatting changes. I've also committed this to add feature-test macros and update the docs. Tested powerpc64le-linux, committed to trunk. commit 2606fe2072160

Re: [rfc, stage 1] default to -fno-delete-null-pointer-checks on nios2-elf

2015-05-01 Thread Sandra Loosemore
Re https://gcc.gnu.org/ml/gcc-patches/2015-03/msg01510.html : On 04/15/2015 10:42 PM, Jeff Law wrote: It looks very sane to me. This is probably how the AVR and CR16 should have been handled to begin with IMHO. FWIW, I generally discourage ports overriding default options, but this is a case w

Re: [patch] Implement ISO/IEC TS 18822 C++ File system TS

2015-05-01 Thread Jonathan Wakely
On 01/05/15 20:38 +0100, Jonathan Wakely wrote: On 01/05/15 21:28 +0200, Daniel Krügler wrote: But if I read your implementation of path::compare(const path& p) correctly it *also* may allocate memory by copying _M_pathname into a _Cmpt object. Yes, I agree that there's a bug here that could c

Re: [patch] Implement ISO/IEC TS 18822 C++ File system TS

2015-05-01 Thread Jonathan Wakely
On 01/05/15 11:53 +0100, Jonathan Wakely wrote: On 01/05/15 10:05 +0900, Luke Allardyce wrote: This fails on mingw-w64 4.02 due to Yay, thanks for trying it! an extra typename: /mnt/build/native/gcc/x86_64-w64-mingw32/libstdc++-v3/include/experimental/fs_path.h:784:35: error: expected neste

Re: [patch] Implement ISO/IEC TS 18822 C++ File system TS

2015-05-01 Thread Jonathan Wakely
On 01/05/15 21:28 +0200, Daniel Krügler wrote: 2015-05-01 20:22 GMT+02:00 Jonathan Wakely : On 01/05/15 19:03 +0200, Daniel Krügler wrote: b/libstdc++-v3/src/filesystem/path.cc: - path::compare(const path& p) const noexcept: Shouldn't the implementation of this noexcept function not try to c

Re: [patch] Implement ISO/IEC TS 18822 C++ File system TS

2015-05-01 Thread Daniel Krügler
2015-05-01 20:22 GMT+02:00 Jonathan Wakely : > On 01/05/15 19:03 +0200, Daniel Krügler wrote: >> >> b/libstdc++-v3/src/filesystem/path.cc: >> >> - path::compare(const path& p) const noexcept: >> >> Shouldn't the implementation of this noexcept function not try to >> create copies of path objects? C

Re: [c++-concepts] code review

2015-05-01 Thread Andrew Sutton
> It looks like things are coming together pretty well. What's your feeling > about readiness to merge into the trunk? Is the branch down to no > regressions? They are coming together pretty well. We have one major unit test failure involving template introductions (Braden is working on it), one

Re: More type narrowing in match.pd

2015-05-01 Thread Jeff Law
On 04/30/2015 05:07 AM, Richard Biener wrote: On Thu, Apr 30, 2015 at 12:53 PM, Marc Glisse wrote: On Thu, 30 Apr 2015, Richard Biener wrote: I have in my local dev tree (so completely untested...) @@ -1040,31 +1052,22 @@ (define_operator_list CBRT BUILT_IN_CBRT operation and convert the

Re: [PATCH][expr.c] PR 65358 Avoid clobbering partial argument during sibcall

2015-05-01 Thread Jeff Law
On 04/28/2015 03:54 AM, Kyrill Tkachov wrote: On 27/04/15 21:13, Jeff Law wrote: On 04/21/2015 11:33 AM, Kyrill Tkachov wrote: On 21/04/15 15:09, Jeff Law wrote: On 04/21/2015 02:30 AM, Kyrill Tkachov wrote: From reading config/stormy16/stormy-abi it seems to me that we don't pass argumen

Re: More type narrowing in match.pd

2015-05-01 Thread Jeff Law
On 04/30/2015 03:38 PM, Marc Glisse wrote: On Thu, 30 Apr 2015, Jeff Law wrote: On 04/30/2015 01:17 AM, Marc Glisse wrote: +/* This is another case of narrowing, specifically when there's an outer + BIT_AND_EXPR which masks off bits outside the type of the innermost + operands. Like the

[c++-concepts] code review

2015-05-01 Thread Jason Merrill
It looks like things are coming together pretty well. What's your feeling about readiness to merge into the trunk? Is the branch down to no regressions? See you on Monday! @@ -4146,21 +4146,21 @@ build_new_function_call (tree fn, vec **args, bool koenig_p, if (TREE_CODE (fn) == TEMP

Re: [PATCH 2/4] match.pd: Add x & ~(x & y) -> x & ~y pattern

2015-05-01 Thread Jeff Law
On 01/21/2015 03:49 AM, Rasmus Villemoes wrote: gcc.dg/20150120-2.c: New test Clearing a certain subset of bits, for example to round down x to a multiple of a power of 2, is sometimes written x & ~(x & y), where y may or may not be a constant. It is shorter to use x & ~y, particularly when y is

Re: [PATCH 1/4] match.pd: Add x + (x & 1) -> (x + 1) & ~1 pattern

2015-05-01 Thread Jeff Law
On 01/21/2015 03:49 AM, Rasmus Villemoes wrote: gcc.dg/20150120-1.c: New test Rounding an integer to the next even integer is sometimes written x += x & 1. The equivalent x = (x+1)&~1 usually uses one less register, and in practical cases only the new value of x will be used (making it unlikely

Re: [Patch, fortran] PR65792 - allocation of scalar elemental function with structure constructor fails

2015-05-01 Thread Paul Richard Thomas
Dear All, By the time I went to commit, something had changed and the patch caused a regression. I presume that the version that I had of Andre's patch was not the same as the one committed. I'll cast an eye over it this weekend and see if I can understand what gives. Cheers paul On 27 April 20

[PATCH] pr65771 testcase

2015-05-01 Thread David Edelsohn
The TLS testcase needs to add target-specific TLS options. Committed as obvious. * gcc.dg/debug/pr65771.c: Add "dg-add-options tls". Index: debug/pr65771.c === --- debug/pr65771.c (revision 222680) +++ debug/pr65771.c

Re: [patch] Implement ISO/IEC TS 18822 C++ File system TS

2015-05-01 Thread Jonathan Wakely
On 01/05/15 19:03 +0200, Daniel Krügler wrote: b/libstdc++-v3/src/filesystem/path.cc: - path::compare(const path& p) const noexcept: Shouldn't the implementation of this noexcept function not try to create copies of path objects? Couldn't _Cmpt just hold references to _M_pathname? All your ot

Re: [patch] Perform anonymous constant propagation during inlining

2015-05-01 Thread Eric Botcazou
> OK, how aggressive then? We could as well do the substitution for all > copies: > > /* For EXPAND_INITIALIZER try harder to get something simpler. >Otherwise, substitute copies on the RHS, this can propagate >constants at -O0 and thus simplify arithmetic operations. */ >

Re: [RFC][PATCH][X86_64] Eliminate PLT stubs for specified external functions via -fno-plt=

2015-05-01 Thread Sriraman Tallam
On Fri, May 1, 2015 at 9:26 AM, Xinliang David Li wrote: > yes -- it is good to turn this on by default in LTO mode without > requiring user to specify the option. Yes, with LTO, we would exactly know what the "truly extern" functions are and PLT stubs can be eliminated for all extern functions w

Re: [patch] Implement ISO/IEC TS 18822 C++ File system TS

2015-05-01 Thread Jonathan Wakely
On 01/05/15 19:03 +0200, Daniel Krügler wrote: - There are three places where you refer to std::__addressof and one where you refer to std::addressof. Is this difference intentional? No, I'll make them all use __addressof. - I found all together six non-uglified usages of parameter name "rhs"

Re: [RFC][PATCH][X86_64] Eliminate PLT stubs for specified external functions via -fno-plt=

2015-05-01 Thread Sriraman Tallam
On Fri, May 1, 2015 at 8:01 AM, Andi Kleen wrote: > Sriraman Tallam writes: >> >> This comes with caveats. This cannot be generally done for all >> functions marked extern as it is impossible for the compiler to say if >> a function is "truly extern" (defined in a shared library). If a >> funct

Re: [C++ patch] PR 65858

2015-05-01 Thread Jason Merrill
OK (= true). Jason

Re: [patch] Implement ISO/IEC TS 18822 C++ File system TS

2015-05-01 Thread Daniel Krügler
2015-04-30 19:32 GMT+02:00 Jonathan Wakely : > This is the complete implementation I intend > to commit shortly. (It's also been pushed to the redi/filesystem-ts > branch in the git mirror). - There are three places where you refer to std::__addressof and one where you refer to std::addressof. Is

Re: [PATCH] libstdc++: Fix list.cc xmethods test.

2015-05-01 Thread Jonathan Wakely
On 29/04/15 17:18 +0100, Jonathan Wakely wrote: On 29 April 2015 at 17:04, Doug Evans wrote: Tested the same patch on the gcc 5.0 branch. Just double checking ... ok to apply there too? Yes, OK for the branch too. btw, the test is currently marked as unsupported by the test run. I don't know

[patch] Use deleted functions to make std::locale::facet non-copyable

2015-05-01 Thread Jonathan Wakely
Replace the private & unimplemented idiom with deleted functions when available. This gives slightly better error messages (as well as stricter conformance to the precise wording of the standard). Tested powerpc64le-linux, committed to trunk. commit dd256f9035b4e32f1bb53eaf82919b0b69c6c999 Author

Re: [patch] Perform anonymous constant propagation during inlining

2015-05-01 Thread Eric Botcazou
> Yeah, I think that's a way better place for the hack. OK, how aggressive then? We could as well do the substitution for all copies: /* For EXPAND_INITIALIZER try harder to get something simpler. Otherwise, substitute copies on the RHS, this can propagate constants at -O

Re: [PATCH 1/3] optabs.c: Make vector_compare_rtx cope with VOIDmode constants (e.g. const0_rtx)

2015-05-01 Thread Jeff Law
On 05/01/2015 09:12 AM, Alan Lawrence wrote: Alan Lawrence wrote: As per introduction, this allows vector_compare_rtx to work on DImode vectors. Bootstrapped + check-gcc on x86-unknown-linux-gnu. gcc/ChangeLog: * optabs.c (vector_compare_rtx): Handle RTL operands having VOIDmode. Ping.

Re: [RFC][PATCH][X86_64] Eliminate PLT stubs for specified external functions via -fno-plt=

2015-05-01 Thread Xinliang David Li
yes -- it is good to turn this on by default in LTO mode without requiring user to specify the option. David On Fri, May 1, 2015 at 9:23 AM, H.J. Lu wrote: > On Fri, May 1, 2015 at 9:19 AM, Xinliang David Li wrote: >> On Fri, May 1, 2015 at 8:01 AM, Andi Kleen wrote: >>> Sriraman Tallam write

Re: [RFC][PATCH][X86_64] Eliminate PLT stubs for specified external functions via -fno-plt=

2015-05-01 Thread H.J. Lu
On Fri, May 1, 2015 at 9:19 AM, Xinliang David Li wrote: > On Fri, May 1, 2015 at 8:01 AM, Andi Kleen wrote: >> Sriraman Tallam writes: >>> >>> This comes with caveats. This cannot be generally done for all >>> functions marked extern as it is impossible for the compiler to say if >>> a functi

Re: [RFC][PATCH][X86_64] Eliminate PLT stubs for specified external functions via -fno-plt=

2015-05-01 Thread Xinliang David Li
On Fri, May 1, 2015 at 8:01 AM, Andi Kleen wrote: > Sriraman Tallam writes: >> >> This comes with caveats. This cannot be generally done for all >> functions marked extern as it is impossible for the compiler to say if >> a function is "truly extern" (defined in a shared library). If a >> funct

Re: [PATCH][doc] Update definition location of attribute_spec in documentation

2015-05-01 Thread Jeff Law
On 05/01/2015 08:51 AM, Kyrill Tkachov wrote: Hi all, Like the subject says, struct attribute_spec is now defined in tree-core.h rather than tree.h. The patch updates the reference that I spotted. Ok for trunk? Thanks, Kyrill 2015-05-01 Kyrylo Tkachov * target.def (attribute_table):

[PATCH][tree-ssa-math-opts] Expand pow (x, CONST) using square roots when possible

2015-05-01 Thread Kyrill Tkachov
Hi all, GCC has some logic to expand calls to pow (x, 0.75), pow (0.25) and pow (x, (int)k + 0.5) using square roots. So, for the above examples it would generate sqrt (x) * sqrt (sqrt (x)), sqrt (sqrt (x)) and powi (x, k) * sqrt (x) (assuming k > 0. For k < 0 it will calculate the reciprocal

Re: [PATCH][AARCH64]Use mov for add with large immediate.

2015-05-01 Thread Marcus Shawcroft
On 1 May 2015 at 16:30, Renlin Li wrote: > Thank you, Marcus. I have updated the patch accordingly, please check.. > > Regards, > Renlin Li OK, thanks /Marcus

Re: [PATCH][doc] Update definition location of attribute_spec in documentation

2015-05-01 Thread Kyrill Tkachov
On 01/05/15 16:22, Sandra Loosemore wrote: On 05/01/2015 08:51 AM, Kyrill Tkachov wrote: Hi all, Like the subject says, struct attribute_spec is now defined in tree-core.h rather than tree.h. The patch updates the reference that I spotted. Ok for trunk? Thanks, Kyrill 2015-05-01 Kyrylo Tka

Re: [PATCH] PR ld/18355: --enable-shared doesn't work

2015-05-01 Thread H.J. Lu
On Wed, Apr 29, 2015 at 9:12 AM, H.J. Lu wrote: > When bfd is configured as a shared library, we need to configure zlib > with --enable-host-shared since zlib is used by bfd. Any comments, > feedbacks, objections? > > > H.J. > -- > PR ld/18355 > * Makefile.def: Add extra_configure

Re: [PATCH][AARCH64]Use mov for add with large immediate.

2015-05-01 Thread Renlin Li
Hi Marcus, On 01/05/15 11:19, Marcus Shawcroft wrote: On 21 April 2015 at 17:10, Renlin Li wrote: Hi all, This is a simple patch to generate a move instruction to temporarily hold the large immediate for a add instruction. GCC regression test has been run using aarch64-none-elf toolchain. NO

Re: [PATCH][doc] Update definition location of attribute_spec in documentation

2015-05-01 Thread Sandra Loosemore
On 05/01/2015 08:51 AM, Kyrill Tkachov wrote: Hi all, Like the subject says, struct attribute_spec is now defined in tree-core.h rather than tree.h. The patch updates the reference that I spotted. Ok for trunk? Thanks, Kyrill 2015-05-01 Kyrylo Tkachov * target.def (attribute_table):

[patch] Rename template parameter of std::__alloc_rebind

2015-05-01 Thread Jonathan Wakely
The name of this alias template is a copy&paste error, it should be _Alloc. Tested powerpc64le-linux, committed to trunk. commit a772309ec9fc300e57edd750fa32b8320d68004a Author: Jonathan Wakely Date: Fri May 1 16:12:16 2015 +0100 * include/bits/alloc_traits.h (__alloc_rebind): Change para

Re: [RFC]: Remove Mem/address type assumption in combiner

2015-05-01 Thread Segher Boessenkool
On Wed, Apr 29, 2015 at 12:03:35PM -0500, Segher Boessenkool wrote: > On Wed, Apr 29, 2015 at 09:25:21AM +, Kumar, Venkataramanan wrote: > > diff --git a/gcc/combine.c b/gcc/combine.c > > index 5c763b4..945abdb 100644 > > --- a/gcc/combine.c > > +++ b/gcc/combine.c > > @@ -7703,8 +7703,6 @@ mak

Re: [PATCH, AArch64] Add Cortex-A53 erratum 843419 configure-time option

2015-05-01 Thread Marcus Shawcroft
On 1 May 2015 at 14:56, Yvan Roux wrote: > 2015-05-01 Yvan Roux > > * configure.ac: Add --enable-fix-cortex-a53-843419 option. > * configure: Regenerate. > * config/aarch64/aarch64-elf-raw.h (CA53_ERR_843419_SPEC): Define. > (LINK_SPEC): Include CA53_ERR_843419_SPEC. >

Re: [PATCH][docs] Re: Update __atomic builtins documentation.

2015-05-01 Thread Jeff Law
On 04/30/2015 04:44 AM, Matthew Wahab wrote: [added tags to subject] Ping. On 20/04/15 14:29, Matthew Wahab wrote: Hello, The documentation for the __atomic builtins isn't clear about their expectations and behaviour. In particular, assumptions about the C11/C++11 restrictions on programs sho

Re: [patch] Implement ISO/IEC TS 18822 C++ File system TS

2015-05-01 Thread Rainer Orth
Jonathan Wakely writes: > Sorry about that, I'll add a check for fchmodat, in the meantime this > will fix bootstrap. Committed as obvious. > > > commit f4768ebcfd68e2fa6e4763d0b681e8fe710c64c4 > Author: Jonathan Wakely > Date: Fri May 1 16:09:28 2015 +0100 > > * acinclude.m4 (GLIBCXX_EN

Re: [PATCH 1/3] optabs.c: Make vector_compare_rtx cope with VOIDmode constants (e.g. const0_rtx)

2015-05-01 Thread Alan Lawrence
Alan Lawrence wrote: As per introduction, this allows vector_compare_rtx to work on DImode vectors. Bootstrapped + check-gcc on x86-unknown-linux-gnu. gcc/ChangeLog: * optabs.c (vector_compare_rtx): Handle RTL operands having VOIDmode. Ping. (DImode vectors are explicitly allowed by

Re: [patch] Implement ISO/IEC TS 18822 C++ File system TS

2015-05-01 Thread Jonathan Wakely
On 01/05/15 14:45 +0200, Rainer Orth wrote: Jonathan Wakely writes: I've tested this on GNU/Linux and DragonFly BSD, but as it's probably not going to build everywhere I've added the configure option --enable-libstdcxx-filesystem-ts which defaults to enabled on GNU, BSD and Solaris targets, an

Re: [PATCH]Add sync_int_long target selector to gcc.dg/pr65345-2.c

2015-05-01 Thread Jeff Law
On 05/01/2015 08:44 AM, Renlin Li wrote: Hi all, The new test case requires the target to support atomic operations on "int" type. By adding the correct target selector, it fixes the regression on arm-none-elf target. Okay to commit? gcc/testsuite/ChangeLog: 2015-05-01 Renlin Li * g

Re: [RFC][PATCH][X86_64] Eliminate PLT stubs for specified external functions via -fno-plt=

2015-05-01 Thread Andi Kleen
Sriraman Tallam writes: > > This comes with caveats. This cannot be generally done for all > functions marked extern as it is impossible for the compiler to say if > a function is "truly extern" (defined in a shared library). If a > function is not truly extern(ends up defined in the final execu

[PATCH][doc] Update definition location of attribute_spec in documentation

2015-05-01 Thread Kyrill Tkachov
Hi all, Like the subject says, struct attribute_spec is now defined in tree-core.h rather than tree.h. The patch updates the reference that I spotted. Ok for trunk? Thanks, Kyrill 2015-05-01 Kyrylo Tkachov * target.def (attribute_table): Mention that struct attribute_spec is defin

[PATCH]Add sync_int_long target selector to gcc.dg/pr65345-2.c

2015-05-01 Thread Renlin Li
Hi all, The new test case requires the target to support atomic operations on "int" type. By adding the correct target selector, it fixes the regression on arm-none-elf target. Okay to commit? gcc/testsuite/ChangeLog: 2015-05-01 Renlin Li * gcc.dg/pr65345-2.c: Add sync_int_long tar

[PATCH][ARM] Add debug dumping of cost table fields

2015-05-01 Thread Kyrill Tkachov
Hi all, This patch adds a macro to wrap cost field accesses into a helpful debug dump, saying which field is being accessed at what line and with what values. This helped me track down cases where the costs were doing the wrong thing by allowing me to see which path in arm_new_rtx_costs was taken

Re: [patch] Perform anonymous constant propagation during inlining

2015-05-01 Thread Richard Biener
On May 1, 2015 12:27:17 PM GMT+02:00, Eric Botcazou wrote: >> Hmm, special-casing this in the inliner looks odd to me. ISTR the >inliner >> already propagates constant parameters to immediate uses, so I guess >> you run into the casting case you handle specially. > >Right on both counts, the ori

Re: [PATCH] Mark help string in DEFPARAM as no-c-format

2015-05-01 Thread Jeff Law
On 05/01/2015 05:27 AM, Andreas Schwab wrote: Tested with make gcc.pot. Andreas. PR translation/65959 * params.h (DEFPARAM): Rename msgid to nocmsgid. OK. jeff

Re: Mostly rewrite genrecog

2015-05-01 Thread Jeff Law
On 05/01/2015 06:41 AM, Richard Sandiford wrote: Richard Sandiford writes: Richard Biener writes: On Thu, Apr 30, 2015 at 2:08 PM, Andreas Schwab wrote: Richard Sandiford writes: Andreas Schwab writes: Richard Sandiford writes: /* Represents a test and the action that should be take

Re: [PATCH, AArch64] Add Cortex-A53 erratum 843419 configure-time option

2015-05-01 Thread Yvan Roux
On 1 May 2015 at 13:39, Yvan Roux wrote: > Hi Marcus, > > (Sorry wanted to cc you in my first mail but seems that gmail prefers > Maxim to Marcus ! ;) > > On 1 May 2015 at 13:11, Marcus Shawcroft wrote: >> On 1 May 2015 at 10:11, Yvan Roux wrote: >>> Hi all, >>> >>> As described in the thread be

Re: [rs6000] Fix compare debug failure on AIX

2015-05-01 Thread David Edelsohn
On Thu, Apr 30, 2015 at 7:28 AM, Eric Botcazou wrote: >> We might want to check if doing -Og and not just -O0. > > You're right, thanks, amended patch attached, same ChangeLog. Why should GCC unnecessarily create stack frames to avoid compare-debug testcase failures? - David

[libstdc++ PATCH] Implement observer_ptr

2015-05-01 Thread Ville Voutilainen
Tested on Linux-x64. 2015-05-01 Ville Voutilainen Implement observer_ptr. * include/Makefile.am: Add new exported header. * include/Makefile.in: Regenerate. * include/experimental/memory: New. * testsuite/experimental/memory/observer_ptr/assignment/assign.cc: Likewise. *

Re: [PATCH][AArch64] Make aarch64_min_divisions_for_recip_mul configurable

2015-05-01 Thread Kyrill Tkachov
On 01/05/15 14:11, Wilco Dijkstra wrote: Marcus Shawcroft wrote: On 1 May 2015 at 12:26, Wilco Dijkstra wrote: Marcus Shawcroft wrote: On 27 April 2015 at 14:43, Wilco Dijkstra wrote: static unsigned int -aarch64_min_divisions_for_recip_mul (enum machine_mode mode ATTRIBUTE_UNUSED) +aar

RE: [PATCH][AArch64] Make aarch64_min_divisions_for_recip_mul configurable

2015-05-01 Thread Wilco Dijkstra
> Marcus Shawcroft wrote: > On 1 May 2015 at 12:26, Wilco Dijkstra wrote: > > > > > >> Marcus Shawcroft wrote: > >> On 27 April 2015 at 14:43, Wilco Dijkstra wrote: > >> > >> >> static unsigned int > >> >> -aarch64_min_divisions_for_recip_mul (enum machine_mode mode > >> >> ATTRIBUTE_UNUSED) >

Re: [PATCH][AArch64] Fix Cortex-A53 shift costs

2015-05-01 Thread Kyrill Tkachov
On 01/05/15 14:07, Wilco Dijkstra wrote: Marcus Shawcroft wrote: On 5 March 2015 at 14:49, Wilco Dijkstra wrote: This patch fixes the shift costs for Cortex-A53 so they are more accurate - immediate shifts use SBFM/UBFM which takes 2 cycles, register controlled shifts take 1 cycle. Bootstr

RE: [PATCH][AArch64] Fix Cortex-A53 shift costs

2015-05-01 Thread Wilco Dijkstra
> Marcus Shawcroft wrote: > On 5 March 2015 at 14:49, Wilco Dijkstra wrote: > > This patch fixes the shift costs for Cortex-A53 so they are more accurate - > > immediate shifts > use > > SBFM/UBFM which takes 2 cycles, register controlled shifts take 1 cycle. > > Bootstrap and > regression > > O

Re: [PATCH][AArch64] Fix aarch64_rtx_costs of PLUS/MINUS

2015-05-01 Thread Kyrill Tkachov
On 01/05/15 11:22, Marcus Shawcroft wrote: On 4 March 2015 at 15:37, Wilco Dijkstra wrote: Include the cost of op0 and op1 in all cases in PLUS and MINUS in aarch64_rtx_costs. Bootstrap & regression OK. ChangeLog: 2015-03-04 Wilco Dijkstra * gcc/config/aarch64/aarch64.c (aarch64

Re: [PATCH][AArch64] Fix geniterators.sh to use standard BRE syntax in sed

2015-05-01 Thread Marcus Shawcroft
On 23 March 2015 at 17:06, Szabolcs Nagy wrote: > GCC can be compiled for aarch64 target with busybox sed except for > the geniterators.sh script which uses nonstandard basic regex. > > I explicitly set LC_ALL=C too because the regex depends on collation > order. > > I tested that the script gives

Re: [patch] Implement ISO/IEC TS 18822 C++ File system TS

2015-05-01 Thread Rainer Orth
Jonathan Wakely writes: > I've tested this on GNU/Linux and DragonFly BSD, but as it's probably > not going to build everywhere I've added the configure option > --enable-libstdcxx-filesystem-ts which defaults to enabled on GNU, BSD > and Solaris targets, and disabled elsewhere for now. If it fai

Re: Mostly rewrite genrecog

2015-05-01 Thread Richard Sandiford
Richard Sandiford writes: > Richard Biener writes: >> On Thu, Apr 30, 2015 at 2:08 PM, Andreas Schwab >> wrote: >>> Richard Sandiford writes: >>> Andreas Schwab writes: > Richard Sandiford writes: > >> /* Represents a test and the action that should be taken on the result. >

Re: [PATCH][AArch64] Handle FLOAT and UNSIGNED_FLOAT in rtx costs

2015-05-01 Thread Marcus Shawcroft
On 1 May 2015 at 09:21, Kyrill Tkachov wrote: > 2015-05-01 Kyrylo Tkachov > > * config/aarch64/aarch64.c (aarch64_rtx_costs): Handle FLOAT and > UNSIGNED_FLOAT. OK /Marcus

Re: [ARM,AArch64][testsuite] AdvSIMD intrinsics tests cleanup: remove useless expected values.

2015-05-01 Thread Marcus Shawcroft
On 30 April 2015 at 20:38, Christophe Lyon wrote: > This is a cleanup of the series of tests I added some time ago. > > During the latest reviews, I got comments about the fact that some > intrinsics do not support all the vector types but the corresponding > tests would still contain dummy expect

Re: [PATCH][AArch64] Make aarch64_min_divisions_for_recip_mul configurable

2015-05-01 Thread Marcus Shawcroft
On 1 May 2015 at 12:26, Wilco Dijkstra wrote: > > >> Marcus Shawcroft wrote: >> On 27 April 2015 at 14:43, Wilco Dijkstra wrote: >> >> >> static unsigned int >> >> -aarch64_min_divisions_for_recip_mul (enum machine_mode mode >> >> ATTRIBUTE_UNUSED) >> >> +aarch64_min_divisions_for_recip_mul (en

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

2015-05-01 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.1.0.sv.po',

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

2015-05-01 Thread Translation Project Robot
cpplib-5.1.0.sv.po.gz Description: Binary data The Translation Project robot, in the name of your translation coordinator.

Re: [PATCH, AArch64] Add Cortex-A53 erratum 843419 configure-time option

2015-05-01 Thread Yvan Roux
Hi Marcus, (Sorry wanted to cc you in my first mail but seems that gmail prefers Maxim to Marcus ! ;) On 1 May 2015 at 13:11, Marcus Shawcroft wrote: > On 1 May 2015 at 10:11, Yvan Roux wrote: >> Hi all, >> >> As described in the thread bellow, there is a link-time workaround for >> an erratum

[PATCH] Mark help string in DEFPARAM as no-c-format

2015-05-01 Thread Andreas Schwab
Tested with make gcc.pot. Andreas. PR translation/65959 * params.h (DEFPARAM): Rename msgid to nocmsgid. --- gcc/params.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/params.h b/gcc/params.h index 28d077f..f53426d 100644 --- a/gcc/params.h +++ b/gcc/par

RE: [PATCH][AArch64] Make aarch64_min_divisions_for_recip_mul configurable

2015-05-01 Thread Wilco Dijkstra
> Marcus Shawcroft wrote: > On 27 April 2015 at 14:43, Wilco Dijkstra wrote: > > >> static unsigned int > >> -aarch64_min_divisions_for_recip_mul (enum machine_mode mode > >> ATTRIBUTE_UNUSED) > >> +aarch64_min_divisions_for_recip_mul (enum machine_mode mode) > >> { > >> - return 2; > >> +

Re: [PATCH, AArch64] Add Cortex-A53 erratum 843419 configure-time option

2015-05-01 Thread Marcus Shawcroft
On 1 May 2015 at 10:11, Yvan Roux wrote: > Hi all, > > As described in the thread bellow, there is a link-time workaround for > an erratum (843419) of some early revision of Cortex-A53. Similarly > to what was done for a previous erratum, this patch adds a new > configure-time option --enable-fix

Re: [PATCH] [libstdc++] Add uniform container erasure.

2015-05-01 Thread Jonathan Wakely
On 01/05/15 07:01 -0400, Ed Smith-Rowland wrote: On 04/30/2015 02:01 PM, Daniel Krügler wrote: Shouldn't the "one-liner" forwarding function templates be declared as inline? - Daniel You are right. This builds and tests clean on x86_64-linux. OK? Yes OK, thanks.

[patch] Update libstdc++ status docs, add C++17 status

2015-05-01 Thread Jonathan Wakely
This updates the C++11 and C++14/TS status tables, and adds a new table for C++17 features. Committed to trunk. commit c95071e70eaa69a47ae49c99c8ddb8d20c2e9085 Author: Jonathan Wakely Date: Thu Apr 30 21:35:55 2015 +0100 * doc/xml/manual/intro.xml: Link to new status_cxx2017.xml file.

[patch] Implement std::pointer_safety etc.

2015-05-01 Thread Jonathan Wakely
I forgot to add these (useless) functions that are required for full C++11 conformance. Tested powerpc64le-linux, committed to trunk. I will probably backport these to 5.2 as well. commit 95d0e38e22b85e6ffeea15678c6dce114c029be7 Author: Jonathan Wakely Date: Tue Jan 20 12:14:11 2015 +

Re: [PATCH] [libstdc++] Add uniform container erasure.

2015-05-01 Thread Ed Smith-Rowland
On 04/30/2015 02:01 PM, Daniel Krügler wrote: Shouldn't the "one-liner" forwarding function templates be declared as inline? - Daniel You are right. This builds and tests clean on x86_64-linux. OK? 2015-05-01 Edward Smith-Rowland <3dw...@verizon.net> Inline one-line erasure dispat

Re: [patch] Perform anonymous constant propagation during inlining

2015-05-01 Thread Eric Botcazou
> Hmm, special-casing this in the inliner looks odd to me. ISTR the inliner > already propagates constant parameters to immediate uses, so I guess > you run into the casting case you handle specially. Right on both counts, the original GIMPLE looks like: right.3 = (system__storage_elements__in

Re: [PATCH][AArch64] Fix aarch64_rtx_costs of PLUS/MINUS

2015-05-01 Thread Marcus Shawcroft
On 4 March 2015 at 15:37, Wilco Dijkstra wrote: > Include the cost of op0 and op1 in all cases in PLUS and MINUS in > aarch64_rtx_costs. > Bootstrap & regression OK. > > ChangeLog: > 2015-03-04 Wilco Dijkstra > > * gcc/config/aarch64/aarch64.c (aarch64_rtx_costs): > Calculate c

Re: [PATCH][AARCH64]Use mov for add with large immediate.

2015-05-01 Thread Marcus Shawcroft
On 21 April 2015 at 17:10, Renlin Li wrote: > Hi all, > > This is a simple patch to generate a move instruction to temporarily hold > the large immediate for a add instruction. > > GCC regression test has been run using aarch64-none-elf toolchain. NO new > issues. > > Okay for trunk? > > Regards,

Re: [libstdc++ PATCH] Implement __is_nothrow_swappable and use it

2015-05-01 Thread Ville Voutilainen
On 1 May 2015 at 12:47, Ville Voutilainen wrote: > This patch partially solves the problem described in N4426, which is > basically LWG > issue 2456, which in turn is caused by CWG DR 1330. Some remarks: ..and I forgot to mention that it fixes PR 63860.

[libstdc++ PATCH] Implement __is_nothrow_swappable and use it

2015-05-01 Thread Ville Voutilainen
This patch partially solves the problem described in N4426, which is basically LWG issue 2456, which in turn is caused by CWG DR 1330. Some remarks: - the __is_swappable and __is_nothrow_swappable are at this time not meant to be general traits, they make the shortcut of automatically transforming

[PATCH] Fix OpenACC shutdown and PTX image unloading (PR65904)

2015-05-01 Thread Julian Brown
Hi, This patch fixes PR65904, a double-free error that started occurring after recent libgomp changes to the way offload images are registered with the runtime. Offload images now map all functions/data using just two malloc'ed blocks, but the function gomp_free_memmap did not take that into acco

Re: [PATCH] [AArch32] Additional bics patterns.

2015-05-01 Thread Kyrill Tkachov
On 24/04/15 16:41, Alex Velenko wrote: Hi, This patch adds rtl patterns to generate bics instructions with shift. Added attribute predicable_short_it since last respin. Done full regression run on arm-none-eabi and arm-none-gnueabihf. Bootstrapped on arm-none-gnueabihf. Is this patch ok? gc

  1   2   >