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

On Tue, 1 Sep 2015, Martin Sebor wrote: > Attached is an updated patch that avoids diagnosing taking the address > of implicitly declared library builtins like abs, bootstrapped and > tested on ppc64le with no regressions. > > The tweak below was added to reject_gcc_builtin make it possible. >

Re: [Fortran, committed] XFAIL read_dir.f90 on FreeBSD

On Tue, Sep 01, 2015 at 11:16:27AM -0700, Steve Kargl wrote: > open(unit=10, file='junko.dir',iostat=ios,action='read',access='stream') > if (ios.ne.0) call abort > read(10, iostat=ios) c > - if (ios.ne.21) call abort > + if (ios.ne.21) then > + close(10) I forgot to mention

patch for PR61578

The following patch is for https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61578 The patch was bootstrapped and tested on x86 and x86-64. Committed as rev. 227382. 2015-09-01 Vladimir Makarov PR target/61578 * lra-lives.c (process_bb_lives): Process

Re: [PATCH][wwwdocs][AArch64] Add entry for target attributes and pragmas

On Tue, 1 Sep 2015, Kyrill Tkachov wrote: This wwwdocs patch adds an entry to the GCC 6 changes page about the aarch64 target attributes and pragmas support. Thanks for thinking of this, Kyrill. Index: htdocs/gcc-6/changes.html

[Fortran, committed] XFAIL read_dir.f90 on FreeBSD

I've committed the patch that follows my .sig. 2015-09-01 Steven G. Kargl * gfortran.dg/read_dir.f90: XFAIL this testcase on FreeBSD. Clean-up a created directory if testcase fails. I suspect that this testcase will fail on all *BSD OS's. -- Steve Index:

Re: [PATCH] add initial support for J2 core to sh target

On Wed, Sep 02, 2015 at 01:24:55AM +0900, Oleg Endo wrote: > > I'm not sure what the best way to achieve multiple goals is, but the > > current behavior makes it so you need --isa=any (and a final binary > > with weird ABI tag) to have a binary that supports atomic operations > > on any SH model.

Re: [PATCH] [libstdc++] Run tests on RTEMS

