Re: [PATCH 2/6] Introduce new edge_summary class and replace ipa_edge_args_sum.

2015-08-03 Thread Martin Liška
On 07/16/2015 04:05 PM, Martin Liška wrote: On 07/10/2015 03:31 PM, Martin Jambor wrote: Hi, thanks for working on this and sorry for a tad late review: On Thu, Jul 09, 2015 at 11:13:52AM +0200, Martin Liska wrote: gcc/ChangeLog: 2015-07-03 Martin Liska mli...@suse.cz * cgraph.c

Re: [Bug fortran/52846] [F2008] Support submodules - part 3/3

2015-08-03 Thread Mikael Morin
Le 03/08/2015 14:36, Paul Richard Thomas a écrit : Dear Mikael, Thanks for your green light! I have been mulling over the trans-decl part of the patch and having been wondering if it is necessary. You mean marking entities as public? Or setting the hidden visibility attribute? Or both? I

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

2015-08-03 Thread Kyrill Tkachov
On 03/08/15 16:45, Uros Bizjak wrote: On Mon, Aug 3, 2015 at 5:37 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Looking at the x86 movcc expansion code (ix86_expand_int_movcc) I don't think this is a good idea. In the expander, there is already quite some target-dependent code that goes

Re: [PATCH] Build *-match.o as early as possible

2015-08-03 Thread Richard Biener
On August 3, 2015 5:03:05 PM GMT+02:00, Segher Boessenkool seg...@kernel.crashing.org wrote: The two files *-match.o files always finish building last, so if we start building them as soon as possible (instead of pretty late) the total build time will be less on a parallel build. Bootstrapped

Re: [BUILDROBOT] Go runtime: calling ‘__builtin_frame_address’ with a nonzero argument is unsafe

2015-08-03 Thread Jeff Law
On 08/03/2015 08:48 AM, Martin Sebor wrote: On 08/03/2015 05:55 AM, Jan-Benedict Glaw wrote: On Sun, 2015-08-02 17:15:27 -0600, Martin Sebor mse...@gmail.com wrote: OK for the trunk. Sorry for the delay. Thank you. Committed in revision 226480. ...und breaks native builds. When doing

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

2015-08-03 Thread Kyrill Tkachov
On 03/08/15 15:15, Uros Bizjak wrote: On Mon, Aug 3, 2015 at 3:43 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: On 03/08/15 14:37, Uros Bizjak wrote: On Mon, Aug 3, 2015 at 3:02 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: On 03/08/15 13:33, Uros Bizjak wrote: Hello! 2015-07-30

Re: C++ delayed folding branch review

2015-08-03 Thread Jason Merrill
On 08/03/2015 05:42 AM, Kai Tietz wrote: 2015-08-03 5:49 GMT+02:00 Jason Merrill ja...@redhat.com: On 07/31/2015 05:54 PM, Kai Tietz wrote: The STRIP_NOPS-requirement in 'reduced_constant_expression_p' I could remove, but for one case in constexpr. Without folding we don't do

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

2015-08-03 Thread Uros Bizjak
On Mon, Aug 3, 2015 at 5:37 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Looking at the x86 movcc expansion code (ix86_expand_int_movcc) I don't think this is a good idea. In the expander, there is already quite some target-dependent code that goes great length to utilize sbb insn as much

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

2015-08-03 Thread Ilya Enkovich
2015-08-03 17:04 GMT+03:00 Uros Bizjak ubiz...@gmail.com: On Mon, Aug 3, 2015 at 3:02 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: On 03/08/15 13:33, Uros Bizjak wrote: Hello! 2015-07-30 Kyrylo Tkachov kyrylo.tkac...@arm.com * ifcvt.c (noce_try_store_flag_constants): Make logic

[patch] libstdc++/67078 change _N bad name in new container access functions

2015-08-03 Thread Jonathan Wakely
We can't use _N as an identifier, as per https://gcc.gnu.org/onlinedocs/libstdc++/manual/source_code_style.html Tested ppc64le-linux, committed to trunk. commit ae65b96cfb69c64d636f86d9f9154da2364e0440 Author: Jonathan Wakely jwak...@redhat.com Date: Mon Aug 3 16:29:44 2015 +0100 PR

Re: [BUILDROBOT] Go runtime: calling ‘__builtin_frame_address’ with a nonzero argument is unsafe

2015-08-03 Thread Martin Sebor
On 08/03/2015 05:55 AM, Jan-Benedict Glaw wrote: On Sun, 2015-08-02 17:15:27 -0600, Martin Sebor mse...@gmail.com wrote: OK for the trunk. Sorry for the delay. Thank you. Committed in revision 226480. ...und breaks native builds. When doing builds using config-list.mk, I first build a GCC

[gomp4] OpenACC first private

2015-08-03 Thread Nathan Sidwell
I've committed this patch to gomp4. The existing implementation of firstprivate presumes the existence of memory at the CTA level. This patch does away with that, treating firstprivate as thread-private variables initialized from the host. During development there was some fallout from

