Re: [hsa merge 09/10] Majority of the HSA back-end

2016-01-16 Thread Jakub Jelinek
On Sat, Jan 16, 2016 at 12:49:12AM +0100, Martin Jambor wrote: > bootstrapping on i686-linux revealed the need for the following simple > patch. I've run into two types of compilation errors on > powerpc-ibm-aix (no htolenn functions and ASM_GENERATE_INTERNAL_LABEL > somehow expanding to undeclare

Re: [hsa merge 09/10] Majority of the HSA back-end

2016-01-16 Thread Jakub Jelinek
On Sat, Jan 16, 2016 at 09:58:51AM +0100, Jakub Jelinek wrote: > IMHO you'd be best to write your own helpers for conversion to little > endian (and back). > gcc configure already has AC_C_BIGENDIAN (dunno how it handles pdp endian > host though, so not sure if it is safe to rely on that), for rece

Re: [PATCH v2] libstdc++: Make certain exceptions transaction_safe.

2016-01-16 Thread Dominique d'Humières
> Addressed these, fixed a problem with using GLIBCXX_WEAK_DEFINITION > (which is only set on Darwin despite the generic-sounding name -- so > just use __attribute__((weak)) directly), and also updated > testsuite_abi.cc so that it knows about CXXABI_1.3.10. > > Approved by Jonathan Wakely. Commit

Re: [hsa merge 07/10] IPA-HSA pass

2016-01-16 Thread Jan Hubicka
> On 01/15/2016 10:52 AM, Jan Hubicka wrote: > > Do we really need to look that up in the hsa summary? Why these can not be > > partitioned the > > usual way? > > Hi. > > Yes, it's needed as hsa-brig.c uses host function declaration of a kernel as > a key for libgomp. > That's why we want to pu

Re: reject decl with incomplete struct/union type in check_global_declaration()

2016-01-16 Thread Prathamesh Kulkarni
On 16 January 2016 at 02:56, Joseph Myers wrote: > On Fri, 15 Jan 2016, Prathamesh Kulkarni wrote: > >> On 15 January 2016 at 03:27, Joseph Myers wrote: >> > On Thu, 14 Jan 2016, Prathamesh Kulkarni wrote: >> > >> >> Hi, >> >> For test-case containing only the following declaration: >> >> static

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-16 Thread Jan Hubicka
> On Fri, 15 Jan 2016, Jan Hubicka wrote: > > > Hi, > > this is last of my patches to improve partitionability of programs. > > This patch compacts BLOCK_VARs lists and trows away TYPE_DECLs for -g0 > > and redundant TYPE_DECLs on all levels. It does make noticeable difference > > on firefox, but

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-16 Thread Richard Biener
On January 16, 2016 11:27:09 AM GMT+01:00, Jan Hubicka wrote: >> On Fri, 15 Jan 2016, Jan Hubicka wrote: >> >> > Hi, >> > this is last of my patches to improve partitionability of programs. > >> > This patch compacts BLOCK_VARs lists and trows away TYPE_DECLs for >-g0 >> > and redundant TYPE_DECL

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-16 Thread Jakub Jelinek
On Sat, Jan 16, 2016 at 11:31:49AM +0100, Richard Biener wrote: > On January 16, 2016 11:27:09 AM GMT+01:00, Jan Hubicka wrote: > >> On Fri, 15 Jan 2016, Jan Hubicka wrote: > >> > >> > Hi, > >> > this is last of my patches to improve partitionability of programs. > > > >> > This patch compacts BL

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-16 Thread Jan Hubicka
> >Usually the things stay to make DECL_UID stable across -g and -g0. > > Only DECL_UID order needs to be stable I think. Well, when you remove a reference to a declaration, the DECL_UID order will change. We stream declarations in the order they are inserted to the encoder and that depends on

Re: [hsa merge 08/10] HSAIL BRIG description header file

2016-01-16 Thread Jakub Jelinek
On Fri, Jan 15, 2016 at 06:23:05PM +0100, Martin Jambor wrote: > BRIG_KIND_OPERAND_REGISTER = 0x300a, > BRIG_KIND_OPERAND_STRING = 0x300b, > BRIG_KIND_OPERAND_WAVESIZE = 0x3009c, > BRIG_KIND_OPERAND_END = 0x300d The above looks weird, I'd have expected BRIG_KIND_OPERAND_WAVESIZE to be 0x30

[PING][PATCH][DJGPP][libgfortran] Do not use S_ISVTX for DJGPP in libfortran/intrinsic/chmod.c

2016-01-16 Thread Andris Pavenis
Patch avoids using S_ISVTX if it is not defined. https://gcc.gnu.org/ml/gcc-patches/2015-11/msg01899.html Initial version of patch avoided it additionally for DJGPP (https://gcc.gnu.org/ml/gcc-patches/2015-11/msg01880.html) Andris

Re: [PATCH v2] libstdc++: Make certain exceptions transaction_safe.

2016-01-16 Thread David Edelsohn
This patch broke bootstrap on AIX. Not all targets support TM. This patch makes libstdc++ unconditionally refer to TM symbols. Please fix. - David