On Sep 1, 2015, at 3:05 AM, Sebastian Huber wrote: > libstdc++-v3/ChangeLog > 2015-09-01 Sebastian Huber > >testsuite/*: Use 's/dg-do run.*\*-\*-cygwin\* /&*-*-rtems* /' to >add RTEMS target selector to all tests

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

On 08/26/2015 08:53 AM, Oleg Endo wrote: On 26 Aug 2015, at 23:27, Oleg Endo wrote: On 19 Aug 2015, at 22:35, Jeff Law wrote: On 08/19/2015 06:29 AM, David Sherwood wrote: I asked Richard S. to give this a once-over which he did. However, he

[RS6000] Weak functions may not be file local

A weak symbol defined in the current object file may not turn out to be the definition used at link time, if other copies of the symbol exist. This means they can't be considered file local; Calls to weak functions need a nop so that the linker can replace it with a toc restoring insn if the

[gomp4] useless reduction locks and other bug fixes

This patch teaches lower_oacc_reductions not to generate calls to GOACC_{UN}LOCK if they aren't any reductions. That situation can happen when there is a fake gang reduction on a private variable. I also found a bug where the lower_rec_input_clauses expects there to be a data mapping for the

fixed misplaced testcase

2015-09-01 Kenneth Zadeck * gcc.c-torture/execute/ieee/2320-1.c Fixed misplaced test case. This was approved offline by Mike Stump. committed as revision 227389. Kenny --- gcc/testsuite/gcc.c-torture/execute/ieee/2320-1.c (revision 227385) +++

[PATCH PR66388]Add sizetype cand for BIV of smaller type if it's used as index of memory ref

Hi, This patch is a new approach to fix PR66388. IVO today computes iv_use with iv_cand which has at least same type precision as the use. On 64bit platforms like AArch64, this results in different iv_cand created for each address type iv_use, and register pressure increased. As a matter of

RE: [PATCH] PR 60586

> -Original Message- > From: Jeff Law [mailto:l...@redhat.com] > Sent: Tuesday, September 1, 2015 3:26 PM > To: Iyer, Balaji V; gcc-patches@gcc.gnu.org > Cc: Zamyatin, Igor > Subject: Re: [PATCH] PR 60586 > > On 08/31/2015 06:04 PM, Iyer, Balaji V wrote: > > Hello Everyone, > > This

Re: [RS6000] Don't pass --oformat to ld

On Tue, Sep 1, 2015 at 9:35 PM, Alan Modra wrote: > bugzilla.redhat.com/show_bug_cgi?id=1255946 shows that gcc built with > both powerpc64-linux and powerpc64le-linux support passes wrong linker > options when trying to link in the non-default endian. A --oformat > option

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

On Tue, 1 Sep 2015, Martin Sebor wrote: > I also noticed uses of DECL_LANG_FLAG_4 in the definitions of > what appear to be C-specific macros in c-family/c-common.h, > and then uses of the same macro in definitions of a C++-specific > macro in cp/cp-tree.h. That seems like a bug waiting to

Re: [gomp4] useless reduction locks and other bug fixes

[Attaching patch this time.] On 09/01/2015 04:21 PM, Cesar Philippidis wrote: > This patch teaches lower_oacc_reductions not to generate calls to > GOACC_{UN}LOCK if they aren't any reductions. That situation can happen > when there is a fake gang reduction on a private variable. > > I also

Go patch committed: handle out of range constant integer to string conversion

This patch by Chris Manghane fixes the Go compiler to correctly handle an integer to string conversion when the integer is an out of range constant. This fixes https://golang.org/issue/11525 . Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian Index:

RE: [PATCH] PR 60586

> -Original Message- > From: Iyer, Balaji V > Sent: Tuesday, September 1, 2015 6:17 PM > To: 'Jeff Law'; gcc-patches@gcc.gnu.org > Cc: Zamyatin, Igor > Subject: RE: [PATCH] PR 60586 > > > > > -Original Message- > > From: Jeff Law [mailto:l...@redhat.com] > > Sent: Tuesday,

[RS6000] Don't pass --oformat to ld

bugzilla.redhat.com/show_bug_cgi?id=1255946 shows that gcc built with both powerpc64-linux and powerpc64le-linux support passes wrong linker options when trying to link in the non-default endian. A --oformat option coming from LINK_TARGET_SPEC is only correct for 32-bit. It turns out that GNU ld

Re: [PATCH v2] [libstdc++] Run tests on RTEMS

On 09/01/2015 05:02 AM, Sebastian Huber wrote: v2: Include all options and not only "dg-do run ...". libstdc++-v3/ChangeLog 2015-09-01 Sebastian Huber testsuite/*: Use 's/\*-\*-cygwin\* /&*-*-rtems* /' to add RTEMS target selector to all

Re: [PATCH] PR 60586

On 08/31/2015 06:04 PM, Iyer, Balaji V wrote: Hello Everyone, This patch will fix the bug reported in Bugzilla, PR 60586. The issue was that the spawned function's function arguments must not be pushed into the nested/lambda function. This patch should fix that issue. I have tested

Re: [Fortran, committed] XFAIL read_dir.f90 on FreeBSD

On 09/01/2015 11:18 AM, Steve Kargl wrote: > On Tue, Sep 01, 2015 at 11:16:27AM -0700, Steve Kargl wrote: >> open(unit=10, file='junko.dir',iostat=ios,action='read',access='stream') >> if (ios.ne.0) call abort >> read(10, iostat=ios) c >> - if (ios.ne.21) call abort >> + if

Re: [Patch, libfortran] PR 67414 Improve error handling

On Tue, Sep 1, 2015 at 1:04 AM, FX wrote: >> the attached patch improves the error handling for backtrace failing, >> by printing the error number or the error string in addition to the >> message. It also fixes a potential null pointer crash in gf_strerror. >> >> Regtested

RE: [PATCH, MIPS] Compact branch support for MIPS32R6/MIPS64R6

Moore, Catherine writes: > Hi Matthew: > > > -Original Message- > > From: Matthew Fortune [mailto:matthew.fort...@imgtec.com] > > Sent: Monday, August 17, 2015 6:47 PM > > To: Moore, Catherine; 'gcc-patches@gcc.gnu.org' (gcc-patches@gcc.gnu.org) > > Subject:

Re: [PATCH] [ARM, Callgraph] Fix PR67280: function incorrectly marked as nothrow

On 08/28/2015 09:03 AM, Charles Baylis wrote: Hi This patch is an attempt to fix https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67280. I have written up an analysis of the bug there. When cgraph_node::create_wrapper() updates the callgraph for the new function, it sets the can_throw_external

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

On 09/01/2015 11:29 AM, Joseph Myers wrote: On Tue, 1 Sep 2015, Martin Sebor wrote: Attached is an updated patch that avoids diagnosing taking the address of implicitly declared library builtins like abs, bootstrapped and tested on ppc64le with no regressions. The tweak below was added to

Re: [PATCH][optabs][ifcvt][1/3] Define negcc, notcc optabs

On 09/01/2015 09:04 AM, Kyrill Tkachov wrote: Hi all, This first patch introduces the negcc and notcc optabs that should expand to a conditional negate or a conditional bitwise complement operation. These are used in ifcvt.c to transform code of the form: if (test) x = -A; else x = A; into: x

[FYI][PATCH] libgo/runtime: S/390: Workaround for broken setcontext in Gcc.

The attached patch is a workaround for a Bug in Glibc versions on S/390 where setcontext() screws up the processes signal mask. The patch is meant to be an optional addon for people who need it. Please check the commit comment in the patch for a detailed description. ** Please do NOT apply this

Fix 61441

The following patch fixes 61441. It converts sNaN to qNaN on folding when -fno-signaling-nans is used. Bootstrap and regression tests on x86_64-linux-gnu and aarch64-unknown-linux-gnu passed with changes done on trunk. Is this fix fine ? Regards, Sujoy 2015-09-01 Sujoy Saraswati

Re: [PATCH] Import liboffloadmic from upstream

Ilya Verbin writes: (...) > abidiff: ../../src/abg-comparison.cc:10731: virtual void > abigail::comparison::fn_parm_diff::report(std::ostream&, const string&) > const: Assertion `get_type_diff() && get_type_diff()->to_be_reported()' > failed. > Aborted (core dumped)

Re: [PATCH][AArch64] Fix FAIL: gcc.target/aarch64/target_attr_crypto_ice_1.c (internal compiler error)

Ping. https://gcc.gnu.org/ml/gcc-patches/2015-08/msg01084.html Thanks, Kyrill On 19/08/15 14:41, Kyrill Tkachov wrote: Hi all, This fixes the ICE exposed by Alexandre's patch (https://gcc.gnu.org/ml/gcc-patches/2015-08/msg00873.html) The solution I came up with is to re-layout the parameter

Re: [PATCH] Fix TERed insn locations

On Tue, Sep 01, 2015 at 12:09:48PM +0200, Richard Biener wrote: > > So I finally got around to dig into why some testcases I inspected for > early LTO debug (and some existing guality FAILs) happen. They happen > because we screw up locations of insns coming from TERed SSA defs > which end up

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

2015-09-01 10:15 GMT+02:00 Kai Tietz : > 2015-08-31 22:19 GMT+02:00 Kai Tietz : >> 2015-08-31 21:43 GMT+02:00 Kai Tietz : >>> 2015-08-31 21:29 GMT+02:00 Jason Merrill : On 08/31/2015 03:08 PM, Kai

[PATCH] Fix TERed insn locations

So I finally got around to dig into why some testcases I inspected for early LTO debug (and some existing guality FAILs) happen. They happen because we screw up locations of insns coming from TERed SSA defs which end up using the TERed-into stmt location. So final re-building the BLOCK tree

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

Hi James, On 01/09/15 10:25, James Greenhalgh wrote: On Thu, Aug 13, 2015 at 01:36:50PM +0100, Kyrill Tkachov wrote: Some comments below. Thanks, I'll incorporate them, with one clarification inline. diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c index

Re: [PATCH] Updated LTO early debug patch

On 2015.09.01 at 10:00 +0200, Richard Biener wrote: > On Mon, 31 Aug 2015, Markus Trippelsdorf wrote: > > > On 2015.08.31 at 16:44 +0200, Richard Biener wrote: > > > > > > So the state below now will pass LTO bootstrap (fingers crossing, > > > stage3 running) as well as regular bootstrap. Iff I

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

Ping. Thanks, Kyrill On 20/08/15 09:24, Kyrill Tkachov wrote: Ping. https://gcc.gnu.org/ml/gcc-patches/2015-08/msg00710.html Thanks, Kyrill On 03/08/15 14:01, James Greenhalgh wrote: On Fri, Jul 24, 2015 at 11:55:33AM +0100, Kyrill Tkachov wrote: Hi all, This patch implements an

Re: [PATCH] Fix ICE when generating a vector shift by scalar

On Mon, Aug 31, 2015 at 10:28 PM, Bill Schmidt wrote: > Hi, > > The following simple test fails when attempting to convert a vector > shift-by-scalar into a vector shift-by-vector. > > typedef unsigned char v16ui __attribute__((vector_size(16))); > > v16ui

[PATCH][AArch64] Use preferred aliases for CSNEG, CSINC, CSINV

Hi all, The ARMv8-A reference manual says: "CNEG , , is equivalent to CSNEG , , , invert() and is the preferred disassembly when Rn == Rm && cond != '111x'." That is, when the two input registers are the same we can use the shorter CNEG mnemonic with the inverse condition instead of the

Re: [PATCH] Updated LTO early debug patch

On Mon, 31 Aug 2015, Markus Trippelsdorf wrote: > On 2015.08.31 at 16:44 +0200, Richard Biener wrote: > > > > So the state below now will pass LTO bootstrap (fingers crossing, > > stage3 running) as well as regular bootstrap. Iff I didn't break > > sth in the last minute. You need up-to-date

Re: [PATCH][ARM][3/3] Expand mod by power of 2

Ping. Thanks, Kyrill On 19/08/15 13:50, Kyrill Tkachov wrote: Ping. https://gcc.gnu.org/ml/gcc-patches/2015-08/msg00448.html Thanks, Kyrill -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Kyrill Tkachov Sent: 10 August 2015

[Patch] PR67351 Implement << N & >> N optimizers

Hi, Please find attached the patch "pr67351.patch" that implements the pattern << N & >> N optimizers. Please review and let me know if its okay. Regression tested on AARH64 and x86_64. Thanks, Naveen 2015-09-01 Naveen H.S gcc/ChangeLog:

Re: [Patch] PR67351 Implement << N & >> N optimizers

On Tue, Sep 1, 2015 at 10:57 AM, Hurugalawadi, Naveen wrote: > Hi, > > Please find attached the patch "pr67351.patch" that implements the > pattern << N & >> N optimizers. + (bit_and @0 (lshift { build_minus_one_cst (type); } @1 please use

Re: [PATCH][AArch64] Fix FAIL: gcc.target/aarch64/target_attr_crypto_ice_1.c (internal compiler error)

On Wed, Aug 19, 2015 at 02:41:35PM +0100, Kyrill Tkachov wrote: > Hi all, > > This fixes the ICE exposed by Alexandre's patch > (https://gcc.gnu.org/ml/gcc-patches/2015-08/msg00873.html) > The solution I came up with is to re-layout the parameter decls not during > expansion time (when RTL has

Commit: MSP430: Fix typo in --target-help text

Hi DJ, I am applying the patch below as an obvious fix for a typo in the msp430's --target-help output. Cheers Nick gcc/ChangeLog 2015-09-01 Nick Clifton * config/msp430/msp430.opt (mcpu): Fix typo. Index: gcc/config/msp430/msp430.opt

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

2015-08-31 22:19 GMT+02:00 Kai Tietz : > 2015-08-31 21:43 GMT+02:00 Kai Tietz : >> 2015-08-31 21:29 GMT+02:00 Jason Merrill : >>> On 08/31/2015 03:08 PM, Kai Tietz wrote: I will need to verify that this patch doesn't

Re: [PATCH] Updated LTO early debug patch

On Tue, 1 Sep 2015, Markus Trippelsdorf wrote: > On 2015.09.01 at 10:00 +0200, Richard Biener wrote: > > On Mon, 31 Aug 2015, Markus Trippelsdorf wrote: > > > > > On 2015.08.31 at 16:44 +0200, Richard Biener wrote: > > > > > > > > So the state below now will pass LTO bootstrap (fingers

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

On Thu, Aug 13, 2015 at 01:36:50PM +0100, Kyrill Tkachov wrote: Some comments below. > diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c > index 1394ed7..c8bd8d2 100644 > --- a/gcc/config/aarch64/aarch64.c > +++ b/gcc/config/aarch64/aarch64.c > @@ -6652,6 +6652,25 @@

Re: [gomp4] expunge shared_size from launch API

On 31/08/15 19:39, Nathan Sidwell wrote: * builtin-types.def (DEF_FUNCTION_TYPE_VAR_6): Define. Committed attached follow-up patch to fix the ada build. Thanks, - Tom Fix gomp-4_0-branch ada build 2015-09-01 Tom de Vries * gcc-interface/utils.c

RE: Fix intelmic-mkoffload.c if the temp path contains a '-'

> -Original Message- > From: Jakub Jelinek [mailto:ja...@redhat.com] > Sent: Tuesday, September 01, 2015 1:50 PM > To: Hahnfeld, Jonas > Cc: gcc-patches@gcc.gnu.org; Ilya Verbin; Kirill Yukhin > Subject: Re: Fix intelmic-mkoffload.c if the temp path contains a '-' > > On Tue, Sep 01, 2015

[testsuite, fortran] XFAIL gfortran.dg/execute_command_line_2.f90 (PR libfortran/67412)

As discussed in the PR, gfortran.dg/execute_command_line_2.f90 execution FAILs on Solaris 10 due to a bug in /bin/sh's exit code handling. Since the fix is involved and the issue a corner case, we decided to XFAIL execution instead (at least until an eventual fix is forthcoming). Tested with the

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

2015-09-01 10:43 GMT+02:00 Kai Tietz : > 2015-09-01 10:15 GMT+02:00 Kai Tietz : >> 2015-08-31 22:19 GMT+02:00 Kai Tietz : >>> 2015-08-31 21:43 GMT+02:00 Kai Tietz : 2015-08-31 21:29 GMT+02:00

[PATCH] [RTEMS] Update RTEMS thread model

For synchronization objects use the self-contained objects available via Newlib . https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;a=commit;h=ecaef05f6601f1e8acb78fb65b411a258f39988a Enable the C++11 threads using . So, the threads are only supported in case the POSIX API is enabled in

Re: Fix intelmic-mkoffload.c if the temp path contains a '-'

On Tue, Sep 01, 2015 at 11:35:15AM +, Hahnfeld, Jonas wrote: > >From 884b6199179e7a604474bc6a828a6861d3ff4501 Mon Sep 17 00:00:00 2001 > From: Jonas Hahnfeld > Date: Thu, 20 Aug 2015 12:13:55 +0200 > Subject: [PATCH] Fix intelmic-mkoffload.c if the temp path

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

Hi, Fix/patch committed. Tested: x86_64-linux: bootstrap built fine, no regressions aarch64-none-elf: no regressions sh-elf crossbuild: builds fine ChangeLog: 2015-09-01 David Sherwood gcc/ * genmodes.c: Add CONST_MODE_UNIT_SIZE modifier. Thanks,

Re: Fix 61441

On Tue, Sep 1, 2015 at 12:23 PM, Sujoy Saraswati wrote: > The following patch fixes 61441. It converts sNaN to qNaN on folding > when -fno-signaling-nans is used. > > Bootstrap and regression tests on x86_64-linux-gnu and > aarch64-unknown-linux-gnu passed with changes done

[PATCH] libstdc++/67408

2015-09-01 Sebastian Huber PR libstdc++/67408 * include/std/mutex (__timed_mutex_impl): Use __timedlock(). (timed_mutex): Add friend class __timed_mutex_impl. Add __timedlock(). (recursive_timed_mutex): Likewise. ---

Re: [PATCH] Updated LTO early debug patch

On 2015.09.01 at 13:15 +0200, Jan Hubicka wrote: > > > And it breaks quickly during Firefox build, e.g.: > > > > > > trippels@gcc2-power8 angle % cat glslang_lex.ii > > > struct A; > > > template class B { > > > typedef A value_type; > > > void m_fn1(const value_type &); > > > }; > > >

RE: Fix intelmic-mkoffload.c if the temp path contains a '-'

Hopefully CC'ing the right people... > -Original Message- > From: Hahnfeld, Jonas > Sent: Thursday, August 20, 2015 12:25 PM > To: 'gcc-patches@gcc.gnu.org' > Subject: Fix intelmic-mkoffload.c if the temp path contains a '-' > > Hi all, > > during my test of OpenMP 4.0 offloading

[testsuite] Don't xfail gcc.dg/vect/no-scevccp-outer-11.c

It seems that since 20150717, gcc.dg/vect/no-scevccp-outer-11.c XPASSes everywhere: XPASS: gcc.dg/vect/no-scevccp-outer-11.c scan-tree-dump-times vect "OUTER LOOP VECTORIZED." 1 To reduce testsuite noise, I'd like to remove the xfail as follows. Tested with the appropriate runtest invocations

[Patch, libstdc++] Fix data races in basic_string implementation

Hello, The refcounted basic_string implementation contains several data races on _M_refcount: 1. _M_is_leaked loads _M_refcount concurrently with mutations of _M_refcount. This loads needs to be memory_order_relaxed load, as _M_is_leaked predicate does not change under the feet. 2. _M_is_shared

Re: [gomp4] expunge shared_size from launch API

On 09/01/15 06:15, Tom de Vries wrote: On 31/08/15 19:39, Nathan Sidwell wrote: * builtin-types.def (DEF_FUNCTION_TYPE_VAR_6): Define. Committed attached follow-up patch to fix the ada build. thanks

Re: [PATCH] Updated LTO early debug patch

> > And it breaks quickly during Firefox build, e.g.: > > > > trippels@gcc2-power8 angle % cat glslang_lex.ii > > struct A; > > template class B { > > typedef A value_type; > > void m_fn1(const value_type &); > > }; > > struct A { > > int staticUse; > > B<> fields; > > }; > > > >

Re: [PATCH] Import liboffloadmic from upstream

On Tue, Sep 01, 2015 at 09:58:22 +0200, Dodji Seketeli wrote: > Woops. can you send me the exact two libraries so that I can see what's > going wrong? You can quickly file an issue to > https://sourceware.org/bugzilla/enter_bug.cgi?product=libabigail or just > send me the two binaries by email.

Re: [gomp4] New reduction infrastructure for OpenACC

On 19/08/15 21:42, Cesar Philippidis wrote: +DEFHOOK +(reduction, +"This hook is used by the oacc_transform pass to expand calls to the\n\ +internal functions @var{GOACC_REDUCTION_SETUP},\n\ +@var{GOACC_REDUCTION_INIT},\n\ @var{GOACC_REDUCTION_FINI} and\n\ @var{GOACC_REDUCTION_TEARDOWN} into a

Re: [testsuite] Don't xfail gcc.dg/vect/no-scevccp-outer-11.c

Rainer Orth wrote: It seems that since 20150717, gcc.dg/vect/no-scevccp-outer-11.c XPASSes everywhere: XPASS: gcc.dg/vect/no-scevccp-outer-11.c scan-tree-dump-times vect "OUTER LOOP VECTORIZED." 1 To reduce testsuite noise, I'd like to remove the xfail as follows. Tested with the appropriate

[PATCH GCC]Try to fold (long)(A-B) into (long)A - (long)B for canonicalization in tree affine

Hi, Generally we don't try to fold (long)(A-B) into (long)A - (long)B because it results in more operations. On the other hand, this fold is wanted when we want to explore as many canonical opportunities as possible. Tree affine is definitely such a place. This patch supports this in

RE: [PATCH] PR 60586

Hi Jeff, I thought about this for a minute and I don't think I need to use the lang_hooks. I could do this change right before calling gimplify_cilk_spawn. I have attached the fixed patch and have answered your questions below. Here are the ChangeLog entries: gcc/c-family/ChangeLog:

[PATCH GCC]Look into unnecessary conversion when checking mult_op in get_shiftadd_cost

Hi, When calling get_shiftadd_cost, the mult_op is stripped at caller places. We should look into unnecessary conversion in op1 before checking equality, otherwise it computes wrong shiftadd cost. This patch picks this small issue up. Bootstrap and test on x86_64 and aarch64 along with other

Re: [gomp4] New reduction infrastructure for OpenACC

On 01/09/15 14:53, Tom de Vries wrote: On 19/08/15 21:42, Cesar Philippidis wrote: +DEFHOOK +(reduction, +"This hook is used by the oacc_transform pass to expand calls to the\n\ +internal functions @var{GOACC_REDUCTION_SETUP},\n\ +@var{GOACC_REDUCTION_INIT},\n\ @var{GOACC_REDUCTION_FINI} and\n\

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

On 27/08/15 03:37, Cesar Philippidis wrote: - ctx->ganglocal_size_host = align_and_expand (_host, host_size, align); I suspect this caused a bootstrap failure (align_and_expand unused). Worked-around as attached. Thanks, - Tom Mark align_and_expand as unused 2015-09-01 Tom de Vries

Re: [gomp4] declare directive

On 12/08/15 20:31, James Norris wrote: diff --git a/gcc/cp/pt.c b/gcc/cp/pt.c index 056b2c1..8ace93c 100644 --- a/gcc/cp/pt.c +++ b/gcc/cp/pt.c @@ -13907,6 +13907,7 @@ tsubst_expr (tree t, tree args, tsubst_flags_t complain, tree in_decl, integral_constant_expression_p) tree

Re: Location of "dg-final" directives? (was Re: [PATCH][GCC] Algorithmic optimization in match and simplify)

On Tue, Sep 01, 2015 at 12:50:27PM -0400, David Malcolm wrote: > I can't comment on the patch itself, but I noticed that in the testsuite > addition, you've gathered all the "dg-final" clauses at the end. > > I think that this is consistent with existing practice in gcc, but > AFAIK, the dg-final

Re: [PATCH] add initial support for J2 core to sh target

On 01 Sep 2015, at 23:18, Rich Felker wrote: > On Tue, Sep 01, 2015 at 10:45:10PM +0900, Oleg Endo wrote: >> It seems that this J2 atomic instruction(s ?) is not available to >> the public. I've skimmed through the currently available J2 hardware >> sources, but couldn't find

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

On 27/08/15 03:37, Cesar Philippidis wrote: - tree ganglocal_size - = gimple_call_arg (goacc_kernels_internal, /* TODO */ 6); - gimple_omp_target_set_ganglocal_size (stmt, ganglocal_size); This caused a bootstrap failure. Committed as attached. Thanks, - Tom Remove unused var