[gomp4] Fix ptx warning

2015-08-03 Thread Nathan Sidwell
I've committed this to gomp4. I was reusing a size_t variable in a loop where unsigned would do, and fprintf got upset. nathan 2015-08-03 Nathan Sidwell nat...@codesourcery.com * config/nvptx/mkoffload.c (process): Avoid printf formatting warning. Index: config/nvptx/mkoffload.c

[PING][PATCH][AArch64] Change aarch64 vector cost to match vectorizer

2015-08-03 Thread Pawel Kupidura
Hi, Ping for the patch submitted here https://gcc.gnu.org/ml/gcc-patches/2015-07/msg02251.html Thanks, Pawel Kupidura

Re: Fix two more memory leaks in threader

2015-08-03 Thread Jeff Law
On 07/20/2015 08:19 AM, James Greenhalgh wrote: I think we either want to defer the unordered_remove until we're done processing all the vector elements, or make sure to look at element 'i' again after we've moved something new in to it. Correct. Two loops had this mistake -- while others got

Re: [PATCH][AArch64] Change aarch64 vector cost to match vectorizer

2015-08-03 Thread James Greenhalgh
On Mon, Jul 27, 2015 at 02:22:41PM +0100, Pawel Kupidura wrote: diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 10df325..ffafc3f 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,7 @@ +2015-07-27 Pawel Kupidura pawel.kupid...@arm.com Two spaces between your name and your email

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

2015-08-03 Thread Jeff Law
On 08/03/2015 09:50 AM, Ilya Enkovich wrote: The original code looks better, tree height is just 2 and therefore it can be executed in 2 cycles. New code has more dependencies and tree height becomes 5. It is always hard to say for all x86 targets but as a generic code the original version is

[gomp4] Remove superfluous code

2015-08-03 Thread Nathan Sidwell
I've committed this to gomp4 branch. It tidies up the error message processing and removes some code that shows a remarkable confusion about how shared libraries work. nathan -- Nathan Sidwell - Director, Sourcery Services - Mentor Embedded 2015-08-03 Nathan Sidwell nat...@codesourcery.com

Re: [PING] Re: [PATCH] c/66516 - missing diagnostic on taking the address of a builtin function

2015-08-03 Thread Martin Sebor
I suspect the back end or even the middle end route isn't going to work even if there was enough context to diagnose the problem expressions because some of them will have been optimized away by then (e.g., 'if ( __builtin_foo != 0)' is optimized into if (1) by gimple). I was thinking that if

[PATCH], PR target/67071, Improve easy_altivec_constants on PowerPC

2015-08-03 Thread Michael Meissner
In preparing the next IEEE 128-bit floating point patch, I needed a quick way to load -0.0q into a vector registers (i.e. just the MSB set). I originally had a special purpose insn to load this value, but I decided to widen it to allow the easy_altivec_constant support to generate constants where

Re: [fortran,patch] Extend IEEE support to all real kinds

2015-08-03 Thread Uros Bizjak
On Mon, Aug 3, 2015 at 11:14 PM, FX fxcoud...@gmail.com wrote: The attached patch extends the IEEE modules to all floating-point kinds. Last time, when I added IEEE support, I restricted it to the float and double types (real kinds 4 and 8), to be extra safe. After discussion with Uros

Re: [fortran,patch] Extend IEEE support to all real kinds

2015-08-03 Thread Uros Bizjak
On Mon, Aug 3, 2015 at 11:14 PM, FX fxcoud...@gmail.com wrote: The attached patch extends the IEEE modules to all floating-point kinds. Last time, when I added IEEE support, I restricted it to the float and double types (real kinds 4 and 8), to be extra safe. After discussion with Uros

Re: [fortran,patch] Extend IEEE support to all real kinds

2015-08-03 Thread Steve Kargl
On Mon, Aug 03, 2015 at 11:14:41PM +0200, FX wrote: So, this enables full IEEE support for all real kinds. Nothing changes to the underlying architecture, it???s almost exclusively mechanical changes (adding the necessary variants to the interfaces, etc.). Bootstrapped and regtested on

Re: [PR64164] drop copyrename, integrate into expand

2015-08-03 Thread Alexandre Oliva
On Jul 30, 2015, H.J. Lu hjl.to...@gmail.com wrote: aoliva/pr64164 is fine on x32. Thanks. I have made a large number of changes since you tested it, fixing all the reported issues and then some. Now, x86_64-linux-gnu (-m64 and -m32), i686-pc-linux-gnu, powerpc64-linux-gnu and

Re: [gomp4] Remove superfluous code

2015-08-03 Thread Nathan Sidwell
On 08/03/15 19:52, Nathan Sidwell wrote: I've committed this to gomp4 branch. It tidies up the error message processing and removes some code that shows a remarkable confusion about how shared libraries work. I've also committed it to trunk as obvious. nathan

Re: [gofrontend-dev] Re: Go patch committed: Fix error reporting for invalid builtin calls

