Re: [Patch, gcc/flag-types.h + Fortran] PR54687 - Fortran options cleanup

OK, I think, from the Fortran POV. I hope I didn't miss some logic issue in the middle of the trivial stuff. Thanks for doing that work! FX Le 12 déc. 2014 à 08:43, Tobias Burnus a écrit : > This patch cleans up Fortran's option handling and moves it closer to the > common way of option han

Re: [PATCH 1/n] OpenMP 4.0 offloading infrastructure

Hi! On Tue, 30 Sep 2014 13:16:37 +0200, I wrote: > On Fri, 26 Sep 2014 16:36:21 +0400, Ilya Verbin wrote: > > --- a/configure.ac > > +++ b/configure.ac > > @@ -286,6 +286,24 @@ case ${with_newlib} in > >yes) skipdirs=`echo " ${skipdirs} " | sed -e 's/ target-newlib / /'` ;; > > esac > > >

Re: [PATCH 1/n] OpenMP 4.0 offloading infrastructure

On Fri, Dec 12, 2014 at 09:14:28AM +0100, Thomas Schwinge wrote: > On Tue, 30 Sep 2014 13:16:37 +0200, I wrote: > > On Fri, 26 Sep 2014 16:36:21 +0400, Ilya Verbin wrote: > > > --- a/configure.ac > > > +++ b/configure.ac > > > @@ -286,6 +286,24 @@ case ${with_newlib} in > > >yes) skipdirs=`ech

RE: [PATCH, combine] Try REG_EQUAL for nonzero_bits

> -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Eric Botcazou > Sent: Monday, November 24, 2014 5:41 PM > To: Zhenqiang Chen > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [PATCH, combine] Try REG_EQUAL for nonzero_bits > > >

Patch ping

Hi! I'd like to ping 3 patches: http://gcc.gnu.org/ml/gcc-patches/2014-12/msg00546.html PR63831 - P1 - fix __has_attribute/__has_cpp_attribute handling http://gcc.gnu.org/ml/gcc-patches/2014-12/msg00568.html PR64023 - P3 - fix flags passed to non-bootstrapped host modules during bootstrap

Re: [PATCH][testsuite] Mark as UNSUPPORTED tests that don't fit into tiny memory model

Kyrill Tkachov writes: > * lib/target-utils.exp: New file. ERROR: Couldn't find library file target-utils.exp. make[4]: *** [check-DEJAGNU] Error 1 make[4]: Leaving directory `/usr/local/gcc/gcc-20141212/Build/ia64-suse-linux/libgomp/testsuite' make[3]: *** [check-am] Err

Re: r218609 - in /trunk/gcc: ChangeLog common.opt d...

> hubi...@gcc.gnu.org writes: > > > Author: hubicka > > Date: Wed Dec 10 21:17:28 2014 > > New Revision: 218609 > > > > URL: https://gcc.gnu.org/viewcvs?rev=218609&root=gcc&view=rev > > Log: > > * doc/invoke.texi: (-devirtualize-at-ltrans): Document. > > * lto-cgraph.c (lto_output_varpool_

Re: Overload HONOR_INFINITIES, etc macros

On Thu, Dec 11, 2014 at 9:47 PM, Marc Glisse wrote: > Hello, > > after HONOR_NANS, I am turning the other HONOR_* macros into functions. As a > reminder, the goal is both to make uses shorter and to fix the answer for > non-native vector types. > > Bootstrap+testsuite on x86_64-linux-gnu. Ok. Th

Re: Remove unused arguments of bulitin_unreachable