Location of "dg-final" directives? (was Re: [PATCH][GCC] Algorithmic optimization in match and simplify)

On Fri, 2015-08-28 at 17:56 +0100, Andre Vieira wrote: [..snip..] > diff --git a/gcc/testsuite/gcc.dg/tree-ssa/forwprop-33.c > b/gcc/testsuite/gcc.dg/tree-ssa/forwprop-33.c > new file mode 100644 > index > ..984d8b37a01defe0e6852070a7dfa7ace5027c51 > ---

[PATCH][wwwdocs][AArch64] Add entry for target attributes and pragmas

Hi all, This wwwdocs patch adds an entry to the GCC 6 changes page about the aarch64 target attributes and pragmas support. Ok for the wwwdocs repo? Thanks, Kyrill Index: htdocs/gcc-6/changes.html === RCS file:

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

On 08/31/2015 03:43 PM, Kai Tietz wrote: 2015-08-31 21:29 GMT+02:00 Jason Merrill : On 08/31/2015 03:08 PM, Kai Tietz wrote: I will need to verify that this patch doesn't introduce regressions. The wacky thing here is the encapsulation of overflowed-arguments in

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

Attached is an updated patch that avoids diagnosing taking the address of implicitly declared library builtins like abs, bootstrapped and tested on ppc64le with no regressions. The tweak below was added to reject_gcc_builtin make it possible. Since the expression is in c-family/c-common.c, the