2015-08-03 Thread Michael Hudson-Doyle
Now I get ../../../gcc/libgo/runtime/mprof.goc: In function ‘runtime_Stack’: ../../../gcc/libgo/runtime/mprof.goc:437:19: error: ‘enablegc’ may be used uninitialized in this function [-Werror=maybe-uninitialized] mstats.enablegc = enablegc; ^

Re: [PATCH, rs6000] Fix source operand contraints for tabort. pattern

2015-08-03 Thread Peter Bergner
On Mon, 2015-08-03 at 16:27 -0400, David Edelsohn wrote: On Mon, Aug 3, 2015 at 3:39 PM, Peter Bergner berg...@vnet.ibm.com wrote: This has passed bootstrapping and regtesting on trunk. Ok for mainline? I'd like to also backport this to the release branches. Is this ok for them once

Re: [gofrontend-dev] Re: Go patch committed: Fix error reporting for invalid builtin calls

2015-08-03 Thread Ian Lance Taylor
On Mon, Aug 3, 2015 at 7:24 PM, Michael Hudson-Doyle michael.hud...@canonical.com wrote: Now I get ../../../gcc/libgo/runtime/mprof.goc: In function ‘runtime_Stack’: ../../../gcc/libgo/runtime/mprof.goc:437:19: error: ‘enablegc’ may be used uninitialized in this function

Minor typo fixes

2015-08-03 Thread Jeff Law
I was starting to look at Abe's changes to the gimple if-converter and realized a handful of the changes were just fixing comments. No reason those shouldn't go in immediately. So I pulled them out and applied those changes to the trunk. Abe -- if you find more of those kind of changes,

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

2015-08-03 Thread Jeff Law
On 07/17/2015 01:57 PM, Abe wrote: Dear all, Relative to the previous submission of this same patch, the below corrects some minor spacing and/or indentation issues, misc. other formatting fixes, and makes the disabled vectorization tests be disabled via xfail rather than by adding spaces to

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

2015-08-03 Thread Joseph Myers
On Mon, 3 Aug 2015, Michael Meissner wrote: The intention of theese changes (currently unwritten) is to change the existing problematical names that use TF in their name to be something else, and provide via a weak reference an alias for the old name. So if for example, we change the

libgo patch committed: Fix type of RawSockaddr Data field for PPC

2015-08-03 Thread Ian Lance Taylor
The gc compiler determines the type of the Data field in the syscall.RawSockaddr type by looking at the system header files. The gccgo library just has a fixed definition for it. Unfortunately the fixed definition does not match the header files on PPC64 systems. This is

Re: [PATCH] Avoid recomputing multiple times the defining predicate-chain for the same PHI

2015-08-03 Thread Richard Biener
On Sun, Aug 2, 2015 at 9:40 PM, Patrick Palka patr...@parcs.ath.cx wrote: On Sun, Aug 2, 2015 at 2:44 PM, Bernhard Reutner-Fischer rep.dot@gmail.com wrote: On August 2, 2015 8:18:51 PM GMT+02:00, Patrick Palka patr...@parcs.ath.cx wrote: No comment on the patch itself, but please

Re: [PATCH][AArch64][10/14] Implement target pragmas

2015-08-03 Thread Kyrill Tkachov
On 24/07/15 09:36, Kyrill Tkachov wrote: On 21/07/15 17:52, James Greenhalgh wrote: On Thu, Jul 16, 2015 at 04:21:05PM +0100, Kyrill Tkachov wrote: Hi all, This patch implements target pragmas for aarch64. The pragmas accepted are the same as for target attributes (as required). In addition

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

2015-08-03 Thread James Greenhalgh
On Fri, Jul 24, 2015 at 09:21:46AM +0100, Kyrill Tkachov wrote: Thanks for the review, here's an updated version. In the end, I chose to retain the use alloca (other patches in the series are reworked to use it too). How's this? A nit or two from code you were moving or that got caught up

[PATCH] Improve genmatch code-gen

2015-08-03 Thread Richard Biener
Since we allow arbitrary IDs in captures the capture array is dense and thus we don't need to clear it before initializing. We can also emit more compact source by simply using an initializer. - tree captures[3] ATTRIBUTE_UNUSED = {}; - captures[0] = o20; -

Re: [PATCH 3/4] Add libgomp plugin for Intel MIC

2015-08-03 Thread Maxim Blumental
Could you probably review the patch, please? 2015-07-28 18:42 GMT+03:00 Maxim Blumental bvm...@gmail.com: Applied the idea with python script alternative. Review, please. 2015-07-24 17:18 GMT+03:00 David Malcolm dmalc...@redhat.com: On Fri, 2015-07-24 at 10:01 +0200, Jakub Jelinek wrote:

Re: [gomp4, committed] Handle double reduction in oacc kernels pass group

2015-08-03 Thread Thomas Schwinge
Hi! On Tue, 28 Jul 2015 10:20:39 +0200, Tom de Vries tom_devr...@mentor.com wrote: this patch adds a test-case with a double reduction in an oacc kernels region. In order to get it in the proper shape for parloops to deal with, I needed to repeat the pass_lim/pass_copy_prop sequence.