On Thu, Dec 11, 2014 at 7:16 PM, Jan Hubicka wrote: >> On Thu, Dec 11, 2014 at 06:06:55PM +0100, Jan Hubicka wrote: >> > Hi, >> > in firefox .optimized dumps one can see few places where >> > __builtin_unreachable >> > is called (as a result of devirtualization code proving the code path to be >>

Re: [patch] Fix ICE on unaligned record field

On Thu, Dec 11, 2014 at 10:52 PM, Eric Botcazou wrote: >> Note that I think the place of the check is unfortunate as you for example >> will not remove the argument if it is unused. In fact I'm not yet sure >> what transform exactly we are disabling. I am guessing we are >> passing an aggregate

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

Hi! On Mon, 10 Nov 2014 17:30:38 +0300, Ilya Verbin wrote: > --- /dev/null > +++ b/liboffloadmic/plugin/Makefile.am > @@ -0,0 +1,123 @@ > +# Plugin for offload execution on Intel MIC devices. > +libgomp_src_dir = $(top_srcdir)/../../libgomp > +libgomp_dir = $(build_dir)/../../libgomp Hmm, I'm n

Re: [PATCH 5/n] OpenMP 4.0 offloading infrastructure: libgomp

Hi! I know, I'm a little late, but: On Mon, 6 Oct 2014 19:53:17 +0400, Ilya Verbin wrote: > This patch adds plugin support to libgomp, as well as memory mapping and > interaction with target devices through plugin's interface. > libgomp/ > * libgomp_target.h: New file. > --- /dev/null >

Re: [PATCH][testsuite] Mark as UNSUPPORTED tests that don't fit into tiny memory model

On 12/12/14 08:34, Andreas Schwab wrote: Kyrill Tkachov writes: * lib/target-utils.exp: New file. ERROR: Couldn't find library file target-utils.exp. make[4]: *** [check-DEJAGNU] Error 1 make[4]: Leaving directory `/usr/local/gcc/gcc-20141212/Build/ia64-suse-linux/libgomp/test

Re: [PATCH][testsuite] Mark as UNSUPPORTED tests that don't fit into tiny memory model

On 12/12/14 10:22, Kyrill Tkachov wrote: On 12/12/14 08:34, Andreas Schwab wrote: Kyrill Tkachov writes: * lib/target-utils.exp: New file. ERROR: Couldn't find library file target-utils.exp. make[4]: *** [check-DEJAGNU] Error 1 make[4]: Leaving directory `/usr/local/gcc/gcc-201

Re: [PATCH 2/4] Add liboffloadmic

Hi! On Tue, 21 Oct 2014 21:20:34 +0400, Ilya Verbin wrote: > This patch contains liboffloadmic library. > liboffloadmic/ > Initial commit. Imported from upstream: > https://www.openmprtl.org/sites/default/files/liboffload_oss.tgz > * Makefile.am: New file. > * Makefile.i

Re: [patch] Fix ICE on unaligned record field

> Well, I call it a convenience that MEM_EXPR, unlike INDIRECT_REF, can > be used to encapsulate an arbitrary byte-offset and view-conversion. Of > course it's still a dereference of an address so that convenience doesn't > work on sth non-addressable. No discussion on the merits of MEM_EXPR vs I

[patch c++]: Fix PR/63996

Hi, The loop-expression loops endless in c++14's case for cases the statement-list isn't constant. Bug 63996 - Infinite loop in invalid C++14 constexpr fn ChangeLog 2014-12-12 Kai Tietz PR c++/63996 * constexpr.c (cxx_eval_loop_expr): Don't loop endless on none-constant expressi

[PATCH][rtlanal.c][BE][1/2] Fix vector load/stores to not use ld1/st1

[Cleaning this thread up to submit patch again, with better explanation] This patch causes subreg_get_info() to exit early in the simple cases where we are extracting a whole register from a multi register. In aarch64 for Big Endian we were producing a subreg of a OImode (256bits) from a CImode (

Re: [PATCH] Fix PR 61225

On Fri, Dec 12, 2014 at 03:27:17PM +0800, Zhenqiang Chen wrote: > > Presumably you're thinking about a PARALLEL that satisfies single_set_p? > > No. It has nothing to do with single_set_p. I just want to reuse the code to > match the instruction pattern. > > In common, the new PARALLEL is like >

Re: [PATCH][AArch64] Use std::swap instead of manually swapping

Ping. Marcus: Uros pointed out to me that these kinds of changes are considered obvious (with precedent athttps://gcc.gnu.org/ml/gcc-patches/2014-12/msg00309.html) but did you have some concerns about backporting to other branches? Kyrill On 05/12/14 16:40, Kyrill Tkachov wrote: Ping. https

Re: [PATCH][testsuite] Mark as UNSUPPORTED tests that don't fit into tiny memory model

> Here it is. Committed as r218662 to get things going again. The same change should be done for libitm and libatomic. TIA Dominique

[WWW] Update index.html and gcc-5/changes.html to reflect offloading changes.

Hello, These change adds mention of OpenMP4 offloading support in GCC: in release notes and in news section of main page. Index: htdocs/index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/index.html,v retrieving revision 1.942 diff -p -r

[WWW] Update index.html and gcc-5/changes.html to AVX-512* changes.

Hello, This change mantions AVX-512* new instructions support in GCC: news section of index.html and gcc-5/changes.html. Index: htdocs/index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/index.html,v retrieving revision 1.942 diff -p -r1

[PATCH] Fix PR64280

The following fixes PR64280 by properly guarding the assert with whether we are going to change SSA_NAME_OCCURS_IN_ABNORMAL_PHI. Recent changes in what we allow to propagate otherwise happily will trigger it. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. 2014-12-12 Ri

[PATCH] Fix PR64284

The following patch fixes PR64284 by removing loops we copied the header for with FSM threading. There may be better solutions, but at least for the testcase it looks difficult to update loops within the constraints of the calling passes. Bootstrap and regtest ongoing on x86_64-unknown-linux-gnu

Re: [PATCH][testsuite] Mark as UNSUPPORTED tests that don't fit into tiny memory model

On 12/12/14 11:38, Dominique Dhumieres wrote: Here it is. Committed as r218662 to get things going again. The same change should be done for libitm and libatomic. Committed as r218664. I grepped for where else we include gcc-defs.exp and included the file there as well. That was libvtv and l

Re: [gofrontend-dev] Re: [PATCH 00/13] Go closures, libffi, and the static chain

On Thu, Dec 11, 2014 at 11:56:00AM -0800, Richard Henderson wrote: > On 12/11/2014 04:25 AM, Dominik Vogt wrote: > > Update: If I disable the custom s390x code and switch to the > > implementation just using libffi for reflection calls, the same > > crash occurs with the testing/quick libgo test c

[PATCH, x86][PIC] Making check for PIC register in address cost calculation only on RTL level

Hi! When adding checks for PIC register in address cost calculation (http://gcc.gnu.org/ml/gcc-cvs/2014-10/msg00411.html) it was meant to affect only RTL passes. Since !pic_offset_table_rtx is not enough for it (I see that pic_offset_table_rtx enabled on GIMPLE level) following change explicitl

[committed] Add two testcases for recently fixed PRs

Hi! I've committed these two testcases as obvious to trunk, they have been fixed with the PR63917 r218658 fix. 2014-12-12 Jakub Jelinek PR rtl-optimization/64255 * gcc.c-torture/execute/pr64255.c: New test. PR rtl-optimization/64260 * gcc.c-torture/execute/pr6

[PATCH] Fix simplify_builtin_call forwprop ICE (PR tree-optimization/64269)

Hi! This testcase ICEs because I wasn't checking for overflow in the size computation. Only max (diff + len2, len1) <= 1024 cases are considered, so this patch gives up if either len2 or diff is > 1024. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk/4.9/4.8? 2014-12-12 Jak

[committed, moxie] Add use of zex instruction

I've committed the following patch for the moxie port... 2014-12-12 Anthony Green * config/moxie/moxie.md: Add use of zex instruction. Index: gcc/config/moxie/moxie.md === --- gcc/config/moxie/moxie.md (revision 21866

Re: [PATCH] Fix simplify_builtin_call forwprop ICE (PR tree-optimization/64269)

On Fri, 12 Dec 2014, Jakub Jelinek wrote: > Hi! > > This testcase ICEs because I wasn't checking for overflow in the size > computation. Only max (diff + len2, len1) <= 1024 cases are considered, > so this patch gives up if either len2 or diff is > 1024. > > Bootstrapped/regtested on x86_64-lin

[patch] LWG DR 2285: std::make_reverse_iterator

This is a trivial addition, implementing http://cplusplus.github.io/LWG/lwg-defects.html#2285 I've also added the __cpp_lib_tuple_element_t macro missing from the commit that implemented that. Tested x86_64-linux, committed to trunk. commit 363e23c37b089661c7533476cc7b3e5b05f20f9f Author: Jonath

[PATCH] Fix TYPE_OVERFLOW_* cleanup fallout

On ARM we ICE on fixed-point-exec.c testcase because TYPE_OVERFLOW_WRAPS was missing the ANY_INTEGRAL_TYPE_P check. Bootstrapped/regtested on x86_64-linux and ppc64-linux, ok for trunk? 2014-12-12 Marek Polacek PR middle-end/64274 * fold-const.c (fold_binary_loc): Add ANY_INTE

Re: [PATCH] Fix TYPE_OVERFLOW_* cleanup fallout

On Fri, 12 Dec 2014, Marek Polacek wrote: > On ARM we ICE on fixed-point-exec.c testcase because TYPE_OVERFLOW_WRAPS > was missing the ANY_INTEGRAL_TYPE_P check. > > Bootstrapped/regtested on x86_64-linux and ppc64-linux, ok for trunk? Ok. THanks, Richard. > 2014-12-12 Marek Polacek > >

Re: [PATCH][ARM] Make issue rate part of per-core tuning structs

Ping (after the macro fusion patch)... https://gcc.gnu.org/ml/gcc-patches/2014-11/msg02706.html Thanks, Kyrill On 20/11/14 16:48, Kyrill Tkachov wrote: I should say that the patch context depends on the macro fusion hook implementation posted here: https://gcc.gnu.org/ml/gcc-patches/2014-11/msg0

Re: [gofrontend-dev] Re: [PATCH 00/13] Go closures, libffi, and the static chain

On Thu, Dec 11, 2014 at 07:51:44PM +1030, Alan Modra wrote: > I was worried about exactly the same "problem" on powerpc with r11 > being used for the static chain and also destroyed in linkage stubs. > It turns out we don't traverse any linkage stubs. > > See https://gcc.gnu.org/ml/gcc-patches/201

Re: [PATCH] Fix TYPE_OVERFLOW_* cleanup fallout

On 12/12/14 13:48, Marek Polacek wrote: On ARM we ICE on fixed-point-exec.c testcase because TYPE_OVERFLOW_WRAPS was missing the ANY_INTEGRAL_TYPE_P check. Bootstrapped/regtested on x86_64-linux and ppc64-linux, ok for trunk? Also arm-none-linux-gnueabihf bootstrap is in late stage3 and looks

Re: [PATCH 0/4] [AARCH64,SIMD] PR63870 Improve error messages for single lane load/store

On 10 December 2014 at 10:34, Alan Lawrence wrote: > Thanks, Charles. A couple of thoughts. > > I think the approach in patches 2+3+4 of using > __builtin_aarch64_im_lane_boundsi is justified and works quite neatly. > Modulo the question of argument ordering and __AARCH64_LANE_CHECK, those > patch

[hsa] HSA: function comments are added.

Hello. In this small patch I add missing comments. Thanks, Martin gcc/ChangeLog: 2014-12-12 Martin Liska * hsa-brig.c: Function comments are added. * hsa-gen.c: Likewise. --- gcc/hsa-brig.c | 5 + gcc/hsa-gen.c | 19 ++- 2 files changed, 23 ins

[hsa] HSA: memory leaks are fixed.

Hello. Attached patch removes all memory leaks which come from HSA-related source files. Thanks, Martin gcc/ChangeLog: 2014-12-05 Martin Liska * hsa-brig.c (brig_string_slot_hasher::remove): Memory free is added. * hsa-gen.c (hsa_deinit_data_for_cfun): Destructors are called

[hsa] HSA: support for direct function call is introduced.

Hello. Following patch introduces support for direct call instructions for HSAIL. Thanks, Martin gcc/c-family/ChangeLog: 2014-12-05 Martin Liska * c-common.c: New 'hsafunc' attribute is added. gcc/ChangeLog: 2014-12-05 Martin Liska * hsa-brig.c (struct function_linkage

[PATCH][AArch64] Generalize code alignment

This patch generalizes the code alignment and lets each CPU set function, jump and loop alignment independently. The defaults for A53/A57 are based the original patch by James Greenhalgh. OK for trunk? ChangeLog: 2014-12-13 Wilco Dijkstra wdijk...@arm.com * gcc/config/aarch64/aarch64-proto

[patch] libstdc++/64241 fix std::make_exception_ptr() for -fno-exceptions

It's not clear to me that std::make_exception_ptr() is actually useful with -fno-exceptions, but we might as well return something well-defined rather than garbage that might crash the program. Tested x86_64-linux, committed to trunk. commit ab57ab82fb2f5eec3bbb081f63d7a53de26bc7c8 Author: Jonath

[PATCH][AArch64] Add TARGET_MIN_DIVISIONS_FOR_RECIP_MUL

Add an override for TARGET_MIN_DIVISIONS_FOR_RECIP_MUL and set the minimum number of divisions to 2. This gives ~0.5% speedup on SPECFP2000/2006. OK for trunk? ChangeLog: 2014-12-13 Wilco Dijkstra wdijk...@arm.com * gcc/config/aarch64/aarch64.c (TARGET_MIN_DIVISIONS_FOR_RECIP_MUL):

Re: [Patch, Fortran] Convert gfc_notify_std to common diagnostics

> PING - https://gcc.gnu.org/ml/gcc-patches/2014-12/msg00731.html OK.

[PATCH][AArch64] Fix PR 64263: Do not try to split constants when destination is SIMD reg

Hi all, Since the movsi_aarch64 and movdi_aarch64 patterns became splitters we want to make sure that the splitting happens only when we deal with GP registers. This patch guards the splitting part by GP_REGNUM_P rather than trying to complicate aarch64_expand_mov_immediate too much to try a

Re: [PATCH] [AArch64, NEON] Fix testcases add by r218484

On 11 December 2014 at 08:50, Yangfei (Felix) wrote: > Hi, > > We find that the committed patch is not correctly generated from our local > branch. This caused some code necessary for the testcases missing. > As pointed out by Christophe in > https://gcc.gnu.org/ml/gcc-patches/2014-12/msg00

Re: [PATCH, Libatomic, Darwin] Initial libatomic port for *darwin*.

Iain, What is the status of this patch? Jack On Thu, Nov 13, 2014 at 3:34 PM, Iain Sandoe wrote: > Hello Richard, Joseph, > > Thanks for your reviews, > > On 13 Nov 2014, at 07:40, Richard Henderson wrote: > >> On 11/12/2014 10:18 PM, Iain Sandoe wrote: >> >>> # ifndef USE_ATOMIC

Re: PR64182: Fix rounding division and modulus

Richard Biener writes: > On Thu, Dec 11, 2014 at 1:26 PM, Richard Sandiford > wrote: >> As pointed out in PR 64182, wide-int rounded division gets the >> ties-away-from-zero case wrong for odd-numbered dividends, while >> double_int gets the unsigned case wrong by unconditionally treating >> a di

Re: [patch c++]: Fix PR/63996

OK. Jason

[patch] libstdc++/64276 replace __EXCEPTIONS and __GXX_RTTI with SD-6 macros

This replaces the GCC-specific macros with the portable feature-testing macros that are now supported by GCC. Tested x86_64-linux, committed to trunk. commit 6565657776c8c9ebe4055510f2485ccc695e23ef Author: Jonathan Wakely Date: Fri Dec 12 15:11:39 2014 + PR libstdc++/64276 * d

[gomp4] Merge trunk r217500 (2014-11-13) into gomp-4_0-branch

Hi! In r218677, I have committed a merge from trunk r217500 (2014-11-13) into gomp-4_0-branch. This spans the "offloading" commits in trunk (plus two interleaved non-"offloading" commits). Grüße, Thomas pgp4Q8SRXYvvi.pgp Description: PGP signature

Re: [PATCH][rtlanal.c][BE][1/2] Fix vector load/stores to not use ld1/st1

Alan Hayward writes: > [Cleaning this thread up to submit patch again, with better explanation] > > This patch causes subreg_get_info() to exit early in the simple cases > where we are extracting a whole register from a multi register. > > In aarch64 for Big Endian we were producing a subreg of a

Re: [PATCH 5/n] OpenMP 4.0 offloading infrastructure: libgomp

On Fri, Dec 12, 2014 at 10:58:39AM +0100, Thomas Schwinge wrote: > > --- /dev/null > > +++ b/libgomp/libgomp_target.h > > @@ -0,0 +1,44 @@ > > +/* Copyright (C) 2014 Free Software Foundation, Inc. > > + > > + This file is part of the GNU OpenMP Library (libgomp). > > > +#ifndef LIBGOMP_TARGET_H

C++ PATCH for c++/61402

This patch fixes the ICE in 61402, though I'll leave it open for the unsequenced execution warning issue. Tested x86_64-pc-linux-gnu, applying to trunk. commit 05beed857dc4e01061a38b764c26f1ff857788dd Author: Jason Merrill Date: Fri Dec 12 10:43:59 2014 -0500 PR c++/61402 * lambda.

[PATCH] Fix for PR ipa/64278

Hello. This is patch for PR ipa/64278, where I replace ambiguous std::abs with absu_hwi. Patch can bootstrap on ppc64-linux and no new regression was seen. Ready for thunk? Thanks, Martin >From 03a15009e5c9a9045669a4987588d8abf8cc67f1 Mon Sep 17 00:00:00 2001 From: mliska Date: Fri, 12 Dec 2

Re: [PING][PATCH V2] plugin event for C/C++ function definitions

PING, request for maintainer please 2014-12-02 15:15 GMT-03:00 Andres Tiraboschi : > Hi, this patch adds a new plugin event PLUGIN_START_PARSE_FUNCTION and > PLUGIN_FINISH_PARSE_FUNCTION that are invoked at start_function and > finish_function respectively in the C and C++ frontends. > PLUGIN_ST

Re: [PATCH] Fix for PR ipa/64146

Martin, Your test g++.dg/ipa/pr64146.C fails on darwin: grep bind pr64146.C.051i.icf returns nothing, so the first scan fails, while the second one succeeds. Dominique

C++ PATCH for N3922 (direct-initialization from { x })

N3922 changed Unicorn initialization such that while copy-list-initialization of an auto variable still deduces std::initializer_list, direct-list-initialization from a list with a single element deduces the type of the element and from a list with multiple elements is ill-formed. I've made t

C++ PATCH for abi_tag attribute on namespaces

By treating a namespace name as an ABI tag we can avoid most of the need for tagging class types. Tested x86_64-pc-linux-gnu, applying to trunk. commit 65b7f772bb3240688aae6af41d0a1200972885ae Author: Jason Merrill Date: Tue Dec 2 17:11:37 2014 -0500 * cp-tree.h (NAMESPACE_ABI_TAG): New

Re: [gofrontend-dev] Re: [PATCH 00/13] Go closures, libffi, and the static chain

On 12/12/2014 04:06 AM, Dominik Vogt wrote: > I'm not sure I've posted the missing patch anywhere yet, so it's > attached to this message. At the moment it enables > FFI_TYPE_COMPLEX only for s390[x], but eventually this should be > used unconditionally. Thanks for that. I'd been meaning to get

Re: [gofrontend-dev] Re: [PATCH 00/13] Go closures, libffi, and the static chain

On 12/12/2014 05:57 AM, Dominik Vogt wrote: > On Thu, Dec 11, 2014 at 07:51:44PM +1030, Alan Modra wrote: >> I was worried about exactly the same "problem" on powerpc with r11 >> being used for the static chain and also destroyed in linkage stubs. >> It turns out we don't traverse any linkage stubs

Re: [PATCH, AARCH64] Fix ICE in CCMP (PR64015)

> - tree lhs = gimple_assign_lhs (g); > enum machine_mode mode = TYPE_MODE (TREE_TYPE (lhs)); > rtx target = gen_reg_rtx (mode); > + > + start_sequence (); > tmp = emit_cstore (target, icode, NE, cc_mode, cc_mode, >0, tmp, const0_rtx,

[Patch][testsuite] Fix a few test cases

Hi, Here are a few test tweaks. In 921202-1.c, if STACK_SIZE is used then VLEN will blow the stack with 64bit longs. e.g. if STACK_SIZE == 512K then 3 arrays of 32767 longs means at a minimum 767K of stack will be used at -O0. In pr51447.c, the rbx global register is clobbering the rbx of mai

Re: [Patch]: Check __gthread_setspecific return

On 14-12-05 05:53 PM, Jeff Law wrote: On 12/02/14 10:53, Ryan Mansfield wrote: Hi, Underlying pthread_setspecific can return non-zero with ENOMEM or EINVAL. 2014-12-02 Ryan Mansfield * emutls.c (__emutls_get_address): Check __gthread_setspecific returns. OK? OK. Thanks Jeff. C

[patch c++]: Fix 61228 - noexcept(expression) causes internal compiler error

Hi, following patch fixes reported issue. Tested for x86_64-w64-mingw32. Ok for apply? Regards, Kai ChangeLog 2014-12-12 Kai Tietz PR c++/61228 * call.c (set_flags_from_callee): Assume no throw by deferred noexcept. 2014-12-12 Kai Tietz PR c++/61228 * g++.dg/cpp0x/

Re: [gofrontend-dev] Re: [PATCH 00/13] Go closures, libffi, and the static chain

On Fri, Dec 12, 2014 at 10:49 AM, Richard Henderson wrote: > > Oh, that's interesting. You've found a bug in the x86_64 linking: > > gccgo -g3 -O3 -Wall -Werror -c -fPIC q.go -o q.o > gccgo -shared -Wl,-soname,libq.so -o libq.so q.o > gccgo -g3 -O3 -Wall -Werror -c -fPIC p.go -o p.o > gccgo -shar

Re: [patch c++]: Fix 61228 - noexcept(expression) causes internal compiler error

Hi, On 12/12/2014 08:45 PM, Kai Tietz wrote: #include #include I would recommend reducing the testcase further, is very large. Thanks, Paolo.

Re: OpenACC GIMPLE_OACC_* -- or not?

Hi! On Wed, 10 Dec 2014 11:10:47 +0100, Jakub Jelinek wrote: > On Wed, Dec 10, 2014 at 11:07:37AM +0100, Thomas Schwinge wrote: > > ..., I noticed that GIMPLE_OMP_TARGET doesn't walk the child_fn and > > data_arg. Is that intentional, or should that be done? If the latter > > (but this doesn't

Re: Nested OpenACC/OpenMP constructs

Hi! On Wed, 10 Dec 2014 11:16:08 +0100, Jakub Jelinek wrote: > On Wed, Dec 10, 2014 at 11:10:19AM +0100, Thomas Schwinge wrote: > > --- /dev/null > > +++ gcc/testsuite/c-c++-common/gomp/nesting-1.c > > @@ -0,0 +1,77 @@ > > +void > > +f_omp_parallel (void) > > +{ > > +#pragma omp parallel > > + {

patch to fix PR64110

The following patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64110 The patch was successfully tested and bootstrapped on x86/x86-64. Committed as rev. 218688. 2014-12-12 Vladimir Makarov PR target/64110 * lra-constraints.c (process_alt_operands): Refuse alternativ

Re: [patch c++]: Fix 61228 - noexcept(expression) causes internal compiler error

I think it would be better to call maybe_instantiate_noexcept so that we can have a definite answer. Jason

Re: The nvptx port [10/11+] Target files

Hi! On Mon, 10 Nov 2014 17:19:57 +0100, Bernd Schmidt wrote: > I've now committed it, in the following form. > --- /dev/null > +++ b/gcc/config/nvptx/nvptx.h > @@ -0,0 +1,356 @@ > +#define ASM_OUTPUT_ALIGN(FILE, POWER) Committed to trunk in r218689: commit 61f8a1bd770ded96fcff88f3cbc426a23c4

Re: [patch c++]: Fix PR/63996

Hi, On 12/12/2014 11:58 AM, Kai Tietz wrote: New testcase in g++.dg/cpp1y as pr63996.C // { dg-do compile { target c++14 } } constexpr int foo (int i) { int a[i] = { }; } constexpr int j = foo (1); // { dg-error "is not a constant expression" } The testcase fails spuriously because of Jaso

Re: [patch c++]: Fix 61228 - noexcept(expression) causes internal compiler error

2014-12-12 21:15 GMT+01:00 Jason Merrill : > I think it would be better to call maybe_instantiate_noexcept so that we can > have a definite answer. > > Jason Hmm, for case that decl != NULL_TREE, this is ok. But what if decl is NULL_TREE? Kai

[patch] Fix std::shared_ptr FAILs with -fno-rtti

A couple of small fixes for shared_ptr tests that fail with -fno-rtti. Tested x86_64-linux, committed to trunk. commit ec3619710af701b1030a6eb45862a41dc3e18ad8 Author: Jonathan Wakely Date: Fri Dec 12 16:17:45 2014 + PR libstdc++/58594 * include/bits/shared_ptr_base.h: Cast awa

Re: [Patch, libstdc++/64239] Fix regex_iterator copying

On 11/12/14 00:27 -0800, Tim Shen wrote: As discussed in Bugzilla. Bootstrapped and tested. OK for trunk. Is it Ok to backport it to 4.9 branch, with _M_in_iterator kept unused? Yes, that's OK as well. Thanks.

Re: [PATCH 0/4] GCC port for the Visium

On Dec 11, 2014, at 4:05 PM, Eric Botcazou wrote: > on behalf of Controls and Data Services, AdaCore would like to contribute a > port of the GCC to the Visium. > OK for the mainline? The test suite bits are usual and customary. I’ll Ok them explicitly anyway. I’m fine with new ports going in

[C++ Patch] PR 59628

Hi, I think that to avoid ICE-ing during error recovery when DECL_SAVED_TREE is NULL_TREE we want to simply bail out and return true to the caller. Tested x86_64-linux. Thanks, Paolo. // /cp 2014-12-12 Paolo Carlini PR c++/59628 * semantics.c (finish_omp_r

Re: [C++ Patch] PR 59628

On Fri, Dec 12, 2014 at 10:13:39PM +0100, Paolo Carlini wrote: > Hi, > > I think that to avoid ICE-ing during error recovery when DECL_SAVED_TREE is > NULL_TREE we want to simply bail out and return true to the caller. Tested > x86_64-linux. > > Thanks, > Paolo. > > // > /cp > 2

Re: [PATCH][AArch64] Use std::swap instead of manually swapping

On Dec 12, 2014, at 3:36 AM, Kyrill Tkachov wrote: > Marcus: Uros pointed out to me that these kinds of changes are considered > obvious > (with precedent athttps://gcc.gnu.org/ml/gcc-patches/2014-12/msg00309.html) > but did you have some concerns about backporting to other branches? So in gener

Re: [Patch][testsuite] Fix a few test cases

On Dec 12, 2014, at 11:38 AM, Ryan Mansfield wrote: > > Here are a few test tweaks. In 921202-1.c, if STACK_SIZE is used then VLEN > will blow the stack with 64bit longs. e.g. if STACK_SIZE == 512K then 3 > arrays of 32767 longs means at a minimum 767K of stack will be used at -O0. > In pr5144

[PATCH] Fix -fsanitize=float-cast-overflow with C FE (PR sanitizer/64289)

Hi! -fsanitize=float-cast-overflow sanitization is done in convert.c and calls there save_expr. Unfortunately, save_expr is a no-go for the C FE, we need c_save_expr, but as convert.c is shared by all FEs, the only way to arrange that would be a new langhook. This patch attempts to fix it the sa

[PATCH] Ensure __tsan_func_entry call isn't in a loop (PR sanitizer/64265)

Hi! This patch ensures that if successor of entry bb has multiple predecessors, we emit the __tsan_func_entry call on the edge from entry bb, so it can't be called inside a loop in the same function. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2014-12-12 Jakub Jelinek

RE: [PATCHv2,MIPS 1/2] MIPS64r6 support

Hi Matthew, > -Original Message- > From: Matthew Fortune [mailto:matthew.fort...@imgtec.com] > Sent: Friday, November 14, 2014 6:07 PM Overall, this patch looks really good. It took me a while to get through it, but I only have a couple of minor comments. diff --git a/gcc/config/mips/m

RE: [PATCHv2,MIPS 2/2] Add new triplets for vendor 'img'

> -Original Message- > From: Matthew Fortune [mailto:matthew.fort...@imgtec.com] > Sent: Friday, November 14, 2014 6:20 PM > To: 'gcc-patches@gcc.gnu.org' (gcc-patches@gcc.gnu.org) > Cc: Moore, Catherine; Rich Fuhler; Steve Ellcey; Richard Sandiford > Subject: [PATCHv2,MIPS 2/2] Add new tri

RE: [PATCHv2,MIPS 1/2] MIPS64r6 support

Moore, Catherine writes: > Hi Matthew, > > > -Original Message- > > From: Matthew Fortune [mailto:matthew.fort...@imgtec.com] > > Sent: Friday, November 14, 2014 6:07 PM > > Overall, this patch looks really good. It took me a while to get > through it, but I only have a couple of minor

Re: [PATCH] PR other/63613: Add fixincludes for dejagnu.h

On Dec 11, 2014, at 5:07 AM, Rainer Orth wrote: > David Malcolm writes: >>> * I don't consider this a critical issue that cannot work without >>> current releases. We're already working around several upstream >>> DejaGnu issues in our codebase, and I don't consider this particular >>> one im

Re: [patch] Fix std::shared_ptr FAILs with -fno-rtti

On 12/12/14 21:06 +, Jonathan Wakely wrote: A couple of small fixes for shared_ptr tests that fail with -fno-rtti. Tested x86_64-linux, committed to trunk. Huh, somehow I committed that when the new test wasn't even passing. This fixes it properly. Tested again and committed to trunk. co

libgo patch committed: Avoid GC crash with callbacks in new thread

There is a private program that crashes when using gccgo in a rather complex scenario. Newly created C threads call into Go code, forcing the Go code to allocate new M and G structures. While executing Go code, the stack is split. The Go code then returns. Returning from a Go callback is treate

libgo patch committed: Add testing.MainStart

In Go 1.4 the "go test" command uses the new function testing.MainStart. That function will be normally brought into libgo when we upgrade it to the 1.4 library. However, for convenience for people using gccgo with the Go 1.4 gc release, I've committed this patch to bring it into gccgo now. Boot

Go patch committed: Don't move nil subexpressions into temporaries

This patch from Chris Manghane changes the Go frontend to avoid moving nil subexpressions into temporary variables. This fixes GCC PR 61254, a compiler crash on some rather unlikely, but valid, code. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r

Re: [Patch][testsuite] Fix a few test cases

On 14-12-12 04:29 PM, Mike Stump wrote: On Dec 12, 2014, at 11:38 AM, Ryan Mansfield wrote: Here are a few test tweaks. In 921202-1.c, if STACK_SIZE is used then VLEN will blow the stack with 64bit longs. e.g. if STACK_SIZE == 512K then 3 arrays of 32767 longs means at a minimum 767K of stac

libgo patch committed: Permit delete from map with zero length key

A map with a zero length key is useless, but valid. gccgo was incorrectly crashing when deleting an entry from such a map. This patch by Chris Manghane fixes the problem. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 28f37e5c57fe libgo/runtim

Re: [PATCH] [AArch64, NEON] Fix testcases add by r218484

Thanks for reviewing the patch. See my comments inlined: > > This patch fix this two issues. Three changes: > > 1. vfma_f32, vfmaq_f32, vfms_f32, vfmsq_f32 are only available for > arm*-*-* target with the FMA feature, we take care of this through the macro > __ARM_FEATURE_FMA. > > 2. vfm

Re: [patch c++]: Fix 61228 - noexcept(expression) causes internal compiler error

On 12/12/2014 03:58 PM, Kai Tietz wrote: 2014-12-12 21:15 GMT+01:00 Jason Merrill : I think it would be better to call maybe_instantiate_noexcept so that we can have a definite answer. Jason Hmm, for case that decl != NULL_TREE, this is ok. But what if decl is NULL_TREE? We shouldn't have