[PATCH, PR67405, committed] Avoid NULL pointer dereference

Hi, This fixes an ICE by adding a NULL check. Bootstrapped and regtested for x86_64-unknown-linux-gnu. Applied to trunk. Does this need to be ported to gcc-5-branch? Thanks, Ilya -- gcc/ 2015-09-01 Ilya Enkovich PR target/67405 * tree-chkp.c

[PATCH] Add __builtin_argument_pointer

On Wed, Aug 19, 2015 at 3:35 PM, Segher Boessenkool wrote: > On Wed, Aug 19, 2015 at 03:18:46PM -0700, H.J. Lu wrote: >> @deftypefn {Built-in Function} {void *} __builtin_argument_pointer (void) >> This function is similar to @code{__builtin_frame_address} with an >>

[PATCH][optabs][ifcvt][0/3] Define negcc, notcc optabs and use for if-conversion

Hi all, This patch series aims to improve the use of conditional negate and conditional invert instructions when if-converting code of the form: x := test ? -a : a; or x := test ? ~a : a; I'd like to generate something like: x := a; if (test) x := -x; where the latter if-statement can be

[PATCH][AArch64][2/3] Implement negcc, notcc optabs

Hi all, This second patch implements the new optabs for aarch64. The new expander is almost identical to the movcc expander except that operand 2 has a neg or a not before it to reflect the fact that it should be negated if the comparison in operand 1 holds. These patterns will eventually match