[PATCH] Fix PR66917

2015-08-03 Thread Richard Biener
The following patch fixes PR66917 - the vectorizer assumes element alignment even for unaligned accesses, which in case of packed ones is not a valid assumption. This breaks testcases on arm at least which can do unaligned (but not completely unaligned) loads and stores. Bootstrapped and tested

Re: [PATCH] Unswitching outer loops.

2015-08-03 Thread Richard Biener
On Fri, Jul 31, 2015 at 1:17 PM, Yuri Rumyantsev ysrum...@gmail.com wrote: Hi Richard, I learned your updated patch for 23825 and it is more general in comparison with my. I'd like to propose you a compromise - let's consider my patch only for force-vectorize outer loop only to allow

Re: [ARM] Optimize compare against smin/umin

2015-08-03 Thread Christophe Lyon
Hi Michael, It looks like this patch introduces regressions on armeb in: gcc.dg/vect/vect-reduc-7.c gcc.dg/vect/vect-reduc-8.c gcc.dg/vect/vect-reduc-9.c See http://people.linaro.org/~christophe.lyon/cross-validation/gcc/trunk/226476/report-build-info.html for a bit more details. Can you have

[PATCH, PING*2] Track indirect calls for call site information in debug info.

2015-08-03 Thread Pierre-Marie de Rodat
On 07/20/2015 02:45 PM, Pierre-Marie de Rodat wrote: On PowerPC targets with -mlongcall, most subprogram calls are turned into indirect calls: the call target is read from a register even though it is compile-time known. This makes it difficult for machine code static analysis engines to recover

Re: C++ delayed folding branch review

2015-08-03 Thread Kai Tietz
2015-08-03 5:49 GMT+02:00 Jason Merrill ja...@redhat.com: On 07/31/2015 05:54 PM, Kai Tietz wrote: The STRIP_NOPS-requirement in 'reduced_constant_expression_p' I could remove, but for one case in constexpr. Without folding we don't do type-sinking/raising. Right. So binary/unary

Re: [PATCH][AArch64][8/14] Implement TARGET_OPTION_VALID_ATTRIBUTE_P

2015-08-03 Thread James Greenhalgh
On Fri, Jul 24, 2015 at 11:43:32AM +0100, Kyrill Tkachov wrote: On 21/07/15 16:37, James Greenhalgh wrote: On Thu, Jul 16, 2015 at 04:20:59PM +0100, Kyrill Tkachov wrote: Hi all, This patch implements target attribute support via the TARGET_OPTION_VALID_ATTRIBUTE_P hook. The

Re: [PATCH][ARM] PR target/66731 Fix vnmul insn with -frounding-math

2015-08-03 Thread Kyrill Tkachov
On 28/07/15 11:21, Szabolcs Nagy wrote: On 24/07/15 12:27, Kyrill Tkachov wrote: On 24/07/15 12:10, Szabolcs Nagy wrote: (-a)*b should not be compiled to vnmul a,b with -frounding-math. Added a new -(a*b) pattern for vnmul and the old one is only used if !flag_rounding_math. Updated the

Re: [Bug fortran/52846] [F2008] Support submodules - part 3/3

2015-08-03 Thread Mikael Morin
Le 29/07/2015 17:08, Paul Richard Thomas a écrit : Dear All, On 24 July 2015 at 10:08, Damian Rouson dam...@sourceryinstitute.org wrote: I love this idea and had similar thoughts as well. :D Sent from my iPhone On Jul 24, 2015, at 1:06 AM, Paul Richard Thomas paul.richard.tho...@gmail.com

Re: [PATCH][AArch64][9/14] Implement TARGET_CAN_INLINE_P

2015-08-03 Thread James Greenhalgh
On Thu, Jul 23, 2015 at 11:17:20AM +0100, Kyrill Tkachov wrote: Thanks, I've implemented the suggestions. Re-bootstrapped and tested on aarch64. How's this? This is good. OK for trunk. Thanks, James

Re: [gomp4] openacc default handling

2015-08-03 Thread Thomas Schwinge
Hi! On Wed, 29 Jul 2015 19:13:01 -0400, Nathan Sidwell nat...@acm.org wrote: gcc/ * gimplify.c (oacc_default_clause): Outer scope searching moved to omp_notice_variable. (omp_notice_variable): For OpenACC search enclosing scopes before applying default.

[PATCH, PING*2] PR debug/53927: fix value for DW_AT_static_link

2015-08-03 Thread Pierre-Marie de Rodat
On 07/20/2015 09:39 AM, Pierre-Marie de Rodat wrote: This patch fixes the static link description in DWARF to comply with the specification. In order to do so, it appends a field to all FRAME objects to hold the frame base address (DW_AT_frame_base) so that the nested subrograms can directly

Re: [PATCH] Use single shared memory block pool for all pool allocators