Re: [PATCH v2] libstdc++: Make certain exceptions transaction_safe.

2016-01-16 Thread Jonathan Wakely
What are the errors? I can build libstdc++ on gcc111. Does this patch help? diff --git a/libstdc++-v3/src/c++11/cow-stdexcept.cc b/libstdc++-v3/src/c++11/cow-stdexcept.cc index afc3f6c..8a1b65a 100644 --- a/libstdc++-v3/src/c++11/cow-stdexcept.cc +++ b/libstdc++-v3/src/c++11/cow-stdexcept.cc

Re: [PATCH v2] libstdc++: Make certain exceptions transaction_safe.

2016-01-16 Thread David Edelsohn
stage1 libstdc++ builds just fine. the problem is stage2 configure fails due to missing ITM_xxx symbols when configure tries to compile and run conftest programs. Thanks, David On Sat, Jan 16, 2016 at 7:43 AM, Jonathan Wakely wrote: > What are the errors? > > I can build libstdc++ on gcc111. >

Re: [PATCH v2] libstdc++: Make certain exceptions transaction_safe.

2016-01-16 Thread Jakub Jelinek
On Sat, Jan 16, 2016 at 07:47:33AM -0500, David Edelsohn wrote: > stage1 libstdc++ builds just fine. the problem is stage2 configure > fails due to missing ITM_xxx symbols when configure tries to compile > and run conftest programs. On x86_64-linux, the _ITM_xxx symbols are undef weak ones and th

Re: [PATCH v2] libstdc++: Make certain exceptions transaction_safe.

2016-01-16 Thread Jonathan Wakely
On 16/01/16 14:35 +0100, Jakub Jelinek wrote: On Sat, Jan 16, 2016 at 07:47:33AM -0500, David Edelsohn wrote: stage1 libstdc++ builds just fine. the problem is stage2 configure fails due to missing ITM_xxx symbols when configure tries to compile and run conftest programs. On x86_64-linux, the

[PATCH] s390: Add -fsplit-stack support

2016-01-16 Thread Marcin Kościelnicki
libgcc/ChangeLog: * config.host: Use t-stack and t-stack-s390 for s390*-*-linux. * config/s390/morestack.S: New file. * config/s390/t-stack-s390: New file. * generic-morestack.c (__splitstack_find): Add s390-specific code. gcc/ChangeLog: * common/config/s3

Re: [PATCH v2] libstdc++: Make certain exceptions transaction_safe.

2016-01-16 Thread Jonathan Wakely
On 16/01/16 13:41 +, Jonathan Wakely wrote: On 16/01/16 14:35 +0100, Jakub Jelinek wrote: On Sat, Jan 16, 2016 at 07:47:33AM -0500, David Edelsohn wrote: stage1 libstdc++ builds just fine. the problem is stage2 configure fails due to missing ITM_xxx symbols when configure tries to compile

Re: [RFC][PATCH, ARM 7/8] ARMv8-M Security Extension's cmse_nonsecure_call: use __gnu_cmse_nonsecure_call]