[PATCH][ARM][3/3] Implement negsicc, notsicc optabs

Hi all, This third patch implements the new optabs for arm. Conveniently, we can reuse the recently refactored *if_neg_move pattern and extend it to cover the conditional NOT case. Although arm has conditional execution capabilities I have found that performing the transformation from patch 1/3

[PATCH][optabs][ifcvt][1/3] Define negcc, notcc optabs

Hi all, This first patch introduces the negcc and notcc optabs that should expand to a conditional negate or a conditional bitwise complement operation. These are used in ifcvt.c to transform code of the form: if (test) x = -A; else x = A; into: x = A; if (test) x = -x; where the "if (test) x

Re: [committed] taking transactional credits

On Tue, Sep 1, 2015 at 4:10 PM, Aldy Hernandez wrote: > On 09/01/2015 07:03 AM, Richard Henderson wrote: >> >> On 09/01/2015 06:55 AM, Aldy Hernandez wrote: >>> >>> If you don't like it, sue me. =) >> >> >> Share the blame... ;-) > > > That's right! I ain't going down in

Re: [Patch, libstdc++] Fix data races in basic_string implementation

On 01/09/15 16:56 +0200, Dmitry Vyukov wrote: I don't understand how a new gcc may not support __atomic builtins on ints. How it is even possible? That's a portable API provided by recent gcc's... The built-in function is always defined, but it might expand to a call to an external function in