2015-08-03 Thread Mikhail Maltsev
On Jul 26, 2015, at 11:50 AM, Andi Kleen a...@firstfloor.org wrote: I've been compiling gcc with tcmalloc to do a similar speedup. It would be interesting to compare that to your patch. I repeated the test with TCMalloc and jemalloc. TCMalloc shows nice results, though it required some tweaks:

Re: [PATCH] Optimize certain end of loop conditions into min/max operation

2015-08-03 Thread Richard Biener
On Fri, Jul 31, 2015 at 8:27 PM, Jeff Law l...@redhat.com wrote: On 07/31/2015 12:18 PM, Michael Collison wrote: Hi Jeff, Yes I will create a test case. I'm not quite sure what to check for even in the machine dependent test case. It's quite possible for the instructions that are generated

Re: Go patch committed: Fix error reporting for invalid builtin calls

2015-08-03 Thread Andreas Schwab
Ian Lance Taylor i...@golang.org writes: This patch from Chris Manghane fixes the Go frontend error reporting for invalid builtin calls, by not losing track of whether the call is erroneous. This fixes https://golang.org/issue/11561. Bootstrapped and ran Go testsuite on

Re: [PATCH] Add __builtin_stack_top to x86 backend

2015-08-03 Thread Uros Bizjak
On Thu, Jul 30, 2015 at 8:41 PM, H.J. Lu hongjiu...@intel.com wrote: 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

[PATCH, PING] DWARF: materialize subprogram renamings in Ada as imported declarations

2015-08-03 Thread Pierre-Marie de Rodat
On 07/25/2015 09:44 PM, Pierre-Marie de Rodat wrote: This change makes GCC materialize subprogram renamings in Ada as imported declarations (DW_TAG_imported_declarations). For instance, procedure Foo renames Bar; will output: DW_TAG_imported_declaration: DW_AT_name: foo

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

2015-08-03 Thread H.J. Lu
On Mon, Aug 3, 2015 at 7:04 AM, Uros Bizjak ubiz...@gmail.com wrote: On Mon, Aug 3, 2015 at 3:02 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: On 03/08/15 13:33, Uros Bizjak wrote: Hello! 2015-07-30 Kyrylo Tkachov kyrylo.tkac...@arm.com * ifcvt.c (noce_try_store_flag_constants):

Re: [PATCH 2/5] Add preferred_for_{size,speed} attributes

2015-08-03 Thread H.J. Lu
On Fri, Oct 17, 2014 at 7:48 AM, Richard Sandiford richard.sandif...@arm.com wrote: This is the main patch, to add new preferred_for_size and preferred_for_speed attributes that can be used to selectively disable alternatives when optimising for size or speed. As explained in the docs, the

[PATCH] Build *-match.o as early as possible

2015-08-03 Thread Segher Boessenkool
The two files *-match.o files always finish building last, so if we start building them as soon as possible (instead of pretty late) the total build time will be less on a parallel build. Bootstrapped and tested on powerpc64-linux. Is this okay for trunk? Segher 2014-080-3 Segher

[hsa] HSA: introduce more pure kernel information emission.

2015-08-03 Thread Martin Liška
Hello. Following patch has been installed to HSA branch. Thanks, Martin From 64f7f75a8b16dee2071fef7f546fbf6b06f82c4f Mon Sep 17 00:00:00 2001 From: mliska mli...@suse.cz Date: Fri, 31 Jul 2015 17:53:57 +0200 Subject: [PATCH 1/3] HSA: introduce more pure kernel information emission.

[hsa] HSA: fix kernel initialization for dependent kernels.

2015-08-03 Thread Martin Liška
Hello. Following patch has been installed to HSA branch. Thanks, Martin From 125ee1ed4123527d4ce841631d4930ac96bec281 Mon Sep 17 00:00:00 2001 From: mliska mli...@suse.cz Date: Mon, 3 Aug 2015 13:00:42 +0200 Subject: [PATCH 2/3] HSA: fix kernel initialization for dependent kernels

[hsa] HSA: calculate properly maximum OMP data size which is, necessary.

2015-08-03 Thread Martin Liška
Hello. Following patch has been installed to HSA branch. Thanks, Martin From 163c6a4bb917ae76dad265dca5762a3bd87c947a Mon Sep 17 00:00:00 2001 From: mliska mli...@suse.cz Date: Mon, 3 Aug 2015 14:35:26 +0200 Subject: [PATCH 3/3] HSA: calculate properly maximum OMP data size which is necessary.

Re: [PATCH] Simplify vector compare-not-select sequence

2015-08-03 Thread Bill Schmidt
On Sun, 2015-08-02 at 12:18 +0200, Andreas Schwab wrote: Bill Schmidt wschm...@linux.vnet.ibm.com writes: * gcc.target/powerpc/vec-cmp-sel.c: New test. FAIL: gcc.target/powerpc/vec-cmp-sel.c (test for excess errors) Excess errors:

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

