Re: [patch, libgomp] Re-factor GOMP_MAP_POINTER handling

2015-05-21 Thread Chung-Lin Tang
Ping x2. On 15/5/11 7:19 PM, Chung-Lin Tang wrote: Ping. On 2015/4/21 08:21 PM, Chung-Lin Tang wrote: Hi, while investigating some issues in the variable mapping code, I observed that the GOMP_MAP_POINTER handling is essentially duplicated under the PSET case. This patch abstracts and

Re: [match-and-simplify] reject expanding operator-list to implicit 'for'

2015-05-21 Thread Richard Biener
On Wed, 20 May 2015, Prathamesh Kulkarni wrote: On 20 May 2015 at 18:18, Richard Biener rguent...@suse.de wrote: On Wed, 20 May 2015, Prathamesh Kulkarni wrote: On 20 May 2015 at 17:01, Richard Biener rguent...@suse.de wrote: On Wed, 20 May 2015, Prathamesh Kulkarni wrote: On 20

Re: OpenACC: initialization with unsupported acc_device_t

2015-05-21 Thread Jakub Jelinek
On Thu, May 21, 2015 at 08:55:59AM +0200, Thomas Schwinge wrote: Thanks, looks good to me -- Jakub? Ok for trunk. libgomp/ * oacc-init.c (resolve_device): Add FAIL_IS_ERROR argument. Update function comment. Only call gomp_fatal if new argument is true.

Re: [obvious fix] fix off-by-one error when printing the caret character

2015-05-21 Thread Dodji Seketeli
Manuel López-Ibáñez lopeziba...@gmail.com writes: Index: ChangeLog === --- ChangeLog (revision 223445) +++ ChangeLog (working copy) @@ -1,3 +1,8 @@ +2015-05-20 Manuel López-Ibáñez m...@gcc.gnu.org + + *

[PATCH, CHKP] Fix PR middle-end/66221: lto1: error: type variant has different TYPE_ARG_TYPES

2015-05-21 Thread Ilya Enkovich
Hi, This patch fixes PR66221 by using build_distinct_type_copy instead of copy_node to copy a function type for instrumented function. Bootstrapped and regtested for x86_64-unknown-linux-gnu. Applied to trunk. Is it OK for gcc-5? Thanks, Ilya -- gcc/ 2015-05-21 Ilya Enkovich

Re: [nvptx] Re: Mostly rewrite genrecog

2015-05-21 Thread Thomas Schwinge
Hi! On Thu, 7 May 2015 11:14:37 +0200, Jakub Jelinek ja...@redhat.com wrote: On Thu, May 07, 2015 at 10:59:01AM +0200, Thomas Schwinge wrote: build/genrecog [...]/source-gcc/gcc/common.md [...]/source-gcc/gcc/config/nvptx/nvptx.md \ insn-conditions.md tmp-recog.c

Re: Add statistics to alias.c

2015-05-21 Thread Bernhard Reutner-Fischer
On May 21, 2015 12:13:19 AM GMT+02:00, Jan Hubicka hubi...@ucw.cz wrote: Hi, this patch extends statistics from tree-ssa-alias to also cover TBAA oracle. This is useful to keep track of aliasing effectivity. For example the hack in alias.c putting globbing all pointers to one costs about 20% of

Re: PING: Re: [patch 6/10] debug-early merge: Java front-end

2015-05-21 Thread Andrew Haley
On 20/05/15 23:32, Aldy Hernandez wrote: Perhaps I should've sent this to the java-patches list. PING. OK, I believe it. Andrew.

Re: [PATCH][DRIVER] Wrong C++ include paths when configuring with --with-sysroot=/

2015-05-21 Thread Yvan Roux
Hi, On 8 May 2015 at 00:07, Joseph Myers jos...@codesourcery.com wrote: On Mon, 20 Apr 2015, Pavel Kopyl wrote: Hi all, To build a GCC-4.9.2 ARM cross-compiler for my setting I need to configure it with --with-sysroot=/ --with-gxx-include-dir=/usr/include/c++/4.9.2. But I found that gcc

Re: OpenACC: initialization with unsupported acc_device_t

2015-05-21 Thread Thomas Schwinge
Hi Julian! On Thu, 7 May 2015 16:56:11 +0100, Julian Brown jul...@codesourcery.com wrote: On Tue, 5 May 2015 16:09:18 +0200 Thomas Schwinge tho...@codesourcery.com wrote: On Tue, 5 May 2015 08:43:48 -0400, John David Anglin dave.ang...@bell.net wrote: On 2015-05-05 5:43 AM, Thomas

Re: [Patch AArch64] PR target/66200 - gcc / libstdc++ TLC for weak memory models.

2015-05-21 Thread Ramana Radhakrishnan
And here's an additional patch for the testsuite which was missed in the original posting. This is a testism that's testing code generation as per TARGET_RELAXED_ORDERING being false and therefore needs to be adjusted as attached. Ramana PR target/66200 * g++.dg/abi/aarch64_guard1.C:

Re: [PATCH] [PATCH][ARM] Fix sibcall testcases.