Re: [committed] taking transactional credits

On 09/01/2015 08:05 AM, Patrick Marlier wrote: On Tue, Sep 1, 2015 at 4:10 PM, Aldy Hernandez wrote: On 09/01/2015 07:03 AM, Richard Henderson wrote: On 09/01/2015 06:55 AM, Aldy Hernandez wrote: If you don't like it, sue me. =) Share the blame... ;-) That's right!

Re: [Patch, libstdc++] Fix data races in basic_string implementation

On Tue, Sep 1, 2015 at 4:27 PM, Jonathan Wakely wrote: > On 01/09/15 14:51 +0200, Dmitry Vyukov wrote: >> >> Hello, >> >> The refcounted basic_string implementation contains several data races >> on _M_refcount: >> 1. _M_is_leaked loads _M_refcount concurrently with mutations

Re: [PATCH][ARM][3/3] Implement negsicc, notsicc optabs

> On Sep 1, 2015, at 11:04 PM, Kyrill Tkachov wrote: > > Hi all, > > This third patch implements the new optabs for arm. > Conveniently, we can reuse the recently refactored *if_neg_move pattern > and extend it to cover the conditional NOT case. > Although arm has

[PATCH][ARM][3/3] Implement negsicc, notsicc optabs

[Resending with patch attached] Hi all, This third patch implements the new optabs for arm. Conveniently, we can reuse the recently refactored *if_neg_move pattern and extend it to cover the conditional NOT case. Although arm has conditional execution capabilities I have found that performing