2016-01-16 Thread Senthil Kumar Selvaraj
User-agent: mu4e 0.9.13; emacs 24.5.1 Hi, Apologies for the bad posting style (I don't have the original email handy), but shouldn't _gnu_cmse_nonsecure_call be defined with the .global directive in the below hunk (to make it visible when linking)? diff --git a/libgcc/config/arm/cmse_nonsecure_c

Re: [PATCH v2] libstdc++: Make certain exceptions transaction_safe.

2016-01-16 Thread Torvald Riegel
On Sat, 2016-01-16 at 10:57 +0100, Dominique d'Humières wrote: > > Addressed these, fixed a problem with using GLIBCXX_WEAK_DEFINITION > > (which is only set on Darwin despite the generic-sounding name -- so > > just use __attribute__((weak)) directly), and also updated > > testsuite_abi.cc so that

Re: [PATCH v2] libstdc++: Make certain exceptions transaction_safe.

2016-01-16 Thread Torvald Riegel
On Sat, 2016-01-16 at 14:35 +0100, Jakub Jelinek wrote: > On Sat, Jan 16, 2016 at 07:47:33AM -0500, David Edelsohn wrote: > > stage1 libstdc++ builds just fine. the problem is stage2 configure > > fails due to missing ITM_xxx symbols when configure tries to compile > > and run conftest programs. >

Re: [PATCH v2] libstdc++: Make certain exceptions transaction_safe.

2016-01-16 Thread David Edelsohn
Torvald, The error is a link failure in stage2 configure due to the missing _ITM_xxx and related symbols. I don't have the failed build any more. Maybe Jonathan can reply with the specific failures. There is an AIX system in the GNU Compile Farm: gcc111. - David On Sat, Jan 16, 2016 at 3:12 PM

Re: [PATCH v2] libstdc++: Make certain exceptions transaction_safe.

2016-01-16 Thread David Edelsohn
On Sat, Jan 16, 2016 at 8:35 AM, Jakub Jelinek wrote: > On Sat, Jan 16, 2016 at 07:47:33AM -0500, David Edelsohn wrote: >> stage1 libstdc++ builds just fine. the problem is stage2 configure >> fails due to missing ITM_xxx symbols when configure tries to compile >> and run conftest programs. > > O

[PATCH][committed] libitm: Ensure proxy privatization safety.

2016-01-16 Thread Torvald Riegel
This patch fixes privatization safety support by additionally ensuring proxy privatization safety. It fixes a correctness bugs, but in a simple way for now, which will lead to less scalability with higher thread counts. I plan to finish an optimization once I have enough time for this. Tested on

[Committed, testsuite] Move gcc.dg/parloops-exit-first-loop-alt-*.c to gcc.dg/autopar

2016-01-16 Thread Tom de Vries
Hi, I've moved the gcc.dg/parloops-exit-first-loop-alt-*.c tests to gcc.dg/autopar. Committed to trunk. Thanks, - Tom Move gcc.dg/parloops-exit-first-loop-alt-*.c to gcc.dg/autopar 2016-01-16 Tom de Vries * gcc.dg/parloops-exit-first-loop-alt.c: Move ... * gcc.dg/autopar/parloops-exit-

[testsuite, committed] Add goacc/kernels-alias-ipa-pta-4.c

2016-01-16 Thread Tom de Vries
Hi, this is an additional test, similar to goacc/kernels-alias-ipa-pta.c but with scalars instead of arrays. Committed to trunk. Thanks, - Tom Add goacc/kernels-alias-ipa-pta-4.c 2016-01-16 Tom de Vries * c-c++-common/goacc/kernels-alias-ipa-pta-4.c: New test. --- .../c-c++-common/goa

[committed] Release_defs in expand_omp_atomic_fetch_op

2016-01-16 Thread Tom de Vries
Hi, this patch adds a missing 'release_defs (stmt)' in expand_omp_atomic_fetch_op. This is a merge of this ( https://gcc.gnu.org/ml/gcc-patches/2015-11/msg00447.html ) patch from the gomp-4_0-branch. Bootstrapped and reg-tested on x86_64. Committed to trunk. Thanks, - Tom Release_defs in

[Committed] Move pass_expand_omp_ssa out of pass_parallelize_loops

2016-01-16 Thread Tom de Vries
[ was: Re: [PIING][PATCH, 9/16] Add pass_parallelize_loops_oacc_kernels ] On 14/12/15 16:22, Richard Biener wrote: Can the pass not just use a pass parameter to switch between oacc/non-oacc? It can, but given PR68874 ('Allow pass groups to be cloned'), if we clone pass_parallelize_loops we ca

[PATCH] libstdc++: Fix static_assert.

2016-01-16 Thread Torvald Riegel
This adds a missing string argument to a call to static_assert, thus not making it depend on c++1z extensions. This fixes the build breakage on mingw introduced in 232454. Tested on x86_64-linux. OK? commit 7659ab483954a15c8143f6b1b9d135159a2ecc67 Author: Torvald Riegel Date: Sat Jan 16 23:40

Re: version typeinfo for 128bit types

2016-01-16 Thread H.J. Lu
On Mon, Apr 14, 2014 at 2:39 AM, Marc Glisse wrote: > Hello, > > this is a follow-up for this patch: > http://gcc.gnu.org/ml/gcc-patches/2014-04/msg00618.html > > once committed, g++ will generate typeinfo for __float128, and it needs > versioning. While there, I noticed that __int128 has "typeinf

Re: [PATCH v2] libstdc++: Make certain exceptions transaction_safe.

2016-01-16 Thread H.J. Lu
On Thu, Jan 7, 2016 at 8:47 AM, Torvald Riegel wrote: > The attached patch makes some exceptions transaction-safe, as require by > the Transactional Memory TS. I believe I addressed all feedback for the > previous version of this patch (in particular, there are now more safety > checks for precon

[PATCH] fix #69317 - [6 regression] wrong ABI version in -Wabi warnings

2016-01-16 Thread Martin Sebor
While adding an ABI warning in the patch for bug 69277 I noticed that the ABI version printed by GCC 6.0 in some -Wabi diagnostics is incorrect: while 5.1.0 prints the versions of the ABI given by the -Wabi=X and -fabi-version=Y options (i.e., it mentions both X and Y), 6.0 prints the same versio

[doc, 6/n] invoke.texi: split debugging options into programmer vs developer sections

2016-01-16 Thread Sandra Loosemore
This is the last planned part of my series to try to make it easier for users to find information about command-line options (although as I've been going through this chapter I've been finding places that need copy-editing or technical corrections, which I'll handle separately). In this part, I

Re: [PATCH] [ARC] Add basic support for double load and store instructions

2016-01-16 Thread Joern Wolfgang Rennecke
On 15/01/16 12:40, Claudiu Zissulescu wrote: (arc_save_restore): Use double load/store instruction. (arc_expand_movmem): Likewise. >if (n_pieces >= (unsigned int) (optimize_size ? 3 : 15)) > return false; > - if (piece > 4) > + if (TARGET_LL64 && (piece != 8) && (