2015-08-03 Thread Martin Liška
PING. Thanks, Martin On 07/20/2015 09:51 AM, Martin Liška wrote: On 07/16/2015 01:03 PM, Martin Liška wrote: On 07/09/2015 06:24 PM, Jeff Law wrote: On 07/09/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-07-09 Martin Liska mli...@suse.cz * dbgcnt.def: Add new debug counter.

Re: [PATCH][AArch64][8/14] Implement TARGET_OPTION_VALID_ATTRIBUTE_P

2015-08-03 Thread Kyrill Tkachov
On 03/08/15 11:52, James Greenhalgh wrote: On Fri, Jul 24, 2015 at 11:43:32AM +0100, Kyrill Tkachov wrote: On 21/07/15 16:37, James Greenhalgh wrote: On Thu, Jul 16, 2015 at 04:20:59PM +0100, Kyrill Tkachov wrote: Hi all, This patch implements target attribute support via the

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

2015-08-03 Thread Steve Kargl
On Thu, Jul 30, 2015 at 04:08:47PM +0200, Mikael Morin wrote: Le 29/07/2015 18:45, Steve Kargl a écrit : This builds and passes regression testing on x86_64-*-freebsd. OP found the problem by running the sanatizers. I don't know how to build gcc with this as a build option. I'll

[committed] Fix error: unable to find a register to spill in class 'R1_REGS' reloading indirect call operand

2015-08-03 Thread John David Anglin
The 64-bit register indirect call does not use register %r1. So, %r1 shouldn't be clobbered in the call pattern. Normally, this doesn't matter as register %r1 is call clobbered. However, reload can fail when the register used for the indirect call needs to be reloaded from a static location.

libgo patch committed: Kill sleep processes in testsuite

2015-08-03 Thread Ian Lance Taylor
This patch from Andrew Wilkins kills off extra sleep processes when running the libgo testsuite. Otherwise, the sleep process would continue after the test completes. This doesn't affect the testsuite but the extra sleep processes are not desirable. Committed to mainline. Ian Index:

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

2015-08-03 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))) So presumably we're allowing both

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

2015-08-03 Thread Michael Meissner
On Fri, Jul 31, 2015 at 12:43:20AM +, Joseph Myers wrote: On Wed, 29 Jul 2015, Michael Meissner wrote: #6Add 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

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

2015-08-03 Thread Manuel López-Ibáñez
PING: https://gcc.gnu.org/ml/gcc-patches/2015-07/msg02581.html Thanks, Manuel. On 30 July 2015 at 17:35, Manuel López-Ibáñez lopeziba...@gmail.com wrote: 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

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

2015-08-03 Thread Jeff Law
On 07/28/2015 06:00 AM, Kugan wrote: On 27/07/15 05:38, Andreas Schwab wrote: Kugan kugan.vivekanandara...@linaro.org writes: * cse.c (cse_insn): Fix missing check for STRICT_LOW_PART and minor clean up. This breaks gcc.target/m68k/tls-ie-xgot.c scan-assembler jsr

Re: PR middle-end/16351 NULL dereference warnings

2015-08-03 Thread Manuel López-Ibáñez
PING: https://gcc.gnu.org/ml/gcc-patches/2015-07/msg01860.html Actually, the xfailed test was because the function folded to nothing and the offending code was removed without warning. Fixed in the attached version. Same changelog. On 22 July 2015 at 17:52, Manuel López-Ibáñez

Go patch committed: Don't make erroneous type descriptors

2015-08-03 Thread Ian Lance Taylor
This patch by Chris Manghane changes the Go frontend to not make type descriptor for named types if we have seen errors during the compilation. This avoids a compiler crash, and fixes https://golang.org/issue/11560 . Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to

[committed, PATCH]: Add a testcase for PR tree-optimization/67077

2015-08-03 Thread H.J. Lu
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67077 is fixed by https://gcc.gnu.org/viewcvs/gcc?view=revisionrevision=226089 I checked in this patch to add a testcase for PR tree-optimization/67077. Index: ChangeLog === ---

Re: [RFC] [Patch]: Try and vectorize with shift for mult expr with power 2 integer constant.

2015-08-03 Thread Jeff Law
On 08/02/2015 05:03 AM, Kumar, Venkataramanan wrote: Hi Jakub, Thank you for reviewing the patch. I have incorporated your comments in the attached patch. Note Jakub is on PTO for the next 3 weeks. vectorize_mults_via_shift.diff.txt diff --git

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

2015-08-03 Thread Kyrill Tkachov
On 03/08/15 16:53, Kyrill Tkachov wrote: On 03/08/15 16:45, Uros Bizjak wrote: On Mon, Aug 3, 2015 at 5:37 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Looking at the x86 movcc expansion code (ix86_expand_int_movcc) I don't think this is a good idea. In the expander, there is already

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

2015-08-03 Thread Uros Bizjak
On Mon, Aug 3, 2015 at 7:20 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Looking at the x86 movcc expansion code (ix86_expand_int_movcc) I don't think this is a good idea. In the expander, there is already quite some target-dependent code that goes great length to utilize sbb insn as much

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