[RFC] Try vector as a new representation for vector masks

On 27 Aug 09:55, Richard Biener wrote: > On Wed, Aug 26, 2015 at 5:51 PM, Ilya Enkovich wrote: > > > > Yes, I want to try it. But getting rid of bool patterns would mean > > support for all targets currently supporting vec_cond. Would it be OK > > to have vector mask

Re: [testsuite] Don't xfail gcc.dg/vect/no-scevccp-outer-11.c

On Tue, 1 Sep 2015, Rainer Orth wrote: > It seems that since 20150717, gcc.dg/vect/no-scevccp-outer-11.c XPASSes > everywhere: > > XPASS: gcc.dg/vect/no-scevccp-outer-11.c scan-tree-dump-times vect "OUTER > LOOP VECTORIZED." 1 > > To reduce testsuite noise, I'd like to remove the xfail as

Re: [PATCH] gcc/fold-const.c: Correct the report warning position.

On 8/31/15 19:12, Richard Biener wrote: > On Sat, Aug 29, 2015 at 2:57 PM, Chen Gang > wrote: >> >> It is about bug63510: current input_location isn't precise for reporting >> warning. The correct location is gimple location of current statement. > > Looks ok to me.

Re: [PATCH] add initial support for J2 core to sh target

Hi Rich, On 01 Sep 2015, at 02:49, Rich Felker wrote: > The J2 Core is an open hardware cpu implementing the SH-2 instruction > set, with the addition of barrel shift instructions and an atomic > compare-and-swap instruction. This patch adds a cpu model option -mj2 > to the sh