2015-05-21 Thread Ramana Radhakrishnan
On Wed, May 20, 2015 at 9:11 PM, Joseph Myers jos...@codesourcery.com wrote: On Wed, 20 May 2015, Alex Velenko wrote: Hi, This patch prevents arm_thumb1_ok XPASS in sibcall-3.c and sibcall-4.c testcases. Sibcalls are not ok for Thumb1 and testcases need to be fixed. arm_thumb1_ok means

Re: [patch, testsuite] don't specify dg-do run explicitly for vect test cases

2015-05-21 Thread Richard Biener
On Thu, May 21, 2015 at 7:12 AM, Sandra Loosemore san...@codesourcery.com wrote: On targets such as ARM, some arches are compatible with options needed to enable compilation with vectorization, but the specific hardware (or simulator or BSP) available for execution tests may not implement or

[Patch ARM] Fix PR target/65937

2015-05-21 Thread Ramana Radhakrishnan
Testism introduced by last commit to fix PR26702 on arm-*-linux* targets. The fix is to restore target selector to arm*-*-eabi* as the target macro changes only affect arm*-*-eabi* Applied to trunk as obvious Ramana * gcc.target/arm/pr26702.c: Adjust target selector. Index:

Re: [patch, testsuite, ARM] don't try to execute simd.exp tests on targets without NEON

2015-05-21 Thread Kyrill Tkachov
Hi Sandra, On 21/05/15 06:43, Sandra Loosemore wrote: This is another patch aimed at fixing bugs relating to trying to execute NEON code on a target that doesn't support it revealed by my arm-none-eabi testing on a gazillion different multilibs. Inspired by what vect.exp does and my other

Re: [PATCH, CHKP] Fix PR middle-end/66221: lto1: error: type variant has different TYPE_ARG_TYPES

2015-05-21 Thread Richard Biener
On Thu, May 21, 2015 at 10:38 AM, Ilya Enkovich enkovich@gmail.com wrote: Hi, This patch fixes PR66221 by using build_distinct_type_copy instead of copy_node to copy a function type for instrumented function. Bootstrapped and regtested for x86_64-unknown-linux-gnu. Applied to trunk.

Re: [RFA] Restore combine.c split point for multiply-accumulate instructions

2015-05-21 Thread Richard Biener
On Thu, May 21, 2015 at 7:38 AM, Jeff Law l...@redhat.com wrote: find_split_point will tend to favor splitting complex insns in such a way as to encourage multiply-add insns. It does this by splitting an unrecognizable insn at the (plus (mult)). Now that many MULTs are canonicalized as

[gomp4.1] Taskloop support

2015-05-21 Thread Jakub Jelinek
Hi! This patch finishes the C #pragma omp taskloop support on the gomp 4.1 branch, including library support. 2015-05-21 Jakub Jelinek ja...@redhat.com * tree.h (OMP_STANDALONE_CLAUSES): Adjust to cover OMP_TARGET_{ENTER,EXIT}_DATA. (OMP_CLAUSE_SHARED_FIRSTPRIVATE):

RE: [PATCH, ping 1] Move insns without introducing new temporaries in loop2_invariant

2015-05-21 Thread Uros Bizjak
Hello! From: Jeff Law [mailto:l...@redhat.com] Sent: Wednesday, May 13, 2015 4:05 AM OK for the trunk. Thanks for your patience, Thanks. Committed with the added PR rtl-optimization/64616 to both ChangeLog entries. This patch caused PR66236 [1]. [1]

Re: Add statistics to alias.c

2015-05-21 Thread Richard Biener
On Thu, 21 May 2015, Jan Hubicka wrote: Hi, this patch extends statistics from tree-ssa-alias to also cover TBAA oracle. This is useful to keep track of aliasing effectivity. For example the hack in alias.c putting globbing all pointers to one costs about 20% of all answers on firefox. I.e.

Re: Ping ** 0.5 patch, fortran] Inline matmul with conjugate complex numbers

2015-05-21 Thread Mikael Morin
Le 21/05/2015 19:51, Thomas Koenig a écrit : Am 18.05.2015 um 00:05 schrieb Thomas Koenig: this patch extends the inline matmul functionality to conjugate complex numbers. Regression-tested. OK for trunk? OK (with the trivial change in the follow-up e-mail)? I'd like to start extending

Re: [AArch64][TLSLE][4/N] Recognize -mtls-size

2015-05-21 Thread Jiong Wang
Jiong Wang writes: This patch add -mtls-size option for AArch64. This option let user to do finer control on code generation for various TLS model on AArch64. For example, for TLS LE, user can specify smaller tls-size, for example 4K which is quite usual, to let AArch64 backend generate

Ping ** 0.5 patch, fortran] Inline matmul with conjugate complex numbers

2015-05-21 Thread Thomas Koenig
Am 18.05.2015 um 00:05 schrieb Thomas Koenig: this patch extends the inline matmul functionality to conjugate complex numbers. Regression-tested. OK for trunk? OK (with the trivial change in the follow-up e-mail)? I'd like to start extending this to TRANSPOSE(CONJG(A)) :-) Thomas

Re: C/C++ PATCH to allow deprecating enum values (PR c/47043)

2015-05-21 Thread Jason Merrill
On 05/07/2015 12:22 PM, Marek Polacek wrote: - mark_used (decl); + mark_used (decl, 0); This should use tf_none rather than 0. + build_enumerator (DECL_NAME (decl), value, newtag, + DECL_ATTRIBUTES (decl), DECL_SOURCE_LOCATION (decl)); This is assuming