2015-08-03 Thread Jeff Law
On 07/16/2015 05:03 AM, Martin Liška wrote: So a general question. We're passing in STRICT to several routines, which is fine. But then we're also checking M_TAIL_MERGE_MODE. What's the difference between the two? Can they be unified? Hello. I would say that STRICT is a bit generic

Re: [PR66726] Factor conversion out of COND_EXPR

2015-08-03 Thread Jeff Law
On 07/26/2015 07:05 PM, Kugan wrote: I thought that when !gimple_assign_cast_p (stmt), RHS will always boolean. I have now added this check in the attached patch. Thanks. I also noticed that in maybe_optimize_range_tests, GIMPLE_COND can have non compatible types when new_op is updated

Re: [PATCH] Simplify vector compare-not-select sequence

2015-08-03 Thread Andreas Schwab
Bill Schmidt wschm...@linux.vnet.ibm.com writes: Index: gcc/testsuite/gcc.target/powerpc/vec-cmp-sel.c === --- gcc/testsuite/gcc.target/powerpc/vec-cmp-sel.c(revision 226505) +++ gcc/testsuite/gcc.target/powerpc/vec-cmp-sel.c

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

2015-08-03 Thread Jeff Law
On 07/20/2015 01:52 AM, Martin Liška wrote: Due to changes dump file, it will be necessary to amend scanning of dump files. Those changes are fine and can be installed once the prerequisites have been approved/installed. Thanks, Jeff

Re: [PATCH 1/2] gcc parts of timer refactoring

2015-08-03 Thread Jeff Law
On 07/31/2015 03:45 PM, David Malcolm wrote: In r223092 (aka dd4d567f4b6b498242097c41d63666bdae320ac1) I moved the state of timevar.c from being global data into a class timer with a global instance g_timer. This followup patch generalizes the timing within toplev so that an external timer

Re: Go patch committed: Fix error reporting for invalid builtin calls

2015-08-03 Thread Ian Lance Taylor
On Mon, Aug 3, 2015 at 2:10 AM, Andreas Schwab sch...@suse.de wrote: ../../../libgo/runtime/mprof.goc: In function 'runtime_Stack': ../../../libgo/runtime/mprof.goc:408:5: error: calling '__builtin_frame_address' with a nonzero argument is unsafe [-Werror=frame-address] sp =

Re: Go patch committed: Fix error reporting for invalid builtin calls

2015-08-03 Thread Richard Biener
On Mon, Aug 3, 2015 at 11:10 AM, Andreas Schwab sch...@suse.de wrote: Ian Lance Taylor i...@golang.org writes: This patch from Chris Manghane fixes the Go frontend error reporting for invalid builtin calls, by not losing track of whether the call is erroneous. This fixes

Re: [gomp4] expand acc_on_device earlier

2015-08-03 Thread Thomas Schwinge
Hi! On Mon, 3 Aug 2015 08:00:36 -0400, Nathan Sidwell nat...@codesourcery.com wrote: On 08/03/15 07:37, Thomas Schwinge wrote: On Sun, 2 Aug 2015 21:23:30 -0400, Nathan Sidwell nat...@acm.org wrote: I've committed this to gomp4 branch. It expands the acc_on_device builtin earlier in the

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

2015-08-03 Thread Kyrill Tkachov
On 03/08/15 13:33, Uros Bizjak wrote: Hello! 2015-07-30 Kyrylo Tkachov kyrylo.tkac...@arm.com * ifcvt.c (noce_try_store_flag_constants): Make logic of the case when diff == STORE_FLAG_VALUE or diff == -STORE_FLAG_VALUE more explicit. Prefer to add the flag whenever

Re: [PATCH][AArch64][10/14] Implement target pragmas

2015-08-03 Thread James Greenhalgh
On Mon, Aug 03, 2015 at 10:36:17AM +0100, Kyrill Tkachov wrote: And here is a rebased version to resolve a conflict after Alan's patches went in. OK with the nits below fixed. 2015-08-03 Kyrylo Tkachov kyrylo.tkac...@arm.com * config.gcc (aarch64*-*-*): Specify c_target_objs and

Re: offload data version number

2015-08-03 Thread Thomas Schwinge
Hi! On Sat, 1 Aug 2015 20:20:49 -0400, Nathan Sidwell nat...@acm.org wrote: this is the gomp4 committed version. --- liboffloadmic/plugin/libgomp-plugin-intelmic.cpp (revision 226462) +++ liboffloadmic/plugin/libgomp-plugin-intelmic.cpp (working copy) @@ -327,12 +327,26 @@ offload_image

Re: [PATCH][AArch64][11/14] Re-layout SIMD builtin types on builtin expansion

2015-08-03 Thread James Greenhalgh
On Fri, Jul 24, 2015 at 09:38:34AM +0100, Kyrill Tkachov wrote: Thanks, here's an updated version. 2015-07-24 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/aarch64/aarch64.c (aarch64_option_valid_attribute_p): Initialize simd builtins if TARGET_SIMD. *

