Re: Move remaining flag_unsafe_math_optimizations using simplify and match

2015-08-27 Thread Marc Glisse
On Thu, 27 Aug 2015, Andreas Schwab wrote: "Hurugalawadi, Naveen" writes: * fold-const.c (fold_binary_loc) : Move Optimize root(x)*root(y) as root(x*y) to match.pd. Move Optimize expN(x)*expN(y) as expN(x+y) to match.pd. Move Optimize pow(x,y)*pow(x,z) as pow(x

[Patch, libstdc++] Add specific error message into exceptions

2015-08-27 Thread Tim Shen
Bootstrapped and tested. Thanks! -- Regards, Tim Shen commit 53c1caff442e97a18652ec8b1d984341168fd98d Author: Tim Shen Date: Thu Aug 27 21:42:40 2015 -0700 PR libstdc++/67361 * include/bits/regex_error.h: Add __throw_regex_error that supports string. * includ

Re: [patch] libstdc++/67374 declare valarray begin/end overloads in

2015-08-27 Thread Kohei Takahashi
On 2015/08/28 4:54, Jonathan Wakely wrote: > On 27/08/15 21:48 +0200, Marc Glisse wrote: >> On Thu, 27 Aug 2015, Jonathan Wakely wrote: >> >>> The std::cbegin function in can't call the >>> std::begin function in unless it knows about it, so we need >>> to declare the valarray overloads in . >>

Re: C++ delayed folding branch review

2015-08-27 Thread Jason Merrill
On 08/27/2015 02:12 PM, Kai Tietz wrote: + else if (TREE_CODE (type) == VECTOR_TYPE) +{ + if (TREE_CODE (arg1) == VECTOR_CST + && code == NOP_EXPR + && TYPE_VECTOR_SUBPARTS (type) == VECTOR_CST_NELTS (arg1)) + { + tree r = copy_node (arg1); + TREE_T

Re: C++ delayed folding branch review

2015-08-27 Thread Jason Merrill
On 08/27/2015 09:38 AM, Kai Tietz wrote: 2015-08-27 15:27 GMT+02:00 Jason Merrill : On 08/27/2015 06:39 AM, Kai Tietz wrote: 2015-08-27 4:56 GMT+02:00 Jason Merrill : On 08/24/2015 03:15 AM, Kai Tietz wrote: 2015-08-03 17:39 GMT+02:00 Jason Merrill : On 08/03/2015 05:42 AM, Kai Tietz wro

Re: [PATCH GCC][rework]Improve loop bound info by simplifying conversions in iv base

2015-08-27 Thread Bin.Cheng
On Thu, Aug 27, 2015 at 6:54 PM, Ajit Kumar Agarwal wrote: > > > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On > Behalf Of Bin Cheng > Sent: Thursday, August 27, 2015 3:12 PM > To: gcc-patches@gcc.gnu.org > Subject: [PATCH GCC][rework]I

Go patch committed: report unused vars initialized to function literals

2015-08-27 Thread Ian Lance Taylor
This patch by Chris Manghane fixes an oversight in the Go compiler that caused it to fail to report unused variables initialized to function literals. This fixes https://golang.org/issue/12317 . Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian Index: gcc/g

[PATCH] Make ubsan tests less picky about ansi escape codes in diagnostics.

2015-08-27 Thread Jonathan Roelofs
The attached patch makes the ubsan tests agnostic to ansi escape codes in their diagnostic output. Tested on an x86_64-linux-gnu target. 2015-08-27 Jonathan Roelofs * c-c++-common/ubsan/align-2.c: Don't be picky about ansi escape codes in diagnostics. * c-c++

Go patch committed: allow multiple blank labels

2015-08-27 Thread Ian Lance Taylor
This patch by Chris Manghane fixes the Go frontend to permit multiple blank label definitions. This fixes https://golang.org/issue/12316 . Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian Index: gcc/go/gofrontend/MERGE =

Re: Move remaining flag_unsafe_math_optimizations using simplify and match

2015-08-27 Thread Andreas Schwab
"Hurugalawadi, Naveen" writes: > * fold-const.c (fold_binary_loc) : Move Optimize > root(x)*root(y) as root(x*y) to match.pd. > Move Optimize expN(x)*expN(y) as expN(x+y) to match.pd. > Move Optimize pow(x,y)*pow(x,z) as pow(x,y+z) to match.pd. > Move Optimize a/roo

Re: [PATCH, testsuite] Enable vect_double for PowerPC targets with VSX support

2015-08-27 Thread Jeff Law
On 08/27/2015 12:35 PM, Bill Schmidt wrote: Hi, I observed today that { dg-require-effective-target vect_double } was unintentionally excluding all powerpc* targets. This patch corrects the situation by enabling vect_double when the VSX architectural support is present, which provides the vecto

Re: [PATCH] PR66870 PowerPC64 Enable gold linker with split stack

2015-08-27 Thread Lynn A. Boger
Here is an updated patch, with a summary of the differences from my previous patch: - In my previous patch gcc configure was verifying the gold linker even if it was the default linker, but that is not necessary since in that case -fuse-ld=gold does not need to be set. Gold version checking is

Re: [PATCH 1/2] remove -floop-unroll-and-jam

2015-08-27 Thread Jeff Law
On 08/27/2015 09:07 AM, Sebastian Paul Pop wrote: unroll-and-jam is new in GCC 6, right? So you can remove the option completely. The reason I left the flag around is that it was added in GCC 5: Sounds right to me then. jeff

Re: Fixing sra-12.c

2015-08-27 Thread Jeff Law
On 08/27/2015 10:48 AM, Alan Lawrence wrote: Jeff Law wrote: diff --git a/gcc/testsuite/gcc.dg/tree-ssa/sra-15.c b/gcc/testsuite/gcc.dg/tree-ssa/sra-15.c new file mode 100644 index 000..e251058 --- /dev/null +++ b/gcc/testsuite/gcc.dg/tree-ssa/sra-15.c @@ -0,0 +1,38 @@ +/* Verify that SRA

Pass -foffload targets from driver to libgomp at link time

2015-08-27 Thread Joseph Myers
This patch, a version of cleaned up for trunk, arranges for the -foffload= targets specified at link time to be passed to libgomp via a constructor function generated by the driver. In this patch, I've tried to remove all the miscellaneous

[patch] fix bootstrap if no ISL is available.

2015-08-27 Thread Andreas Tobler
Hi all, I think this is obvious? Thanks, Andreas 2015-08-27 Andreas Tobler * toplev.c (process_options): Remove flag_loop_block, flag_loop_interchange, flag_loop_strip_mine. Index: toplev.c === --- toplev.c

[PATCH COMMITTED] MAINTAINERS (Write After Approval): Add myself.

2015-08-27 Thread Patrick Marlier
With all things going on to convert the repo I noticed that my name was not in the list. Fixed. Committed as obvious. Thanks. Index: ChangeLog === --- ChangeLog (revision 227278) +++ ChangeLog (working copy) @@ -1,3 +1,7 @@ +2

[wwwdocs] Adjust a couple of links to golang.org

2015-08-27 Thread Gerald Pfeifer
Applied. Gerald Index: gcc-4.7/changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.7/changes.html,v retrieving revision 1.140 diff -u -r1.140 changes.html --- gcc-4.7/changes.html6 Dec 2014 16:29:23 - 1.140 +++

Re: [patch] libstdc++/67374 declare valarray begin/end overloads in

2015-08-27 Thread Jonathan Wakely
On 27/08/15 21:48 +0200, Marc Glisse wrote: On Thu, 27 Aug 2015, Jonathan Wakely wrote: The std::cbegin function in can't call the std::begin function in unless it knows about it, so we need to declare the valarray overloads in . Hmm, do you remember why valarray has an ugly overload of std

[wwwdocs] gmplib.org now uses https (and reduce the number of links there)

2015-08-27 Thread Gerald Pfeifer
Applied. Gerald Index: news.html === RCS file: /cvs/gcc/wwwdocs/htdocs/news.html,v retrieving revision 1.146 diff -u -r1.146 news.html --- news.html 11 Apr 2015 12:27:05 - 1.146 +++ news.html 27 Aug 2015 19:47:28 - @

Re: [patch] libstdc++/67374 declare valarray begin/end overloads in

2015-08-27 Thread Marc Glisse
On Thu, 27 Aug 2015, Jonathan Wakely wrote: The std::cbegin function in can't call the std::begin function in unless it knows about it, so we need to declare the valarray overloads in . Hmm, do you remember why valarray has an ugly overload of std::begin instead of a nice member function be

[wwwdocs] www.cilkplus.org now uses https

2015-08-27 Thread Gerald Pfeifer
Applied. Gerald Index: svn.html === RCS file: /cvs/gcc/wwwdocs/htdocs/svn.html,v retrieving revision 1.201 diff -u -r1.201 svn.html --- svn.html10 Apr 2015 21:53:06 - 1.201 +++ svn.html27 Aug 2015 19:46:45 - @@ -

Go patch committed: Don't record dependencies of redefinitions

2015-08-27 Thread Ian Lance Taylor
This patch by Chris Manghane fixes a Go frontend crash on invalid code that redefines a variable. This fixes https://golang.org/issue/11543 . Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian Index: gcc/go/gofrontend/MERGE =

[patch] libstdc++/67374 declare valarray begin/end overloads in

2015-08-27 Thread Jonathan Wakely
The std::cbegin function in can't call the std::begin function in unless it knows about it, so we need to declare the valarray overloads in . I considered adding a header with the declarations, but I don't think that's worth it, so I'm just declaring them directly in range_access.h Tested pow

Re: [PATCH 0/2] Final cleanup in move to ISL

2015-08-27 Thread Sebastian Pop
Tobias Grosser wrote: > From the graphite side, this is right. One thing I am not sure about if we > need to > keep these flags as 'do-nothing' flags to meet certain backward compatibility > guarantees in gcc. The patch keeps all the flags and makes them alias of a single flag -floop-nest-optimiz

[PATCH, i386]: Fix PR67317, silly code generation for _addcarry/_subborrow intrinsics

2015-08-27 Thread Uros Bizjak
Hello! Attached patch improves code generation for _addcarry_* and _subborrow_* intrinsics considerably. For the following testcase: --cut here-- typedef unsigned long long u64; u64 testcarry_u64 (u64 a, u64 b, u64 c, u64 d) { u64 result0, result1; __builtin_ia32_addcarryx_u64 (__builtin_i

[PATCH, testsuite] Enable vect_double for PowerPC targets with VSX support

2015-08-27 Thread Bill Schmidt
Hi, I observed today that { dg-require-effective-target vect_double } was unintentionally excluding all powerpc* targets. This patch corrects the situation by enabling vect_double when the VSX architectural support is present, which provides the vector double type. As a result of this change, I

[PATCH][C++] Fix PR67333

2015-08-27 Thread Mikhail Maltsev
Hi. This patch fixes a rejects-valid bug related to volatile-qualified arguments of constexpr functions. This is essentially a one-line change in constexpr.c. Bootstrapped and regtested on x86_64-pc-linux-gnu. OK for trunk? gcc/cp/ChangeLog: 2015-08-25 Mikhail Maltsev PR c++/67333

[wwwdocs] Adjust links to ARC simulator

2015-08-27 Thread Gerald Pfeifer
Only a minor change, and github redirects, but since it's a permanent redirect, we might as well adjust our links. Applied. Gerald Index: backends.html === RCS file: /cvs/gcc/wwwdocs/htdocs/backends.html,v retrieving revision 1.66 d

Re: C++ delayed folding branch review

2015-08-27 Thread Kai Tietz
With following patch testcase passes: Index: fold-const.c === --- fold-const.c(Revision 227111) +++ fold-const.c(Arbeitskopie) @@ -2110,6 +2110,17 @@ fold_convert_const (enum tree_code code, tree type else if (T

Re: [gomp4, wip] remove references to ganglocal shared memory inside gcc

2015-08-27 Thread Nathan Sidwell
On 08/27/15 13:25, Cesar Philippidis wrote: On 08/27/2015 06:13 AM, Nathan Sidwell wrote: I'll create a follow up patch for that later, probably after I finish ok working on the auto-independent loop patch. In the meantime, I'm found a bug where acc routine calls aren't being checked for c

Re: [c++-delayed-folding] fold_simple

2015-08-27 Thread Richard Biener
On August 27, 2015 6:07:59 PM GMT+02:00, Kai Tietz wrote: >2015-08-27 12:34 GMT+02:00 Richard Biener : >> On Thu, Aug 27, 2015 at 11:21 AM, Kai Tietz >wrote: >>> 2015-08-27 4:57 GMT+02:00 Jason Merrill : Why does fold_simple fold so many patterns? I thought we wanted >something that w

Re: [gomp4, wip] remove references to ganglocal shared memory inside gcc

2015-08-27 Thread Cesar Philippidis
On 08/27/2015 06:13 AM, Nathan Sidwell wrote: > On 08/26/15 21:37, Cesar Philippidis wrote: >> This patch strips out all of the references to ganglocal memory in gcc. >> Unfortunately, the runtime api still takes a shared memory parameter, so >> I haven't made any changes there yet. Perhaps we coul

Re: [PATCH : RL78] Disable interrupts during hardware multiplication routines

2015-08-27 Thread DJ Delorie
> I have worked out an updated patch, which would save the MDUC specific > registers > in the interrupt routine when the option '-msave-mduc-in-interrupts' is > passed. > This gets active only for the G13 targets. Perhaps we should have both a -msave... and -mno-save... (gcc provides these by d

Re: [PATCH : RL78] Disable interrupts during hardware multiplication routines

2015-08-27 Thread Mike Stump
On Aug 27, 2015, at 6:13 AM, Kaushik Phatak wrote:Hi DJ, > I have worked out an updated patch, which would save the MDUC specific > registers > in the interrupt routine when the option '-msave-mduc-in-interrupts' is > passed. > This gets active only for the G13 targets. So, I may have missed t

Re: [PATCH] Fixup loops in DCE (PR middle-end/67005)

2015-08-27 Thread Richard Biener
On August 27, 2015 5:24:55 PM GMT+02:00, Marek Polacek wrote: >The fix for this PR is to also schedule fixup of loops even if we're >removing >an entry into an irreducible region -- that might scramble the loop >structure >so we need to be careful. > >Bootstrapped/regtested on x86_64-linux, ok fo

Fixing sra-12.c (was: Re: [PATCH 2/5] completely_scalarize arrays as well as records)

2015-08-27 Thread Alan Lawrence
Jeff Law wrote: diff --git a/gcc/testsuite/gcc.dg/tree-ssa/sra-15.c b/gcc/testsuite/gcc.dg/tree-ssa/sra-15.c new file mode 100644 index 000..e251058 --- /dev/null +++ b/gcc/testsuite/gcc.dg/tree-ssa/sra-15.c @@ -0,0 +1,38 @@ +/* Verify that SRA total scalarization works on records containi

Re: [patch] Remove unsupported hosts/builds from gcc/config.*

2015-08-27 Thread FX
> This patch was committed without regenerating gcc/configure. Thanks for spotting that. Committed regenerated gcc/configure as revision 227266. FX

Re: [c++-delayed-folding] fold_simple

2015-08-27 Thread Kai Tietz
2015-08-27 12:34 GMT+02:00 Richard Biener : > On Thu, Aug 27, 2015 at 11:21 AM, Kai Tietz wrote: >> 2015-08-27 4:57 GMT+02:00 Jason Merrill : >>> Why does fold_simple fold so many patterns? I thought we wanted something >>> that would just fold conversions and negations of constant values. >> >>

Re: [Patch, fortran] F2008 - implement pointer function assignment

2015-08-27 Thread Paul Richard Thomas
Dear FX and Mikael, Please hold off on reviewing the patch until I say so. Some rather trivial looking tidying up before submission has introduced a dozen or so regressions :-( Cheers Paul On 27 August 2015 at 16:54, FX wrote: > Hi Paul, > > I’ll get to reviewing the patch tonight or tomorrow…

Re: [patch] Remove unsupported hosts/builds from gcc/config.*

2015-08-27 Thread Lynn A. Boger
This patch was committed without regenerating gcc/configure. On 08/21/2015 03:10 AM, FX wrote: Attached patch removes special cases for the following unsupported targets as build and host environments: - uwin* - i370-*-opened* | i370-*-mvs* - i[34567]86-*-sco3.2v5* - i[34567]86-sequent-ptx4* |

Re: [PATCH 2/5] completely_scalarize arrays as well as records

2015-08-27 Thread Alan Lawrence
Martin Jambor wrote: > > First, I would be much > happier if you added a proper comment to scalarize_elem function which > you forgot completely. The name is not very descriptive and it has > quite few parameters too. > > Second, this patch should also fix PR 67283. It would be great if you > cou

Re: [Patch, fortran] F2008 - implement pointer function assignment

2015-08-27 Thread Mikael Morin
Le 27/08/2015 16:54, FX a écrit : I think it shouldn’t be too hard to get the ambiguity lifted. We already can do it for array assignment vs. statement functions, so maybe we could get a hint from that? Well, for array assignments, the array is declared before, whereas for pointer functions,

Re: [PATCH 1/5] Refactor completely_scalarize_var

2015-08-27 Thread Alan Lawrence
Martin Jambor wrote: > > If you change what the function does, you have to change the comment > too. If I am not mistaken, even with the whole patch set applied, the > first sentence would still be: "Create total_scalarization accesses > for all scalar type fields in VAR and for VAR as a whole." A

[PATCH] Fixup loops in DCE (PR middle-end/67005)

2015-08-27 Thread Marek Polacek
The fix for this PR is to also schedule fixup of loops even if we're removing an entry into an irreducible region -- that might scramble the loop structure so we need to be careful. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2015-08-27 Marek Polacek PR middle-end/67005

Patch tree-profile.c to support profile-func-internal-id parameter

2015-08-27 Thread Matt Deeds
This patch is for svn://gcc.gnu.org/svn/gcc/branches/google/gcc-4_9. I add support for the profile_func_internal-id in the instrumentation generated for __coverage_callback. Add support for the profile-func-internal-id parameter to the coverage callback. Without this change, the function identifi

Re: [gomp-4.1] fix incorrect memory size in goacc_new_thread

2015-08-27 Thread Aldy Hernandez
On 08/25/2015 10:44 AM, Aldy Hernandez wrote: This is either blatantly wrong or subtly correct, in which case it needs a comment. My guess is the former. OK for branch? Hi Thomas. Jakub has pre-approved this patch. I would like to commit it to trunk. Do you object? Would you like to test

RE: [PATCH 1/2] remove -floop-unroll-and-jam

2015-08-27 Thread Sebastian Paul Pop
> unroll-and-jam is new in GCC 6, right? So you can remove the option > completely. The reason I left the flag around is that it was added in GCC 5: commit 5240f96a55db98b04275fda34c891fbbc629b825 Author: mircea Date: Mon Nov 17 22:59:07 2014 + Support for unroll and jam optimizatio

Re: Fix order of ENTRY and EXIT in reverse post order

2015-08-27 Thread Richard Biener
On Thu, Aug 27, 2015 at 4:33 PM, Michael Matz wrote: > Hello, > > I've this in my tree since some time already. In-tree there is only one > user of pre_and_rev_post_order_compute{,_fn} that actually wants entry and > exit included, and that one is just a debug routine > (draw_cfg_nodes_no_loops);

Re: [gomp4.1] document more things

2015-08-27 Thread Aldy Hernandez
On 08/27/2015 07:14 AM, Jakub Jelinek wrote: On Thu, Aug 27, 2015 at 07:01:48AM -0700, Aldy Hernandez wrote: commit 45fdbc84123fefde6bf99aaf87099317aaa38f24 Author: Aldy Hernandez Date: Wed Aug 26 07:28:39 2015 -0700 * libgomp.h (enum gomp_task_kind): Comment fields. (struct

Re: [Patch, fortran] F2008 - implement pointer function assignment

2015-08-27 Thread FX
Hi Paul, I’ll get to reviewing the patch tonight or tomorrow… I think it shouldn’t be too hard to get the ambiguity lifted. We already can do it for array assignment vs. statement functions, so maybe we could get a hint from that? touille /tmp $ cat a.f90 integer :: f(2) = 0 integer :: x =

Fix order of ENTRY and EXIT in reverse post order

2015-08-27 Thread Michael Matz
Hello, I've this in my tree since some time already. In-tree there is only one user of pre_and_rev_post_order_compute{,_fn} that actually wants entry and exit included, and that one is just a debug routine (draw_cfg_nodes_no_loops); so this bug right now is harmless. But I've used this for s

Re: [gomp4.1] document more things

2015-08-27 Thread Jakub Jelinek
On Thu, Aug 27, 2015 at 07:01:48AM -0700, Aldy Hernandez wrote: > commit 45fdbc84123fefde6bf99aaf87099317aaa38f24 > Author: Aldy Hernandez > Date: Wed Aug 26 07:28:39 2015 -0700 > > * libgomp.h (enum gomp_task_kind): Comment fields. > (struct gomp_task): Comment some fields. >

Re: [RFC AArch64][PR 63304] Handle literal pools for functions > 1 MiB in size.

2015-08-27 Thread Marcus Shawcroft
On 27 July 2015 at 15:33, Ramana Radhakrishnan wrote: > Ramana Radhakrishnan > > PR target/63304 > * config/aarch64/aarch64.c (aarch64_expand_mov_immediate): Handle > nopcrelative_literal_loads. > (aarch64_classify_address): Likewise. > (aarch64_constan

[gomp4.1] document more things

2015-08-27 Thread Aldy Hernandez
While I'm here... I've added more comments throughout. I'm still not convinced we need anything but GOMP_TASK_WAITING in enum gomp_task_kind, but perhaps you needed it for future usage? If so, could you provide a description for GOMP_TASK_IFFALSE? OK for branch? commit 45fdbc84123fefde6bf99a

Re: [PATCH] Fix and simplify (Re: Fix libbacktrace -fPIC breakage from "Use libbacktrace in libgfortran")

2015-08-27 Thread Ian Lance Taylor
"Ulrich Weigand" writes: > Ian Lance Taylor wrote: >> "Ulrich Weigand" writes: >> >> > I've verified that this works on x86_64: the resulting >> > libgfortran.so uses the -fPIC version of the libbacktrace >> > object, while libgfortran.a uses the non-PIC versions. >> > >> > On SPU, libtool will

Re: [AArch64][TLSLE][3/3] Implement local executable mode for all memory model

2015-08-27 Thread Christophe Lyon
On 27 August 2015 at 15:35, Jiong Wang wrote: > > Christophe Lyon writes: > >> On 27 August 2015 at 12:03, Jiong Wang wrote: >>> >>> Jiong Wang writes: >> >> Those relocation types required by tls-size 12 & 24 are supported by >> binutils-2.25 already, and you have passed compilation

[PATCH 3/3, libgomp] intelmic specific parts

2015-08-27 Thread Chung-Lin Tang
These are the intelmic plugin specific parts (actually beneath liboffloadmic instead of libgomp). The changes are basically to expose the return value of offload() back to libgomp. I only checked parts of the plugin, it appears that there may still be code in the liboffloadmic runtime that can cal

[PATCH 2/3, libgomp] nvptx plugin parts

2015-08-27 Thread Chung-Lin Tang
These are the nvptx plugin specific parts. Chung-Lin * plugin/plugin-nvptx.c (CUDA_CALL_ERET): New convenience macro. (CUDA_CALL): Likewise. (CUDA_CALL_ASSERT): Likewise. (map_init): Change return type to bool, use CUDA_CALL* macros. (map_fini): Likewise.

[PATCH 1/3, libgomp] Adjust offload plugin interface for avoiding deadlock on exit

2015-08-27 Thread Chung-Lin Tang
We've discovered that, for several of the libgomp plugin interface routines, if the target specific routine calls exit() (usually upon a fatal condition), deadlock ensues. We found this using nvptx, but it's possible on intelmic as well. This is due to many of the plugin routines are called with

Re: C++ delayed folding branch review

2015-08-27 Thread Kai Tietz
2015-08-27 15:27 GMT+02:00 Jason Merrill : > On 08/27/2015 06:39 AM, Kai Tietz wrote: >> >> 2015-08-27 4:56 GMT+02:00 Jason Merrill : >>> >>> On 08/24/2015 03:15 AM, Kai Tietz wrote: 2015-08-03 17:39 GMT+02:00 Jason Merrill : > > On 08/03/2015 05:42 AM, Kai Tietz wrote: >> >>>

Re: [PATCH] Fix and simplify (Re: Fix libbacktrace -fPIC breakage from "Use libbacktrace in libgfortran")

2015-08-27 Thread Ulrich Weigand
Ian Lance Taylor wrote: > "Ulrich Weigand" writes: > > > I've verified that this works on x86_64: the resulting > > libgfortran.so uses the -fPIC version of the libbacktrace > > object, while libgfortran.a uses the non-PIC versions. > > > > On SPU, libtool will now automatically only generate the

[PATCH][C++] Avoid PCH dependent mangling

2015-08-27 Thread Richard Biener
With the passes.c hunk in the patch below we FAIL assembly comparison of g++.dg/pch/system-[12].C because with PCH we have computed DECL_ASSEMBLER_NAME and thus appended DW_AT_linkage_name early during PCH generation while without PCH we compute it lazily and end up appending DW_AT_specification e

Re: [AArch64][TLSLE][3/3] Implement local executable mode for all memory model

2015-08-27 Thread Jiong Wang
Christophe Lyon writes: > On 27 August 2015 at 12:03, Jiong Wang wrote: >> >> Jiong Wang writes: > > Those relocation types required by tls-size 12 & 24 are supported by > binutils-2.25 already, and you have passed compilation and failed at > exectuion, so there do have something

Re: C++ delayed folding branch review

2015-08-27 Thread Jason Merrill
On 08/27/2015 06:39 AM, Kai Tietz wrote: 2015-08-27 4:56 GMT+02:00 Jason Merrill : On 08/24/2015 03:15 AM, Kai Tietz wrote: 2015-08-03 17:39 GMT+02:00 Jason Merrill : On 08/03/2015 05:42 AM, Kai Tietz wrote: 2015-08-03 5:49 GMT+02:00 Jason Merrill : On 07/31/2015 05:54 PM, Kai Tietz wrote:

Re: [PATCH, RS6000] Correct constraints for ior_mask

2015-08-27 Thread David Edelsohn
On Thu, Aug 27, 2015 at 9:02 AM, Alan Modra wrote: > ior_mask always splits to rtl matching rotl3_insert. The > latter requires one of its input operands match the output, so let > reload/lra know the equivalent operand match must also occur for > ior_mask. > > Bootstrapped and regression tested

Re: [AArch64][TLSLE][3/3] Implement local executable mode for all memory model

2015-08-27 Thread Christophe Lyon
On 27 August 2015 at 12:03, Jiong Wang wrote: > > Jiong Wang writes: Those relocation types required by tls-size 12 & 24 are supported by binutils-2.25 already, and you have passed compilation and failed at exectuion, so there do have something wrong I guess. Either t

RE: [PATCH : RL78] Disable interrupts during hardware multiplication routines

2015-08-27 Thread Kaushik Phatak
Hi DJ, Thanks for your feedback comments. Sorry it took me a while to get back on this. >> Have you compared the latency of the multiply instructions to the overhead of >> saving those registers in the interrupt handler? >> What about the case where performance is priority, and the developer knows

Re: [gomp4, wip] remove references to ganglocal shared memory inside gcc

2015-08-27 Thread Nathan Sidwell
On 08/26/15 21:37, Cesar Philippidis wrote: This patch strips out all of the references to ganglocal memory in gcc. Unfortunately, the runtime api still takes a shared memory parameter, so I haven't made any changes there yet. Perhaps we could still keep the shared memory argument to GOACC_parall

[PATCH, RS6000] Correct constraints for ior_mask

2015-08-27 Thread Alan Modra
ior_mask always splits to rtl matching rotl3_insert. The latter requires one of its input operands match the output, so let reload/lra know the equivalent operand match must also occur for ior_mask. Bootstrapped and regression tested powerpc64le-linux. OK to apply? PR target/67356

RE: [PATCH 1/4] [MIPS] Add support for MIPS SIMD Architecture (MSA)

2015-08-27 Thread Matthew Fortune
Hi Robert, I'm taking small steps at this review. I did the first few files as below. mips-msa.md is next so don't expect the it quickly. I might do that in pieces too. No need to post an updated patch until I'm all the way through but I'd appreciate an explicit ok/done to each point or discussio

[PATCH] Guard early debug generation with !seen_errors ()

2015-08-27 Thread Richard Biener
This is already done for the TYPE_DECL case, the following avoids trying to generate any debug after we've seen errors for the calls to early_global_decl. Bootstrapped on x86_64-unknown-linux-gnu, testing and gdb testing in progress. Richard. 2015-08-27 Richard Biener * passes.c (r

Re: [patch][libstdc++] Provide nearly complete locale support for DragonFly

2015-08-27 Thread John Marino
On 8/27/2015 2:26 PM, Jonathan Wakely wrote: > > N.B. please CC the libstdc++ list on all libstdc++ patches. > The tests on DragonFly look excellent now, nice work. Okay, noted for the future. > I think that should probably be defined on FreeBSD too now, so that > the _GLIBCXX_USE_C99_DYNAMIC ch

Re: [patch][libstdc++] Provide nearly complete locale support for DragonFly

2015-08-27 Thread Jonathan Wakely
On 13/08/15 15:13 +0200, John Marino wrote: On 7/31/2015 6:06 PM, John Marino wrote: So far, DragonFly only handles the C/POSIX locale through the "generic" locale support files, similar to FreeBSD and Darwin. The following patchset enables nearly complete support for named locales. The except

Re: [PATCH][4/N] Introduce new inline functions for GET_MODE_UNIT_SIZE and GET_MODE_UNIT_PRECISION

2015-08-27 Thread Oleg Endo
Hi, On 27 Aug 2015, at 16:58, David Sherwood wrote: > Hi Oleg, > > Oh I'm so sorry I broke the build - I should have spotted that. Have you > already checked in this fix or do you want me to? No, I haven't committed the patch. Please do so if it's OK. Cheers, Oleg >> -Original Messag

Re: [Patch, libstdc++/67362] Fix non-special character for POSIX basic syntax in regex

2015-08-27 Thread Jonathan Wakely
On 26/08/15 18:41 -0700, Tim Shen wrote: Bootstrapped and tested on x86_64-pc-linux-gnu. Thanks! -- Regards, Tim Shen commit e134e1a835ad15900686351cade36774593b91ea Author: Tim Shen Date: Wed Aug 26 17:51:29 2015 -0700 PR libstdc++/67362 * include/bits/regex_scanner.tcc

[PATCH] New attribute to create target clones

2015-08-27 Thread Evgeny Stupachenko
Hi All, Based on RFC: https://gcc.gnu.org/ml/gcc-patches/2015-08/msg01322.html The patch implement an extension to Function Multiversioning that allows to clone a function for multiple targets. __attribute__((target_clones("avx","arch=slm","default"))) int foo () ... Will create 3 clones of foo(

RE: [PATCH GCC][rework]Improve loop bound info by simplifying conversions in iv base

2015-08-27 Thread Ajit Kumar Agarwal
-Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On Behalf Of Bin Cheng Sent: Thursday, August 27, 2015 3:12 PM To: gcc-patches@gcc.gnu.org Subject: [PATCH GCC][rework]Improve loop bound info by simplifying conversions in iv base Hi, >>This

Re: C++ delayed folding branch review

2015-08-27 Thread Kai Tietz
2015-08-27 4:56 GMT+02:00 Jason Merrill : > On 08/24/2015 03:15 AM, Kai Tietz wrote: >> >> 2015-08-03 17:39 GMT+02:00 Jason Merrill : >>> >>> On 08/03/2015 05:42 AM, Kai Tietz wrote: 2015-08-03 5:49 GMT+02:00 Jason Merrill : > > On 07/31/2015 05:54 PM, Kai Tietz wrote: >>

Re: [c++-delayed-folding] fold_simple

2015-08-27 Thread Richard Biener
On Thu, Aug 27, 2015 at 11:21 AM, Kai Tietz wrote: > 2015-08-27 4:57 GMT+02:00 Jason Merrill : >> Why does fold_simple fold so many patterns? I thought we wanted something >> that would just fold conversions and negations of constant values. > > Yes, initial variant was handling much less pattern

Re: [AArch64][TLSLE][3/3] Implement local executable mode for all memory model

2015-08-27 Thread Jiong Wang
Jiong Wang writes: >>> >>> Those relocation types required by tls-size 12 & 24 are supported by >>> binutils-2.25 already, and you have passed compilation and failed at >>> exectuion, so there do have something wrong I guess. >>> >>> Either the generated instruction sequence or the bare-metal envi

[AArch64][TLSGD][2/2] Implement TLS GD traditional for tiny code model

2015-08-27 Thread Jiong Wang
As described this is the main implementaion patch. 2015-08-26 Jiong Wang gcc/ * configure.ac: Add check for binutils global dynamic tiny code model relocation support. * configure: Regenerate. * config.in: Regenerate. * config/aarch64/aarch64.md (tlsgd_tiny): New define_insn. * co

[AArch64][TLSGD][1/2] Remove unncessary define_expand for TLS GD traditional

2015-08-27 Thread Jiong Wang
Jiong Wang writes: > Currently, there is only small model support for TLS GD on > AArch64. While TLS Global Dynamic (Traditional) is actually the same for > all memory mode. > > For TLSGD, the code logic is always the following: > > RegA = GOT descriptor address of tls variable. > call __tls_get

Re: [AArch64][TLSLE][3/3] Implement local executable mode for all memory model

2015-08-27 Thread Jiong Wang
Christophe Lyon writes: > On 27 August 2015 at 10:35, Jiong Wang wrote: >> >> Christophe Lyon writes: >> >>> On 19 August 2015 at 16:21, Jiong Wang wrote: Marcus Shawcroft writes: > On 21 May 2015 at 17:49, Jiong Wang wrote: > >> 2015-05-14 Jiong Wang >> gcc/

[PATCH GCC][rework]Improve loop bound info by simplifying conversions in iv base

2015-08-27 Thread Bin Cheng
Hi, This is a rework for https://gcc.gnu.org/ml/gcc-patches/2015-07/msg02335.html, with review comments addressed. For now, SCEV may compute iv base in the form of "(signed T)((unsigned T)base + step))". This complicates other optimizations/analysis depending on SCEV because it's hard to dive int

Re: [AArch64][TLSLE][3/3] Implement local executable mode for all memory model

2015-08-27 Thread Christophe Lyon
On 27 August 2015 at 10:35, Jiong Wang wrote: > > Christophe Lyon writes: > >> On 19 August 2015 at 16:21, Jiong Wang wrote: >>> >>> Marcus Shawcroft writes: >>> On 21 May 2015 at 17:49, Jiong Wang wrote: > 2015-05-14 Jiong Wang > gcc/ > * config/aarch64/aarch64.c (aar

Re: [c++-delayed-folding] fold_simple

2015-08-27 Thread Kai Tietz
2015-08-27 4:57 GMT+02:00 Jason Merrill : > Why does fold_simple fold so many patterns? I thought we wanted something > that would just fold conversions and negations of constant values. Yes, initial variant was handling much less patterns. But actually we need for functions (eg. like build_vec_

Re: [AArch64][TLSLE][3/3] Implement local executable mode for all memory model

2015-08-27 Thread Jiong Wang
Christophe Lyon writes: > On 19 August 2015 at 16:21, Jiong Wang wrote: >> >> Marcus Shawcroft writes: >> >>> On 21 May 2015 at 17:49, Jiong Wang wrote: >>> 2015-05-14 Jiong Wang gcc/ * config/aarch64/aarch64.c (aarch64_print_operand): Support tls_size. * config/aarch

Re: [PATCH 2/2] remove -floop-* flags

2015-08-27 Thread Richard Biener
Looks good to me. (ChangeLog entry?) Thanks, Richard. On Thu, Aug 27, 2015 at 12:14 AM, Sebastian Pop wrote: > --- > gcc/Makefile.in|2 - > gcc/common.opt | 16 +- > gcc/doc/invoke.texi

Re: [PATCH 1/2] remove -floop-unroll-and-jam

2015-08-27 Thread Richard Biener
On Thu, Aug 27, 2015 at 12:14 AM, Sebastian Pop wrote: > --- > gcc/common.opt | 4 +- > gcc/doc/invoke.texi | 8 +- > gcc/graphite-isl-ast-to-gimple.c | 102 +- > gcc/graphite-optimize-isl.c | 179 > -

[PATCH] Make late static cdtors DECL_IGNORED_P

2015-08-27 Thread Richard Biener
I noticed that for late cdtors like those from coverage or asan and friends get DIEs created late. Of course generating debug info for them is quite pointless. Bootstrapped and tested on x86_64-unknown-linux-gnu, gdb testsuite clean, applied to trunk. Richard. 2015-08-27 Richard Biener

RE: [PATCH][4/N] Introduce new inline functions for GET_MODE_UNIT_SIZE and GET_MODE_UNIT_PRECISION

2015-08-27 Thread David Sherwood
Hi Oleg, Oh I'm so sorry I broke the build - I should have spotted that. Have you already checked in this fix or do you want me to? Regards, David. > -Original Message- > From: Oleg Endo [mailto:oleg.e...@t-online.de] > Sent: 26 August 2015 15:53 > To: Jeff Law > Cc: David Sherwood; GCC

Re: [Scalar masks 2/x] Use bool masks in if-conversion

2015-08-27 Thread Richard Biener
On Wed, Aug 26, 2015 at 5:51 PM, Ilya Enkovich wrote: > 2015-08-26 17:56 GMT+03:00 Richard Biener : >> On Wed, Aug 26, 2015 at 4:38 PM, Ilya Enkovich >> wrote: >>> 2015-08-26 16:02 GMT+03:00 Richard Biener : On Fri, Aug 21, 2015 at 2:17 PM, Ilya Enkovich wrote: > 2015-08-21 14:00

Re: [PATCH] s390: Add emit_barrier() after trap.

2015-08-27 Thread Andreas Krebbel
On 08/26/2015 04:03 PM, Dominik Vogt wrote: > This patch fixes an ICE on S390 when a trap is generated because > the given -mstack-size is to small. A barrier was missing after > the trap, so on higher optimization levels a NULL pointer fron an > uninitialized basic block was used. The patch also

Re: [AArch64][TLSLE][3/3] Implement local executable mode for all memory model

2015-08-27 Thread Christophe Lyon
On 19 August 2015 at 16:21, Jiong Wang wrote: > > Marcus Shawcroft writes: > >> On 21 May 2015 at 17:49, Jiong Wang wrote: >> >>> 2015-05-14 Jiong Wang >>> gcc/ >>> * config/aarch64/aarch64.c (aarch64_print_operand): Support tls_size. >>> * config/aarch64/aarch64.md (tlsle): Choose proper