Re: [PATCH][GCC] Algorithmic optimization in match and simplify

Hi Marc, On 28/08/15 19:07, Marc Glisse wrote: (not a review, I haven't even read the whole patch) On Fri, 28 Aug 2015, Andre Vieira wrote: 2015-08-03 Andre Vieira * match.pd: Added new patterns: ((X {&,<<,>>} C0) {|,^} C1) {^,|} C2) (X {|,^,&}

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

2015-09-01 13:17 GMT+02:00 Kai Tietz : > 2015-09-01 10:43 GMT+02:00 Kai Tietz : >> 2015-09-01 10:15 GMT+02:00 Kai Tietz : >>> 2015-08-31 22:19 GMT+02:00 Kai Tietz : 2015-08-31 21:43 GMT+02:00

[committed] taking transactional credits

I've had too many sleepless nights caused by this code, not to warrant tattooing my name on it. If you don't like it, sue me. =) Aldy commit 7d92b913093c35cb6fb322d4dc9acb237cecabfe Author: Aldy Hernandez Date: Tue Sep 1 06:51:45 2015 -0700 + * trans-mem.c: Add

Re: [RS6000] Weak functions may not be file local

On Tue, Sep 1, 2015 at 3:51 AM, Alan Modra wrote: > A weak symbol defined in the current object file may not turn out to > be the definition used at link time, if other copies of the symbol > exist. This means they can't be considered file local; Calls to > weak functions need

Re: [Patch, libstdc++] Fix data races in basic_string implementation

On 01/09/15 14:51 +0200, Dmitry Vyukov wrote: Hello, The refcounted basic_string implementation contains several data races on _M_refcount: 1. _M_is_leaked loads _M_refcount concurrently with mutations of _M_refcount. This loads needs to be memory_order_relaxed load, as _M_is_leaked predicate

  1   2   >