Re: [PATCH][AArch64][12/14] Target attributes and target pragmas tests

2015-08-03 Thread James Greenhalgh
On Fri, Jul 24, 2015 at 09:40:28AM +0100, Kyrill Tkachov wrote: On 21/07/15 18:14, James Greenhalgh wrote: On Thu, Jul 16, 2015 at 04:21:15PM +0100, Kyrill Tkachov wrote: Hi all, These are the tests for target attributes and pragmas. I've tried to test for the inlining rules, some of

Re: [PATCH] Work around host compiler placement new aliasing bug

2015-08-03 Thread Ulrich Weigand
Richard Biener wrote: On Wed, Jul 29, 2015 at 3:57 PM, Ulrich Weigand uweig...@de.ibm.com wrote: Hello, this patch is a workaround for the problem discussed here: https://gcc.gnu.org/ml/gcc-patches/2015-07/msg01597.html The problem is that the new pool allocator code relies on C++

Re: [gomp4] expand acc_on_device earlier

2015-08-03 Thread Thomas Schwinge
Hi! On Sun, 2 Aug 2015 21:23:30 -0400, Nathan Sidwell nat...@acm.org wrote: I've committed this to gomp4 branch. It expands the acc_on_device builtin earlier in the new oacc_xform pass. This will allow more optimization earlier on. As far as I remember, the Fortran front end doesn't know

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

2015-08-03 Thread Uros Bizjak
Hello! 2015-07-30 Kyrylo Tkachov kyrylo.tkac...@arm.com * ifcvt.c (noce_try_store_flag_constants): Make logic of the case when diff == STORE_FLAG_VALUE or diff == -STORE_FLAG_VALUE more explicit. Prefer to add the flag whenever possible. (noce_process_if_block): Try

Re: [Bug fortran/52846] [F2008] Support submodules - part 3/3

2015-08-03 Thread Paul Richard Thomas
Dear Mikael, Thanks for your green light! I have been mulling over the trans-decl part of the patch and having been wondering if it is necessary. Without optimization, private entities can be linked to. Given the discussion concerning the combination of submodules and private entities, I wonder

Re: [gomp4] expand acc_on_device earlier

2015-08-03 Thread Nathan Sidwell
On 08/03/15 07:37, Thomas Schwinge wrote: Hi! On Sun, 2 Aug 2015 21:23:30 -0400, Nathan Sidwell nat...@acm.org wrote: I've committed this to gomp4 branch. It expands the acc_on_device builtin earlier in the new oacc_xform pass. This will allow more optimization earlier on. Thanks! The

RE: [PATCH, MIPS, Ping] Inline memcpy for MipsR6

2015-08-03 Thread Simon Dardis
Catherine, Inline-memcpy-2.c updated to not run with -Os. Patch rebased off current gcc sources. Thanks, Simon diff --git a/gcc/config/mips/mips.c b/gcc/config/mips/mips.c index 1733457..627e078 100644 --- a/gcc/config/mips/mips.c +++ b/gcc/config/mips/mips.c @@ -7520,12 +7520,22 @@

Re: [PATCH][AArch64][1/3] Expand signed mod by power of 2 using CSNEG

2015-08-03 Thread James Greenhalgh
On Fri, Jul 24, 2015 at 11:55:33AM +0100, Kyrill Tkachov wrote: Hi all, This patch implements an aarch64-specific expansion of the signed modulo by a power of 2. The proposed sequence makes use of the conditional negate instruction CSNEG. For a power of N, x % N can be calculated with:

Re: [gomp4] expand acc_on_device earlier

2015-08-03 Thread Thomas Schwinge
Hi! On Sun, 2 Aug 2015 21:23:30 -0400, Nathan Sidwell nat...@acm.org wrote: I've committed this to gomp4 branch. It expands the acc_on_device builtin earlier in the new oacc_xform pass. This will allow more optimization earlier on. Thanks! The existing expansion point now only needs to

[gomp4] PTX launch dimensions

2015-08-03 Thread Nathan Sidwell
I've committed this to gomp4. The ptx backend can now examine the openacc attribute to determine launch dimensions and figure out whether vector or worker single neutering is needed. nathan 2015-08-03 Nathan Sidwell nat...@codesourcery.com * config/nvptx/nvptx.c (nvptx_reorg): Check

[PATCH] Merge even more tails (from for expansion) in genmatch.c

2015-08-03 Thread Richard Biener
This changes for lowering for most cases that can end up being tail-merged to not substitute the operator into the result expression but refer to it using a local variable of the name of the for iterator. Boostrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. This drops the

Re: [C++/66443] virtual base of abstract class

2015-08-03 Thread Nathan Sidwell
On 08/02/15 23:44, Jason Merrill wrote: It seems to me that DR 1658 ignores vbases of abstract classes for determining whether a destructor is deleted, but says nothing about exception specifications. DR 1351 specifically ignores vbases of abstract classes for determining the exception

  1   2   >