Re: [PATCH] PR target/66232: -fPIC -fno-plt -mx32 fails to generate indirect branch via GOT

2015-05-21 Thread Uros Bizjak
On Thu, May 21, 2015 at 8:36 PM, H.J. Lu hjl.to...@gmail.com wrote: I am testing this now. I will check it in if there is no regression. OK. Maybe just : ;; Return true if OP is a GOT memory operand. (define_predicate GOT_memory_operand (match_operand 0 memory_operand) { return

Re: [PATCH] Fix memory orders description in atomic ops built-ins docs.

2015-05-21 Thread Torvald Riegel
On Thu, 2015-05-21 at 16:45 +0100, Matthew Wahab wrote: On 19/05/15 20:20, Torvald Riegel wrote: On Mon, 2015-05-18 at 17:36 +0100, Matthew Wahab wrote: Hello, On 15/05/15 17:22, Torvald Riegel wrote: This patch improves the documentation of the built-ins for atomic operations.

Re: [PATCH] PR target/66232: -fPIC -fno-plt -mx32 fails to generate indirect branch via GOT

2015-05-21 Thread H.J. Lu
On Thu, May 21, 2015 at 11:41 AM, Richard Henderson r...@redhat.com wrote: On 05/21/2015 05:59 AM, H.J. Lu wrote: +(define_predicate x32_sibcall_memory_operand + (and (match_operand 0 memory_operand) + (match_test CONSTANT_P (XEXP (op, 0))) + (match_test GET_CODE (XEXP (XEXP (op,

Calculate TYPE_CANONICAL only for types that can be accessed in memory

2015-05-21 Thread Jan Hubicka
Hi, this is next part of the series. It disables canonical type calculation for incomplete types with exception of arrays based on claim that we do not have good notion of those. I can botostrap this with additional checks in alias.c that canonical types are always present with LTO but I need

Re: [patch] testsuite enable PIE tests on FreeBSD

2015-05-21 Thread Andreas Tobler
On 20.05.15 22:30, Jeff Law wrote: On 05/20/2015 11:04 AM, Andreas Tobler wrote: Hi, the attached patch enables some PIE tests on FreeBSD. Ok for trunk? Thanks, Andreas 2015-05-20 Andreas Tobler andre...@gcc.gnu.org * gcc.target/i386/pr32219-1.c: Enable test on FreeBSD. *

Re: [PATCH] PR target/66232: -fPIC -fno-plt -mx32 fails to generate indirect branch via GOT

2015-05-21 Thread Richard Henderson
On 05/21/2015 05:59 AM, H.J. Lu wrote: +(define_predicate x32_sibcall_memory_operand + (and (match_operand 0 memory_operand) + (match_test CONSTANT_P (XEXP (op, 0))) + (match_test GET_CODE (XEXP (XEXP (op, 0), 0)) == UNSPEC) + (match_test XINT (XEXP (XEXP (op, 0), 0), 1) ==

Re: [nvptx] Re: Mostly rewrite genrecog

2015-05-21 Thread Bernd Schmidt
On 05/21/2015 09:12 AM, Thomas Schwinge wrote: OK to commit? gcc/ * config/nvptx/nvptx.md (allocate_stack): Rename to... (allocate_stack_mode): ... this, and add :P on both match_operand and unspec. (allocate_stack): New expander. If you really want

[gomp4] Vector-single predication

2015-05-21 Thread Bernd Schmidt
This uses the patch I committed yesterday which introduces warp broadcasts to implement the vector-single predication needed for OpenACC. Outside a loop with vector parallelism, only one of the threads representing a vector must execute, the others follow along. So we skip the real work in

Re: [patch, testsuite] don't specify dg-do run explicitly for vect test cases

2015-05-21 Thread Jeff Law
On 05/20/2015 11:12 PM, Sandra Loosemore wrote: On targets such as ARM, some arches are compatible with options needed to enable compilation with vectorization, but the specific hardware (or simulator or BSP) available for execution tests may not implement or enable those features. The vect.exp

Re: [patch, libgomp] Re-factor GOMP_MAP_POINTER handling

2015-05-21 Thread Thomas Schwinge
Hi! Jakub, for avoidance of doubt, the proposed refactoring makes sense to me, but does need your approval: On Thu, 21 May 2015 16:30:40 +0800, Chung-Lin Tang clt...@codesourcery.com wrote: Ping x2. On 15/5/11 7:19 PM, Chung-Lin Tang wrote: Ping. On 2015/4/21 08:21 PM, Chung-Lin Tang

Re: [PATCH] PR target/66232: -fPIC -fno-plt -mx32 fails to generate indirect branch via GOT

2015-05-21 Thread Uros Bizjak
On Thu, May 21, 2015 at 2:59 PM, H.J. Lu hjl.to...@gmail.com wrote: X32 doesn't support indirect branch via 32-bit memory slot since indirect branch will load 64-bit address from 64-bit memory slot. Since x32 GOT slot is 64-bit, we should allow indirect branch via GOT slot for x32. I am

Re: [PATCH 6/7] remove #if HAVE_conditional_move

2015-05-21 Thread Jeff Law
On 05/20/2015 08:09 PM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders tbsaunde+...@tbsaunde.org gcc/ChangeLog: 2015-05-20 Trevor Saunders tbsaunde+...@tbsaunde.org * *.c, *.h: DOn't check HAVE_conditional_move with the preprocessor. You know what I'm going to say here :-)

Re: [PATCH 3/7] move default for STACK_PUSH_CODE to defaults.h

2015-05-21 Thread Jeff Law
On 05/20/2015 08:09 PM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders tbsaunde+...@tbsaunde.org gcc/ChangeLog: 2015-05-20 Trevor Saunders tbsaunde+...@tbsaunde.org * defaults.h: Add default for STACK_PUSH_CODE. * expr.c: Don't redefine STACK_PUSH_CODE. *

[Patch]: libbacktrace - add support of PE/COFF

2015-05-21 Thread Tristan Gingold
Hello, this patch adds basic support to libbacktrace for PE32 and PE32+ (Windows and Windows64 object formats). Support is ‘basic’ because neither DLL nor PIE (if that exists) are handled. Furthermore, there is no windows versions of mmapio.c and mmap.c Finally, I have disabled the support of

Re: [gomp4] Vector-single predication

2015-05-21 Thread Julian Brown
On Thu, 21 May 2015 13:57:00 +0200 Jakub Jelinek ja...@redhat.com wrote: On Thu, May 21, 2015 at 01:42:11PM +0200, Bernd Schmidt wrote: This uses the patch I committed yesterday which introduces warp broadcasts to implement the vector-single predication needed for OpenACC. Outside a loop

Re: [PATCH, PR target/65103, 2/3] Propagate address constants into loops for i386

2015-05-21 Thread Ilya Enkovich
Ping 2015-05-05 14:05 GMT+03:00 Ilya Enkovich enkovich@gmail.com: 2015-04-21 8:52 GMT+03:00 Jeff Law l...@redhat.com: On 04/17/2015 02:34 AM, Ilya Enkovich wrote: On 15 Apr 14:07, Ilya Enkovich wrote: 2015-04-14 8:22 GMT+03:00 Jeff Law l...@redhat.com: On 03/15/2015 02:30 PM, Richard

Re: [PATCH 5/7] always define HAVE_conditional_move

2015-05-21 Thread Jeff Law
On 05/20/2015 08:09 PM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders tbsaunde+...@tbsaunde.org gcc/ChangeLog: 2015-05-20 Trevor Saunders tbsaunde+...@tbsaunde.org * genconfig.c (main): Always define HAVE_conditional_move. * *.c: Don't check if HAVE_conditional_move is

Re: [PATCH 2/7] remove most ifdef STACK_GROWS_DOWNWARD

2015-05-21 Thread Jeff Law
On 05/20/2015 08:09 PM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders tbsaunde+...@tbsaunde.org gcc/c-family/ChangeLog: 2015-05-20 Trevor Saunders tbsaunde+...@tbsaunde.org * c-cppbuiltin.c (c_cpp_builtins): Use if instead of #if with STACK_GROWS_DOWNWARD.

Re: [PATCH 1/7] always define STACK_GROWS_DOWNWARD

2015-05-21 Thread Jeff Law
On 05/20/2015 08:09 PM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders tbsaunde+...@tbsaunde.org gcc/c-family/ChangeLog: 2015-05-20 Trevor Saunders tbsaunde+...@tbsaunde.org * c-cppbuiltin.c (c_cpp_builtins): Check the value of STACK_GROWS_DOWNWARD rather than if it

Re: [SH][committed] Fix gcc.target/sh/pr54236-2.c failures

2015-05-21 Thread Oleg Endo
On Tue, 2015-05-19 at 10:04 +0200, Oleg Endo wrote: Since a recent change to the tree optimizers https://gcc.gnu.org/ml/gcc-patches/2015-05/msg00089.html some related SH patterns stopped working. The attached patch fixes this. Tested briefly with 'make all' and with make -k check-gcc

[PATCH] PR target/66232: -fPIC -fno-plt -mx32 fails to generate indirect branch via GOT

2015-05-21 Thread H.J. Lu
X32 doesn't support indirect branch via 32-bit memory slot since indirect branch will load 64-bit address from 64-bit memory slot. Since x32 GOT slot is 64-bit, we should allow indirect branch via GOT slot for x32. I am testing it on x32. OK for master if there is no regression? Thanks. H.J.

Re: [PATCH 4/7] don't compare ARG_FRAME_POINTER_REGNUM and FRAME_POINTER_REGNUM with the preprocessor

2015-05-21 Thread Jeff Law
On 05/20/2015 08:09 PM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders tbsaunde+...@tbsaunde.org gcc/ChangeLog: 2015-05-20 Trevor Saunders tbsaunde+...@tbsaunde.org * *.c: Remove comparison of ARG_FRAME_POINTER_REGNUM and FRAME_POINTER_REGNUM with the preprocessor.

[PATCH] Fix PR66211

2015-05-21 Thread Richard Biener
The following papers over the C++ FE issue that it doesn't track lvalueness before folding stuff. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2015-05-21 Richard Biener rguent...@suse.de PR c++/66211 * match.pd: Guard pattern optimzing

Re: [gomp4] Vector-single predication

2015-05-21 Thread Jakub Jelinek
On Thu, May 21, 2015 at 01:42:11PM +0200, Bernd Schmidt wrote: This uses the patch I committed yesterday which introduces warp broadcasts to implement the vector-single predication needed for OpenACC. Outside a loop with vector parallelism, only one of the threads representing a vector must

Re: [gomp4] Vector-single predication

2015-05-21 Thread Jakub Jelinek
On Thu, May 21, 2015 at 02:05:12PM +0100, Julian Brown wrote: OpenACC handles function calls specially (calling them routines -- of varying sorts, gang, worker, vector or seq, affecting where they can be invoked from). The plan is that all threads will call such routines -- and then some

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

2015-05-21 Thread Sriraman Tallam
On Sun, May 10, 2015 at 10:01 AM, Sriraman Tallam tmsri...@google.com wrote: On Sun, May 10, 2015, 8:19 AM H.J. Lu hjl.to...@gmail.com wrote: On Sat, May 9, 2015 at 9:34 AM, H.J. Lu hjl.to...@gmail.com wrote: On Mon, May 4, 2015 at 7:45 AM, Michael Matz m...@suse.de wrote: Hi, On Thu, 30

Re: [patch 1/10] debug-early merge: Ada front-end

2015-05-21 Thread Aldy Hernandez
On 05/08/2015 06:26 AM, Eric Botcazou wrote: [Jason, question below.] What's the replacement mechanism for the first pass on global_decls? The comment explains that generating debug info must be delayed in this case. My apologies for the delay on Ada. I have reworked the patch to leave

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

2015-05-21 Thread Sriraman Tallam
On Thu, May 21, 2015 at 2:12 PM, Sriraman Tallam tmsri...@google.com wrote: On Sun, May 10, 2015 at 10:01 AM, Sriraman Tallam tmsri...@google.com wrote: On Sun, May 10, 2015, 8:19 AM H.J. Lu hjl.to...@gmail.com wrote: On Sat, May 9, 2015 at 9:34 AM, H.J. Lu hjl.to...@gmail.com wrote: On Mon,

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

2015-05-21 Thread Pedro Alves
On 05/21/2015 10:12 PM, Sriraman Tallam wrote: My original proposal, for x86_64 only, was to add -fno-plt=function-name. This lets the user decide for which functions PLT must be avoided. Let the compiler always generate an indirect call using call *func@GOTPCREL(%rip). We could do this

Re: [PATCH] clarify doc for __builtin_return_address

2015-05-21 Thread Martin Sebor
On 05/21/2015 02:05 PM, Sandra Loosemore wrote: On 05/21/2015 01:19 PM, Martin Sebor wrote: 2015-05-21 Martin Sebor mse...@redhat.com * extend.texi (Return Address): Clarify possible effects of calling the functions with non-zero arguments. diff --git a/gcc/doc/extend.texi

[C++ PATCH] Fix return deduction and ref-quals

2015-05-21 Thread Nathan Sidwell
I've committed this obvious patch to fix PR 60943. When deducing an auto return type we failed to propagate any reference qualification on the function type. built tested on x86_64-linux. nathan 2015-05-21 Nathan Sidwell nat...@acm.org cp/ PR c++/60943 * decl2.c (change_return_type):

Re: [C frontend] Fix construction of TYPE_STUB_DECL

2015-05-21 Thread Jason Merrill
On 05/21/2015 04:32 PM, Jan Hubicka wrote: I think at LTO time it is useful to have two things - be able to say what type comply C++ ODR rule, because we special case these for ODR warnings - be able to say at LTO time what types are anonymous, that is they are not compatible

Re: [C frontend] Fix construction of TYPE_STUB_DECL

2015-05-21 Thread Jan Hubicka
Hello, I would like to ping this. Currently we have a problem with Ada ICE because we consider a global variable produced by ada to have type in C++ anonymous namespace and we get false ODR merging wraning compiling clang because we consider instances of templates with parameter in anonymous

Re: [PATCH] clarify doc for __builtin_return_address

2015-05-21 Thread Pedro Alves
On 05/21/2015 08:19 PM, Martin Sebor wrote: A program I instrumented to help me debug an otherwise unrelated problem in 5.1.0 has been crashing in calls to __builtin_return_address. After checking the manual, I didn't think I was doing anything wrong. I then did some debugging and found that

Re: [patch] testsuite enable PIE tests on FreeBSD

2015-05-21 Thread Andreas Tobler
On 21.05.15 20:14, Andreas Tobler wrote: On 20.05.15 22:30, Jeff Law wrote: On 05/20/2015 11:04 AM, Andreas Tobler wrote: Hi, the attached patch enables some PIE tests on FreeBSD. Ok for trunk? Thanks, Andreas 2015-05-20 Andreas Tobler andre...@gcc.gnu.org *

Re: [PATCH] clarify doc for __builtin_return_address

2015-05-21 Thread Sandra Loosemore
On 05/21/2015 01:19 PM, Martin Sebor wrote: 2015-05-21 Martin Sebor mse...@redhat.com * extend.texi (Return Address): Clarify possible effects of calling the functions with non-zero arguments. diff --git a/gcc/doc/extend.texi b/gcc/doc/extend.texi index 7470e40..b37e893 100644 ---

Re: [C frontend] Fix construction of TYPE_STUB_DECL

2015-05-21 Thread Jan Hubicka
On 05/21/2015 04:32 PM, Jan Hubicka wrote: I think at LTO time it is useful to have two things - be able to say what type comply C++ ODR rule, because we special case these for ODR warnings - be able to say at LTO time what types are anonymous, that is they are not

Re: [PATCH] PR 62173, re-shuffle insns for RTL loop invariant hoisting

2015-05-21 Thread Jiong Wang
Jeff Law writes: On 05/14/2015 03:13 PM, Jiong Wang wrote: Jeff Law writes: For all kinds of reassociation we have to concern ourselves with adding overflow where it didn't already occur. Assuming a 32 bit architecture we could get overflow if A is 0x7fff, b is -4 and and c = 3

Re: [patch, testsuite, ARM] don't try to execute advsimd-intrinsics tests on hardware without NEON

2015-05-21 Thread Christophe Lyon
On 21 May 2015 at 07:33, Sandra Loosemore san...@codesourcery.com wrote: ARM testing shares the AArch64 advsimd-intrinsics execution tests. On ARM, though, the NEON support being tested is optional -- some arches are compatible with the NEON compilation options but hardware available for

Re: acc_on_device for device_type_host_nonshm

2015-05-21 Thread Jakub Jelinek
On Thu, May 21, 2015 at 01:02:12PM +0200, Thomas Schwinge wrote: Hi! On Thu, 7 May 2015 19:32:26 +0100, Julian Brown jul...@codesourcery.com wrote: Here's a new version of the patch [...] OK for trunk? Makes sense to me (with just a request to drop the testsuite changes, see

Re: [Patch AArch64] Add cpu_defines.h for AArch64.

2015-05-21 Thread Szabolcs Nagy
On 19/05/15 17:03, Ramana Radhakrishnan wrote: On Tue, May 19, 2015 at 4:54 PM, pins...@gmail.com wrote: On May 19, 2015, at 5:54 AM, Ramana Radhakrishnan ramana.radhakrish...@foss.arm.com wrote: Like the ARM port, the AArch64 ports needs to set glibc_integral_traps to false as integer

Re: C/C++ PATCH to allow deprecating enum values (PR c/47043)

2015-05-21 Thread Marek Polacek
I'm pinging the C++ parts. Thanks, On Thu, May 07, 2015 at 06:22:40PM +0200, Marek Polacek wrote: This (third) version of the patch entails the change in tsubst_enum Ed suggested + new testcase. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2015-05-07 Marek Polacek

Re: Demangle symbols in debug assertion messages

2015-05-21 Thread Jonathan Wakely
On 20/05/15 21:45 +0200, François Dumont wrote: On 20/05/2015 12:19, Jonathan Wakely wrote: Does this fix https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65392 ? With the patch this code of the bug report generates the following debug message:

Re: acc_on_device for device_type_host_nonshm

2015-05-21 Thread Thomas Schwinge
Hi! On Thu, 7 May 2015 19:32:26 +0100, Julian Brown jul...@codesourcery.com wrote: Here's a new version of the patch [...] OK for trunk? Makes sense to me (with just a request to drop the testsuite changes, see below), to get the existing regressions under control. Jakub? PR

[PATCH][AArch64] Add __extension__ and __always_inline__ to crypto intrinsics

2015-05-21 Thread Kyrill Tkachov
Hi all, The crypto intrinsics are missing an __extension__ and an __always_inline__ attribute that all the other intrinsics have. I don't see any reason for them to be different and the always_inline attribute will be needed if we decide to wrap the intrinsics inside a target SIMD pragma.

[PATCH] Fix (ocvt (icvt@1 @0)) simplification (PR tree-optimization/66233)

2015-05-21 Thread Jakub Jelinek
Hi! We ICE on the following testcase at -O3 on x86_64-linux, because gimple folding attempts to simplify FLOAT_EXPR conversion of signed V4SI to V4SF feeding FIX_TRUNC_EXPR to unsigned V4SI into a FIX_TRUNC_EXPR with unsigned V4SI lhs and signed V4SI rhs1, which is invalid GIMPLE. All the other

Re: [PATCH, fixincludes] AIX headers and extern C

2015-05-21 Thread Bruce Korb
OK. You might consider updating autogen. It seems 5.18 doesn't handle the version test quite right. Any 5.18.n should do fine. I guess I didn't test the version test with older versions. :) On Thu, May 21, 2015 at 6:58 AM, David Edelsohn dje@gmail.com wrote: The AIX port of GCC is one of

Re: RFA: PATCH to use -std=c++98 in stage 1 of bootstrap

2015-05-21 Thread Jason Merrill
On 05/20/2015 06:11 PM, Alexandre Oliva wrote: The only serious problem with the patch is that it changes Makefile.in, but not the corresponding part of Makefile.tpl from which it is generated. Ok with that change. Now, if you'd also update the comments just before it, that still suggest we

Re: [C++ PATCH] Minor fix for warn_args_num

2015-05-21 Thread Marek Polacek
On Thu, May 21, 2015 at 10:11:41AM -0400, Jason Merrill wrote: On 05/21/2015 09:44 AM, Marek Polacek wrote: + if (!DECL_BUILT_IN (fndecl)) I think you want DECL_IS_BUILTIN. OK with that change. Right. With DECL_IS_BUILTIN we print q.c:1:5: note: declared here int printf (const char

v3 PATCH to avoid -Wsized-deallocation warnings with C++14 compiler

2015-05-21 Thread Jason Merrill
When GCC defaults to C++14, it gives -Wsized-deallocation warnings for the non-sized operator deletes. We dealt with this for the sized ones by passing -Wno-sized-deallocation on the command line, but using #pragma GCC diagnostic seems cleaner to me. Applying to trunk. commit

Re: [PATCH] PR target/66224 _GLIBC_READ_MEM_BARRIER

2015-05-21 Thread Steven Munroe
On Wed, 2015-05-20 at 14:40 -0400, David Edelsohn wrote: The current definition of _GLIBC_READ_MEM_BARRIER in libstdc++ is too weak for an ACQUIRE FENCE, which is what it is intended to be. The original code emitted an isync instead of lwsync. All of the guard acquire and set code needs to

[RFA] Fix combine to canonicalize (mult X pow2)) more often

2015-05-21 Thread Jeff Law
When combine needs to split a complex insn, it will canonicalize a simple (mult X (const_int Y)) where Y is a power of 2 into the expected (ashift X (const_int Y')) if the (mult ...) is selected as a split point. However if the split point is (plus (mult (X (const_int Y)) Z) combine fails

[committed] Tweak inform_declaration

2015-05-21 Thread Marek Polacek
See https://gcc.gnu.org/ml/gcc-patches/2015-05/msg01977.html for the rationale. Bootstrapped/regtested on x86_64-linux, applying to trunk. 2015-05-21 Marek Polacek pola...@redhat.com * c-typeck.c (inform_declaration): Use DECL_IS_BUILTIN instead of DECL_BUILT_IN. diff --git

[Ada] Allow constants in SPARK contracts

2015-05-21 Thread Arnaud Charlet
This patch permits constants to appear in the following SPARK annotations: Depends Global Initializes Part_Of Refined_Depends Refined_Global Refined_State -- Source -- -- legal_usage.ads package Legal_Usage with SPARK_Mode,

[PATCH[2/n] Reduction vectorization improvements

2015-05-21 Thread Richard Biener
This is part #2, fixes wrong-code because of bogus reduction op used and factors out common code (which I'd otherwise need to duplicate once more...). Bootstrap and regtest running on x86_64-unknown-linux-gnu. Richard. 2015-05-21 Richard Biener rguent...@suse.de * tree-vect-loop.c

Re: Fix alignment propagation

2015-05-21 Thread Martin Jambor
Hi, I have not managed to ping this, get it approved and commit it in time for gcc 5 but it is a useful cleanup that clarifies a number of things and something I'd like to base further cleanups on. It still applies cleanly and I have re-tested and re-bootstrapped the patch on x86_64-linux

[PATCH, fixincludes] AIX headers and extern C

2015-05-21 Thread David Edelsohn
The AIX port of GCC is one of the few ports that does not define NO_IMPLICIT_EXTERN_C. A user reported a problem that we tracked to an AIX header that explicitly used C++ features (bracketed by #ifdef __cplusplus). AIX headers have included some C++ features, mostly protected by #if defined

Re: PING^3: [PATCH]: New configure options that make the compiler use -fPIE and -pie as default option

2015-05-21 Thread Rainer Orth
H.J. Lu hjl.to...@gmail.com writes: Here is the complete patch. Tested on Linux/x86-64. It is also available on hjl/pie/master branch in git mirror. As always, please keep generated files like configure and config.in out of the submission: it simplifies review. diff --git a/gcc/Makefile.in

Re: [RFA] Restore combine.c split point for multiply-accumulate instructions

2015-05-21 Thread Jeff Law
On 05/21/2015 07:40 AM, Segher Boessenkool wrote: On Wed, May 20, 2015 at 11:38:44PM -0600, Jeff Law wrote: I've also verified this is one of the changes ultimately necessary to resolve the code generation regressions caused by Venkat's combine.c change on the PA across my 300+ testfiles for a

Re: [patch, libgomp] Re-factor GOMP_MAP_POINTER handling

2015-05-21 Thread Jakub Jelinek
On Thu, May 21, 2015 at 03:00:16PM +0200, Thomas Schwinge wrote: Jakub, for avoidance of doubt, the proposed refactoring makes sense to me, but does need your approval: This is ok for trunk. Jakub

Re: [gomp4] Vector-single predication

2015-05-21 Thread Julian Brown
On Thu, 21 May 2015 14:38:19 +0100 Julian Brown jul...@codesourcery.com wrote: On Thu, 21 May 2015 15:21:54 +0200 Jakub Jelinek ja...@redhat.com wrote: On Thu, May 21, 2015 at 02:05:12PM +0100, Julian Brown wrote: OpenACC handles function calls specially (calling them routines -- of

[C++ PATCH] Minor fix for warn_args_num

2015-05-21 Thread Marek Polacek
I've just noticed that we print note: declared here even for builtins. E.g.: void foo (void) { __builtin_return (); } q.cc: In function ‘void foo()’: q.cc:4:21: error: too few arguments to function ‘void __builtin_return(void*)’ __builtin_return (); ^ built-in: note:

Re: [gomp4] Vector-single predication

2015-05-21 Thread Jakub Jelinek
On Thu, May 21, 2015 at 02:38:19PM +0100, Julian Brown wrote: All functions will behave that way, or just some using some magic attribute etc.? Say will newlib functions behave this way (math functions, printf, ...)? It's actually unclear at this point if regular functions are

[PATCH][1/n] Reduction vectorization improvements

2015-05-21 Thread Richard Biener
I'm at the moment tearing apart a large patch that adds support for vectorizing reductions in basic-blocks as well as making loop vectorizing reduction chains with patterns work. This is a first piece - allow the reduction patterns be detected when reduction detection didn't run and remove an

Re: [PATCH] PR target/66232: -fPIC -fno-plt -mx32 fails to generate indirect branch via GOT

2015-05-21 Thread H.J. Lu
On Thu, May 21, 2015 at 6:11 AM, Uros Bizjak ubiz...@gmail.com wrote: On Thu, May 21, 2015 at 2:59 PM, H.J. Lu hjl.to...@gmail.com wrote: X32 doesn't support indirect branch via 32-bit memory slot since indirect branch will load 64-bit address from 64-bit memory slot. Since x32 GOT slot is

Re: [gomp4] Vector-single predication

2015-05-21 Thread Julian Brown
On Thu, 21 May 2015 15:21:54 +0200 Jakub Jelinek ja...@redhat.com wrote: On Thu, May 21, 2015 at 02:05:12PM +0100, Julian Brown wrote: OpenACC handles function calls specially (calling them routines -- of varying sorts, gang, worker, vector or seq, affecting where they can be invoked

Re: [RFA] Restore combine.c split point for multiply-accumulate instructions

2015-05-21 Thread Segher Boessenkool
On Wed, May 20, 2015 at 11:38:44PM -0600, Jeff Law wrote: I've also verified this is one of the changes ultimately necessary to resolve the code generation regressions caused by Venkat's combine.c change on the PA across my 300+ testfiles for a PA cross compiler. How much does it help, do

Re: [C++ PATCH] Minor fix for warn_args_num

2015-05-21 Thread Jason Merrill
On 05/21/2015 09:44 AM, Marek Polacek wrote: + if (!DECL_BUILT_IN (fndecl)) I think you want DECL_IS_BUILTIN. OK with that change. Jason

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

2015-05-21 Thread Sriraman Tallam
On Thu, May 21, 2015 at 2:51 PM, Pedro Alves pal...@redhat.com wrote: On 05/21/2015 10:12 PM, Sriraman Tallam wrote: My original proposal, for x86_64 only, was to add -fno-plt=function-name. This lets the user decide for which functions PLT must be avoided. Let the compiler always generate

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

2015-05-21 Thread H.J. Lu
On Thu, May 21, 2015 at 2:58 PM, Jakub Jelinek ja...@redhat.com wrote: On Thu, May 21, 2015 at 10:51:50PM +0100, Pedro Alves wrote: On 05/21/2015 10:12 PM, Sriraman Tallam wrote: My original proposal, for x86_64 only, was to add -fno-plt=function-name. This lets the user decide for which

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

2015-05-21 Thread Jakub Jelinek
On Thu, May 21, 2015 at 10:51:50PM +0100, Pedro Alves wrote: On 05/21/2015 10:12 PM, Sriraman Tallam wrote: My original proposal, for x86_64 only, was to add -fno-plt=function-name. This lets the user decide for which functions PLT must be avoided. Let the compiler always generate an

Re: [PATCH] PR target/66232: -fPIC -fno-plt -mx32 fails to generate indirect branch via GOT

2015-05-21 Thread H.J. Lu
On Thu, May 21, 2015 at 12:01 PM, H.J. Lu hjl.to...@gmail.com wrote: On Thu, May 21, 2015 at 11:41 AM, Richard Henderson r...@redhat.com wrote: On 05/21/2015 05:59 AM, H.J. Lu wrote: +(define_predicate x32_sibcall_memory_operand + (and (match_operand 0 memory_operand) + (match_test

[C++ PATCH] fix canonical type ICE

2015-05-21 Thread Nathan Sidwell
Jason, this fixes 65936, where we have two identical types with mismatching TYPE_CANONICAL. The problem comes from template decl creation vs template instantiation. At the point we create 'const Cint' in the operator function we have not applied the 'may_alias' attribute to 'const Cint'.

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

2015-05-21 Thread Xinliang David Li
We have -finstrument-functions-exclude-function-list=.. in GCC, though it is not using mangled names. David On Thu, May 21, 2015 at 2:58 PM, Jakub Jelinek ja...@redhat.com wrote: On Thu, May 21, 2015 at 10:51:50PM +0100, Pedro Alves wrote: On 05/21/2015 10:12 PM, Sriraman Tallam wrote: My

Re: [C++ PATCH] fix canonical type ICE

2015-05-21 Thread Jason Merrill
How about adding may_alias support to the code a bit lower down that copies the abi_tag attribute? Jason

Re: [C frontend] Fix construction of TYPE_STUB_DECL

2015-05-21 Thread Jason Merrill
On 05/21/2015 05:34 PM, Jan Hubicka wrote: This would also work, yes. We can set it into something like anonymous. One problem would be that type_with_linkage_p/type_in_anonymous_namespace_p would not work on non-C++ types without LTO (because then we do not produce the type manglings). I

  1   2   >