Re: libgomp nvptx plugin: Debugging output for cuInit failure

2017-05-24 Thread Thomas Schwinge
Hi! On Tue, 23 May 2017 12:41:05 +0200, Jakub Jelinek wrote: > On Mon, May 22, 2017 at 02:57:24PM +0200, Thomas Schwinge wrote: > > libgomp nvptx plugin: Debugging output for cuInit failure > Ok. Thanks. As posted, just with better ChangeLog committed to trunk in r248400: commit d5f081f8c

Re: [PATCH] Fix PR middle-end/80823, ICE: verify_flow_info failed

2017-05-24 Thread Richard Biener
On May 23, 2017 7:46:59 PM GMT+02:00, Peter Bergner wrote: >The fix for PR80775 included a thinko bug that caused us to skip some >case label statements. This leads to problems for test cases where we >have multiple case labels that point to the same unreachable block, but >are not mergeable sin

Re: [patch] build xz (instead of bz2) compressed tarballs and diffs

2017-05-24 Thread Richard Biener
On May 24, 2017 1:22:42 AM GMT+02:00, Matthias Klose wrote: >On 18.05.2017 03:34, Richard Biener wrote: >> On Mon, May 15, 2017 at 3:11 AM, Matthias Klose >wrote: >>> As discussed on IRC with Jakub and Richard here are is a small patch >which >>> builds xz compressed tarballs and diff files. >>>

Re: [PATCH 2/5 v3] Vect peeling cost model

2017-05-24 Thread Robin Dapp
> Not sure I've understood the series TBH, but is the npeel == vf / 2 > there specifically for the "unknown number of peels" case? How do > we distinguish that from the case in which the number of peels is > known to be vf / 2 at compile time? Or have I missed the point > completely? (probably ye

[patch] Fix libstdc++-v3 build for Android

2017-05-24 Thread Eric Botcazou
Hi, libstdc++-v3 already contains support but doesn't build for Android. Now GDB has switched to C++, which means that you need a C++ cross-build for GDBserver in order to debug a program on the target. Tested on x86_64-suse-linux and arm-linux-androideab, OK for mainline? 2017-05-24 Eric B

Re: [PATCH 0/5 v3] Vect peeling cost model

2017-05-24 Thread Richard Biener
On Tue, May 23, 2017 at 5:57 PM, Robin Dapp wrote: > The last version of the patch series caused some regressions for ppc64. > This was largely due to incorrect handling of unsupportable alignment > and should be fixed with the new version. > > p2 and p5 have not changed but I'm posting the whole

[rs6000] Fix ICE with -fstack-limit-register and large frames

2017-05-24 Thread Eric Botcazou
Hi, this fixes an internal error with -fstack-limit-register and large frames: eric@polaris:~/build/gcc/powerpc-linux> gcc/xgcc -Bgcc -S stack-limit-1.c - fstack-limit-register=r2 stack-limit-1.c: In function 'foo': stack-limit-1.c:9:1: error: insn does not satisfy its constraints: } ^ (insn 21

Re: [PATCH 2/5 v3] Vect peeling cost model

2017-05-24 Thread Richard Sandiford
Robin Dapp writes: >> Not sure I've understood the series TBH, but is the npeel == vf / 2 >> there specifically for the "unknown number of peels" case? How do >> we distinguish that from the case in which the number of peels is >> known to be vf / 2 at compile time? Or have I missed the point >>

Re: [PATCH][www] Deprecate MPX for GCC 7

2017-05-24 Thread Richard Biener
On Tue, 23 May 2017, Jeff Law wrote: > On 05/23/2017 06:50 AM, Richard Biener wrote: > > > > Appearantly we forgot to officially deprecate MPX in GCC 7 changes.html. > > > > The following adds the missing note. > > > > Ok? > > > > Thanks, > > Richard. > > > > 2017-05-23 Richard Biener > >

Re: [PATCH] Prevent extract_muldiv from introducing an overflow (PR sanitizer/80800)

2017-05-24 Thread Richard Biener
On Fri, 19 May 2017, Alexander Monakov wrote: > On Fri, 19 May 2017, Joseph Myers wrote: > > On Fri, 19 May 2017, Alexander Monakov wrote: > > > (to be precise, if the multiplication is done in a signed type and the > > > middle > > > constant factor was a negated power of two, the sign change ne

Re: SSA range class and removal of VR_ANTI_RANGEs

2017-05-24 Thread Richard Biener
On Tue, May 23, 2017 at 12:48 PM, Aldy Hernandez wrote: > Howdy! > > For Andrew's upcoming on-demand range work I have created a range class for > use in his engine. Currently, the range class is only for SSA integers, but > there is no reason why we couldn't adapt this for RTL or non-integer typ

Fix var-tracking for special record parameters passed indirectly

2017-05-24 Thread Eric Botcazou
var-tracking doesn't track structures and arrays as a whole because it cannot track values for multiple-part variables, but it has a special provision for Ada fat pointer types, which are structures with integral mode containing a pair of pointers and are pervasive in the language (see param-[12

Re: SSA range class and removal of VR_ANTI_RANGEs

2017-05-24 Thread Richard Biener
..On Tue, May 23, 2017 at 5:24 PM, Andrew MacLeod wrot > On 05/23/2017 10:54 AM, Jakub Jelinek wrote: >> >> On Tue, May 23, 2017 at 10:38:44AM -0400, Andrew MacLeod wrote: >>> >>> As follow on work we have discussed an interface which would be able to >>> calculate a bitmask (for either zeros or o

Fix pessimistic DImode handling in combine.c:make_field_assignment

2017-05-24 Thread Richard Sandiford
The make_field_assignment code: src = force_to_mode (src, mode, GET_MODE_PRECISION (mode) >= HOST_BITS_PER_WIDE_INT ? HOST_WIDE_INT_M1U : (HOST_WIDE_INT_1U << len) - 1, 0); would ignore the field len

Re: Allow some NOP conversions in (X+CST1)+CST2 in match.pd

2017-05-24 Thread Richard Biener
On Sun, May 21, 2017 at 9:22 PM, Marc Glisse wrote: > Hello, > > generalizing a bit one transformation, to avoid a regression with another > patch I am working on. Handling conversions always gets messy :-( It would > have been easier to stick to scalars and wide_int, but since the existing > tran

Re: Ipa function summary pass

2017-05-24 Thread Christophe Lyon
Hi, On 23 May 2017 at 18:23, Jan Hubicka wrote: > Hi, > this patch finishes the breakup of ipa-inline and function analysis. > The analysis is now done by separate pass and I will work on cleaning > up the interfaces now. > > Honza > > * cgraphunit.c (symbol_table::process_new_functions):

Re: signed multiplication for pointer offsets

2017-05-24 Thread Richard Biener
On Sun, May 21, 2017 at 9:45 PM, Marc Glisse wrote: > Hello, > > when we have a double*p, computing p+n, we multiply n by 8 (size of double) > then add it to p. According to the comment just below the modified lines in > the attached patch, the multiplication is supposed to happen in a signed > ty

Re: Relax VIEW_CONVERT_EXPR - CONVERT_EXPR combination

2017-05-24 Thread Richard Biener
On Sun, May 21, 2017 at 11:21 PM, Marc Glisse wrote: > Hello, > > SRA uses char when scalarizing bool, and we end up with > > _6 = u_1(D) == 0.0; > _7 = (unsigned char) _6; > _3 = VIEW_CONVERT_EXPR<_Bool>(_7); > > which we currently do not simplify. I am not completely sure what happens > wi

Re: [PATCH GCC][3/6]Fix PR80815 by handling negative DR_STEPs in runtime alias check

2017-05-24 Thread Bin.Cheng
On Tue, May 23, 2017 at 6:53 PM, Richard Sandiford wrote: > AIUI, the reason the old code mishandled negative steps was that the > associated segment lengths were stored as sizetype and so looked like > big unsigned values. Those values therefore satisfied tree_fits_uhwi_p > even though they were

Re: [PATCH GCC][3/6]Fix PR80815 by handling negative DR_STEPs in runtime alias check

2017-05-24 Thread Richard Sandiford
"Bin.Cheng" writes: > On Tue, May 23, 2017 at 6:53 PM, Richard Sandiford > wrote: >> AIUI, the reason the old code mishandled negative steps was that the >> associated segment lengths were stored as sizetype and so looked like >> big unsigned values. Those values therefore satisfied tree_fits_uh

Re: C/C++ OpenACC: acc_pcopyin, acc_pcreate

2017-05-24 Thread Thomas Schwinge
Hi Jakub! On Tue, 23 May 2017 13:07:15 +0200, Jakub Jelinek wrote: > On Mon, May 22, 2017 at 04:26:48PM +0200, Thomas Schwinge wrote: > > In , we currently describe acc_pcopyin, acc_pcreate as "old > > names", but they're not "old" but really "alternative names", with the > > intention to provide

[PATCH] [ARC] Fix tst_movb pattern.

2017-05-24 Thread Claudiu Zissulescu
From: claziss The tst_movb pattern is missing guarding when spitting. gcc/ 2017-05-24 Claudiu Zissulescu * config/arc/arc.md (tst_movb): Add guard when splitting. testsuite/ 2017-05-24 Claudiu Zissulescu * gcc.target/arc/pr9001195952.c: New test. --- gcc/config/arc/arc.

Re: [PATCH] gcc::context creation

2017-05-24 Thread Jakub Jelinek
On Thu, May 18, 2017 at 11:08:51AM -0400, Nathan Sidwell wrote: > On 05/16/2017 11:26 AM, Nathan Sidwell wrote: > > This patch breaks apart the creation of the pass manager from > > gcc::context's ctor. This will allow us to register additional dumps > ... https://gcc.gnu.org/ml/gcc-patches/2017-0

[PATCH v2 05/N] Add -fdump*-folding suboption.

2017-05-24 Thread Martin Liška
Hello. After discussions with Richi, I decided to end up with the transformation and thus I would like to apply the original patch on top of what was done. Probably last change should be that we should renumber TDF_* values to start from 1 << 0 and holes in numbering should be eliminated? Tha

Re: [PATCH] Try harder to fix recently introduced crashes in ggc_collect

2017-05-24 Thread Trevor Saunders
On Fri, May 19, 2017 at 10:10:23AM -0400, Nathan Sidwell wrote: > On 05/19/2017 10:05 AM, Bernd Edlinger wrote: > > > hwi cannot be the root cause of the problem, > > because it can only be long_integer_type_node > > or long_long_integer_type_node, otherwise > > an error would be triggered. > > t

Re: C/C++ OpenACC: acc_pcopyin, acc_pcreate

2017-05-24 Thread Jakub Jelinek
On Wed, May 24, 2017 at 12:55:27PM +0200, Thomas Schwinge wrote: > Hi Jakub! > > On Tue, 23 May 2017 13:07:15 +0200, Jakub Jelinek wrote: > > On Mon, May 22, 2017 at 04:26:48PM +0200, Thomas Schwinge wrote: > > > In , we currently describe acc_pcopyin, acc_pcreate as "old > > > names", but they'r

Re: [PATCH] gcc::context creation

2017-05-24 Thread Jakub Jelinek
On Wed, May 24, 2017 at 12:57:28PM +0200, Jakub Jelinek wrote: > So, first question, where should we order the -fdump-lang-* dumps, shall > they go before .original/.gimple/.nested, or in between .original and > .gimple/.nested, or after those 3 (the last option looks just wrong to me)? > Shall the

[C++ PATCH] Hidden functions

2017-05-24 Thread Nathan Sidwell
We have functions that are invisible to (most) name lookups: 1) anticipated builtins -- these only become visible once declared, but we need to know they match a builtin. 2) friends injected into namespace scope from a class definition. This patch teaches the overload object about them with a n

Re: Implementing OpenACC's Fortran module

2017-05-24 Thread Thomas Schwinge
Hi Jakub! On Tue, 23 May 2017 13:12:00 +0200, Jakub Jelinek wrote: > On Mon, May 22, 2017 at 05:23:25PM +0200, Thomas Schwinge wrote: > > --- libgomp/openacc_lib.h > > +++ libgomp/openacc_lib.h > > @@ -191,23 +191,9 @@ > >end interface > > > >interface acc_pcopyin > > -s

[Committed] S/390: Fix PR80725.

2017-05-24 Thread Andreas Krebbel
No regressions on s390x. Applied to mainline. Ok for GCC 7 branch as well? Bye, -Andreas- gcc/ChangeLog: 2017-05-24 Andreas Krebbel PR target/80725 * config/s390/s390.c (s390_check_qrst_address): Check incoming address against address_operand predicate. * con

Re: [Committed] S/390: Fix PR80725.

2017-05-24 Thread Jakub Jelinek
On Wed, May 24, 2017 at 01:38:01PM +0200, Andreas Krebbel wrote: > No regressions on s390x. Applied to mainline. > Ok for GCC 7 branch as well? Ok, (though, you don't really need a permission for that, maintainers can decide on backports to release branches themselves except when the branches are

Re: [PATCH] gcc::context creation

2017-05-24 Thread Nathan Sidwell
On 05/24/2017 06:57 AM, Jakub Jelinek wrote: So, first question, where should we order the -fdump-lang-* dumps, shall they go before .original/.gimple/.nested, or in between .original and .gimple/.nested, or after those 3 (the last option looks just wrong to me)? I think the lang dumps should

Re: OpenACC 1.0 compatibility: acc_async_wait, acc_async_wait_all

2017-05-24 Thread Thomas Schwinge
Hi Jakub! On Tue, 23 May 2017 13:17:26 +0200, Jakub Jelinek wrote: > On Mon, May 22, 2017 at 07:42:38PM +0200, Thomas Schwinge wrote: > > For OpenACC 1.0 compatibility, we need to provide the aliases > > acc_async_wait, and acc_async_wait_all for acc_wait, and acc_wait_all, > > respectively. > >

Re: [PATCH 0/5 v3] Vect peeling cost model

2017-05-24 Thread Robin Dapp
> So what did actually change? I'd rather not diff the diffs. Can you > provide an incremental change, aka p6 that would apply to the > previous series instead? -p6.diff attached which also addresses Richard's remark regarding vf/2. Note that this applies to the old series but the old series its

Re: [PATCH] gcc::context creation

2017-05-24 Thread Jakub Jelinek
On Wed, May 24, 2017 at 07:44:49AM -0400, Nathan Sidwell wrote: > On 05/24/2017 06:57 AM, Jakub Jelinek wrote: > > > So, first question, where should we order the -fdump-lang-* dumps, shall > > they go before .original/.gimple/.nested, or in between .original and > > .gimple/.nested, or after thos

Re: OpenACC 1.0 compatibility: acc_async_wait, acc_async_wait_all

2017-05-24 Thread Jakub Jelinek
On Wed, May 24, 2017 at 01:47:34PM +0200, Thomas Schwinge wrote: > Hi Jakub! > > On Tue, 23 May 2017 13:17:26 +0200, Jakub Jelinek wrote: > > On Mon, May 22, 2017 at 07:42:38PM +0200, Thomas Schwinge wrote: > > > For OpenACC 1.0 compatibility, we need to provide the aliases > > > acc_async_wait,

Re: Implementing OpenACC's Fortran module

2017-05-24 Thread Jakub Jelinek
On Wed, May 24, 2017 at 01:34:50PM +0200, Thomas Schwinge wrote: > Thus OK for trunk, as posted? Ok. Jakub

Re: [PATCH] gcc::context creation

2017-05-24 Thread Nathan Sidwell
On 05/24/2017 07:57 AM, Jakub Jelinek wrote: The numbering is not to differentiate between dumps of the same name, that is done through numbered suffixes of the dump names, but to make the order of passes visible to the user and for sorted file listing to be provided in that order. So, rather t

Re: [PATCH] Fix PR middle-end/80823, ICE: verify_flow_info failed

2017-05-24 Thread Peter Bergner
On 5/24/17 2:15 AM, Richard Biener wrote: > On May 23, 2017 7:46:59 PM GMT+02:00, Peter Bergner > wrote: >> gcc/ >> PR middle-end/80823 >> * tree-cfg.c (group_case_labels_stmt): Delete increment of "i"; >> >> gcc/testsuite/ >> PR middle-end/80823 >> * gcc.dg/pr80823.c: New tes

Re: [PATCH] Fix a SPARC -mcbcond compare-and-branch out of range failure.

2017-05-24 Thread Sheldon Lobo
Thanks for the review. On 05/24/2017 02:52 AM, Eric Botcazou wrote: OK for all active branches, but the ChangeLog entry should be something like: * config/sparc/sparc.md (length): Return the correct value for -mflat sibcalls to match output_sibcall. Will do. Note that we *

Re: [PATCH] Match x86 family machine constraints section with constarints.md

2017-05-24 Thread Uros Bizjak
On Tue, May 23, 2017 at 5:33 PM, Sandra Loosemore wrote: > On 04/28/2017 03:30 AM, Peryt, Sebastian wrote: >> >> Hi, >> >> Thank you for your comments. I edited my patch accordingly. As for some of >> your doubts: >> - REX is the opcode prefix to access 64-bit register extensions >> introduced in

Re: C/C++ OpenACC: acc_pcopyin, acc_pcreate

2017-05-24 Thread Thomas Schwinge
Hi! On Wed, 24 May 2017 13:20:54 +0200, Jakub Jelinek wrote: > On Wed, May 24, 2017 at 12:55:27PM +0200, Thomas Schwinge wrote: > > On Tue, 23 May 2017 13:07:15 +0200, Jakub Jelinek wrote: > > > new symbols should never be added to an existing symbol > > > version after a GCC version with that s

Re: [config PATCH] --enable-languages

2017-05-24 Thread Richard Biener
On Fri, May 19, 2017 at 6:59 PM, Nathan Sidwell wrote: > --enable-languages=all confused me, because as RichardB said, 'it really > means default'. So this patch does 2 things > 1) allow --enable-languages=default to mean what =all does now. > 2) change =all to mean all available languages. > 2.1

Re: Translate libgomp.oacc-c-c++-common/lib-32.c into Fortran

2017-05-24 Thread Thomas Schwinge
Hi! On Tue, 23 May 2017 13:08:22 +0200, Jakub Jelinek wrote: > On Mon, May 22, 2017 at 04:29:51PM +0200, Thomas Schwinge wrote: > > On Mon, 22 May 2017 16:26:48 +0200, I wrote: > > > C/C++ OpenACC: acc_pcopyin, acc_pcreate > > > > > libgomp/ > > > [...] > > >

Re: [PATCH] Dump function on internal errors

2017-05-24 Thread Richard Biener
On Mon, May 22, 2017 at 4:19 PM, Andi Kleen wrote: > From: Andi Kleen > > When a verification check fails it is useful to dump the current > function to the dump file, so it's easier to figure out what > actually went wrong. > > v2: Updated version now using a hook in internal_error, and > also p

Re: Implementing OpenACC's Fortran module

2017-05-24 Thread Thomas Schwinge
Hi! On Wed, 24 May 2017 13:58:32 +0200, Jakub Jelinek wrote: > On Wed, May 24, 2017 at 01:34:50PM +0200, Thomas Schwinge wrote: > > Thus OK for trunk, as posted? > > Ok. Thanks. Committed to trunk in r248412: commit b70df2112de1c8fb9b448dc24e3a58055e4bbbdc Author: tschwinge Date: Wed May 2

Re: OpenACC 1.0 compatibility: acc_async_wait, acc_async_wait_all

2017-05-24 Thread Thomas Schwinge
Hi! On Wed, 24 May 2017 13:58:12 +0200, Jakub Jelinek wrote: > On Wed, May 24, 2017 at 01:47:34PM +0200, Thomas Schwinge wrote: > > On Tue, 23 May 2017 13:17:26 +0200, Jakub Jelinek wrote: > > > On Mon, May 22, 2017 at 07:42:38PM +0200, Thomas Schwinge wrote: > > > > For OpenACC 1.0 compatibilit

Re: [PATCH GCC8][31/33]Set range information for niter bound of vectorized loop

2017-05-24 Thread Richard Biener
On Mon, May 22, 2017 at 7:13 PM, Bin.Cheng wrote: > On Fri, May 19, 2017 at 1:51 PM, Richard Biener > wrote: >> On Mon, May 15, 2017 at 5:58 PM, Bin.Cheng wrote: >>> On Thu, May 11, 2017 at 12:02 PM, Richard Biener >>> wrote: On Tue, Apr 18, 2017 at 12:54 PM, Bin Cheng wrote > Hi, >>>

Re: [PATCH v2 05/N] Add -fdump*-folding suboption.

2017-05-24 Thread Richard Biener
On Wed, May 24, 2017 at 12:54 PM, Martin Liška wrote: > Hello. > > After discussions with Richi, I decided to end up with the transformation > and thus > I would like to apply the original patch on top of what was done. Probably > last change > should be that we should renumber TDF_* values to sta

Re: [PATCH 0/5 v3] Vect peeling cost model

2017-05-24 Thread Richard Biener
On Wed, May 24, 2017 at 1:54 PM, Robin Dapp wrote: >> So what did actually change? I'd rather not diff the diffs. Can you >> provide an incremental change, aka p6 that would apply to the >> previous series instead? > > -p6.diff attached which also addresses Richard's remark regarding vf/2. > Not

Re: [PATCH, ARM/AArch64] drop aarch32 support for falkor/qdf24xx

2017-05-24 Thread Richard Earnshaw (lists)
On 12/05/17 03:43, Jim Wilson wrote: > Early steppings had aarch32 support, current steppings don't, so the > aarch32 support for falkor/qdf24xx needs to be dropped. This mostly > involves removing falkor/qdf24xx references from the arm port. The > qdf24xx_extra_costs structure moves from the arm

[PATCH, GCC/ARM] Fix typo in comment in arm_expand_prologue

2017-05-24 Thread Thomas Preudhomme
Committed this obvious typo fix for a comment in arm_expand_prologue. ChangeLog entry is as follows: *** gcc/ChangeLog *** 2017-05-24 Thomas Preud'homme * config/arm/arm.c (arm_expand_prologue): Fix typo in comment. Best regards, Thomas diff --git a/gcc/config/arm/arm.c b/gcc/conf

[testsuite, committed] Reinstate absolute line number in gcc.dg/cpp/19940712-1.c

2017-05-24 Thread Tom de Vries
Hi, the line number refers to a line in a header file, so we want to keep that absolute. [ The absolute line numbers that we want to replace are the ones that start failing once we add a line at the start of the file. If an absolute line number refers to a header file, and we add a line at

Re: C++ PATCH to add __integer_pack built-in for std::make_integer_sequence (c++/80396)

2017-05-24 Thread Andreas Schwab
FAIL: g++.dg/ext/integer-pack2.C -std=gnu++11 (test for excess errors) Excess errors: /daten/aranym/gcc/gcc-20170524/gcc/testsuite/g++.dg/ext/integer-pack2.C:10:48: error: overflow in constant expression [-fpermissive] /daten/aranym/gcc/gcc-20170524/gcc/testsuite/g++.dg/ext/integer-pack2.C:10:48

Re: [PATCH, ARM/AArch64] drop aarch32 support for falkor/qdf24xx

2017-05-24 Thread Jim Wilson
On Wed, May 24, 2017 at 6:56 AM, Richard Earnshaw (lists) wrote: > OK. does this need to go in the gcc-8 changes file? Falkor hasn't shipped yet. I'm dropping features that only existed in preproduction NDA hardware, so there isn't anything end user visible, and hence I don't think that it need

Re: [PATCH] PR c++/80544 strip cv-quals from cast results

2017-05-24 Thread Jonathan Wakely
On 23/05/17 16:26 -0400, Jason Merrill wrote: On Tue, May 23, 2017 at 2:00 PM, Jonathan Wakely wrote: On 19/05/17 15:14 -0400, Jason Merrill wrote: On Thu, Apr 27, 2017 at 12:59 PM, Jonathan Wakely wrote: I also tried to add a warning like EDG's (see the PR) but it gave a false positive fo

Re: [0/67] Add wrapper classes for machine_modes

2017-05-24 Thread Richard Sandiford
Jeff Law writes: > On 12/09/2016 05:48 AM, Richard Sandiford wrote: >> This series includes most of the changes in group C from: >> >> https://gcc.gnu.org/ml/gcc/2016-11/msg00033.html >> >> The idea is to add wrapper classes around machine_mode_enum >> for specific groups of modes, such as

Update libstdc++ baseline symbols (ia64, m68k)

2017-05-24 Thread Andreas Schwab
Committed. Andreas. * config/abi/post/ia64-linux-gnu/baseline_symbols.txt: Update. * config/abi/post/m68k-linux-gnu/baseline_symbols.txt: Likewise. diff --git a/libstdc++-v3/config/abi/post/ia64-linux-gnu/baseline_symbols.txt b/libstdc++-v3/config/abi/post/ia64-linux-gnu/baselin

Re: [PATCH] rs6000: Fix for separate shrink-wrapping for fp (PR80860, PR80843)

2017-05-24 Thread Segher Boessenkool
On Tue, May 23, 2017 at 11:07:07AM +, Segher Boessenkool wrote: > After my r248256, rs6000_components_for_bb allocates an sbitmap of size > only 32 while it can use up to 64. This patch fixes it. It moves the > n_components variable into the machine_function struct so that other > hooks can u

Re: Reorganise machmode.h headers

2017-05-24 Thread Richard Sandiford
Jeff Law writes: > On 11/16/2016 09:32 AM, Richard Sandiford wrote: >> Later patches will make machmode.h rely on wide-int.h and the >> new poly-int.h, so it needs to appear later in the coretypes.h >> include list. >> >> Previously machmode.h included insn-modes.h, which as well as >> the main mo

Re: C++ PATCH to add __integer_pack built-in for std::make_integer_sequence (c++/80396)

2017-05-24 Thread Jakub Jelinek
On Wed, May 24, 2017 at 04:16:30PM +0200, Andreas Schwab wrote: > FAIL: g++.dg/ext/integer-pack2.C -std=gnu++11 (test for excess errors) > Excess errors: > /daten/aranym/gcc/gcc-20170524/gcc/testsuite/g++.dg/ext/integer-pack2.C:10:48: > error: overflow in constant expression [

Re: [PATCH, ARM/AArch64] drop aarch32 support for falkor/qdf24xx

2017-05-24 Thread Richard Earnshaw (lists)
On 24/05/17 15:18, Jim Wilson wrote: > On Wed, May 24, 2017 at 6:56 AM, Richard Earnshaw (lists) > wrote: >> OK. does this need to go in the gcc-8 changes file? > > Falkor hasn't shipped yet. I'm dropping features that only existed in > preproduction NDA hardware, so there isn't anything end us

Re: [PATCH] Fix PR80754

2017-05-24 Thread Vladimir Makarov
On 05/17/2017 07:32 AM, Wilco Dijkstra wrote: When lra-remat rematerializes an instruction with a clobber, it checks that the clobber does not kill live registers. However it fails to check that the clobber also doesn't overlap with the destination register of the final rematerialized instruct

Re: SSA range class and removal of VR_ANTI_RANGEs

2017-05-24 Thread Andrew MacLeod
stlll bounced.. html must have snuck in somewhere, and the mailer sent it anyway -P trying again... On 05/24/2017 11:38 AM, Andrew MacLeod wrote: On 05/24/2017 04:25 AM, Richard Biener wrote: >> What's this overflow flag for anyway? The new on-demand range calculators do operations on ranges,

[C++ PATCH] Hidden name removal

2017-05-24 Thread Nathan Sidwell
Now that overloads are kept ordered, I can replace the existing remove_hidden_names fn with ovl_skip_hidden. The former works by scanning the overloads, then if it discovers a hidden one, copying the list (from the beginning) omitting the hidden ones. Now, overloads aren't usually huge, but t

Re: [PATCH, ARM/AArch64] drop aarch32 support for falkor/qdf24xx

2017-05-24 Thread Jim Wilson
On Wed, May 24, 2017 at 8:17 AM, Richard Earnshaw (lists) wrote: > On 24/05/17 15:18, Jim Wilson wrote: >> On Wed, May 24, 2017 at 6:56 AM, Richard Earnshaw (lists) >> wrote: >>> OK. does this need to go in the gcc-8 changes file? >> >> Falkor hasn't shipped yet. I'm dropping features that only

Re: SSA range class and removal of VR_ANTI_RANGEs

2017-05-24 Thread Richard Biener
On May 24, 2017 5:38:53 PM GMT+02:00, Andrew MacLeod wrote: >On 05/24/2017 04:25 AM, Richard Biener wrote: >> > What's this overflow flag for anyway? > >The new on-demand range calculators do operations on ranges, and that >flag is set when a range calculation may have overflowed... which is of

Re: C++ PATCH to -Wunused with C++17 structured bindings

2017-05-24 Thread Jason Merrill
On Wed, May 24, 2017 at 2:48 AM, Jakub Jelinek wrote: > On Tue, May 23, 2017 at 09:45:10PM -0400, Jason Merrill wrote: >> Someone on IRC complained that there was no way to suppress -Wunused >> on structured bindings. It seemed to me that the way the feature >> works, we shouldn't warn about the

Re: [PATCH, ARM/AArch64] drop aarch32 support for falkor/qdf24xx

2017-05-24 Thread Richard Earnshaw (lists)
On 24/05/17 17:03, Jim Wilson wrote: > On Wed, May 24, 2017 at 8:17 AM, Richard Earnshaw (lists) > wrote: >> On 24/05/17 15:18, Jim Wilson wrote: >>> On Wed, May 24, 2017 at 6:56 AM, Richard Earnshaw (lists) >>> wrote: OK. does this need to go in the gcc-8 changes file? >>> >>> Falkor hasn'

Re: [config PATCH] --enable-languages

2017-05-24 Thread Nathan Sidwell
On 05/24/2017 09:41 AM, Richard Biener wrote: Ok. Did you check =all works with/without a host Ada compiler installed? (warning but not including ada in the without case?) I checked on a system lacking ada, but I do notice there is no warning, which seems undesirable. This patch tweaks thi

Re: [patch, fortran] Handle MATMUL(TRANSPOSE(A),B) in inline matmul

2017-05-24 Thread Jerry DeLisle
On 05/17/2017 02:41 PM, Thomas Koenig wrote: > Hello world, > > after receiving no negative feedback on my RFC patch, I have deciced > to submit the patch. > > The attached patch handles MATMUL(TRANSPOSE(A),B) in inlining matmul. > Speed is a bit faster than the library version. > > Regression-t

Re: [PATCH] add more detail to -Wconversion and -Woverflow (PR 80731)

2017-05-24 Thread Joseph Myers
On Wed, 17 May 2017, Martin Sebor wrote: > @@ -1036,31 +1079,76 @@ warnings_for_convert_and_check (location_t loc, tree > type, tree expr, > /* This detects cases like converting -129 or 256 to >unsigned char. */ > if (!int_fits_type_p (expr, c_common_signed_type (typ

Re: MinGW compilation warnings in libiberty's waitpid.c

2017-05-24 Thread Eli Zaretskii
> From: DJ Delorie > Cc: gcc-patches@gcc.gnu.org, gdb-patc...@sourceware.org > Date: Tue, 23 May 2017 22:26:22 -0400 > > > Is there anything else I need to do about this part to get it solved > > in the upstream repository? > > A ChangeLog entry would be nice, so I don't have to write one ;-) S

Re: C++ PATCH to add __integer_pack built-in for std::make_integer_sequence (c++/80396)

2017-05-24 Thread Jason Merrill
On Wed, May 24, 2017 at 11:08 AM, Jakub Jelinek wrote: > On Wed, May 24, 2017 at 04:16:30PM +0200, Andreas Schwab wrote: >> FAIL: g++.dg/ext/integer-pack2.C -std=gnu++11 (test for excess errors) >> Excess errors: >> /daten/aranym/gcc/gcc-20170524/gcc/testsuite/g++.dg/ext/

Re: [PATCH] PR c++/80544 strip cv-quals from cast results

2017-05-24 Thread Jason Merrill
On Wed, May 24, 2017 at 10:20 AM, Jonathan Wakely wrote: > On 23/05/17 16:26 -0400, Jason Merrill wrote: >> >> On Tue, May 23, 2017 at 2:00 PM, Jonathan Wakely >> wrote: >>> >>> On 19/05/17 15:14 -0400, Jason Merrill wrote: On Thu, Apr 27, 2017 at 12:59 PM, Jonathan Wakely wr

Re: [patch, fortran] Handle MATMUL(TRANSPOSE(A),B) in inline matmul

2017-05-24 Thread Thomas Koenig
Hi Jerry, On 05/17/2017 02:41 PM, Thomas Koenig wrote: Hello world, after receiving no negative feedback on my RFC patch, I have deciced to submit the patch. The attached patch handles MATMUL(TRANSPOSE(A),B) in inlining matmul. Speed is a bit faster than the library version. Regression-tested

[patch, fortran, committed] replace stdout with dumpfile in a few places

2017-05-24 Thread Thomas Koenig
Hello world, I just committed the attached patch as obvious. Regards Thomas 2017-05-24 Thomas Koenig * dump-parse-tree.c (show_expr): Replace stdout with dumpfile for showing values. Index: dump-parse-tree.c

Re: [PATCH] Try harder to fix recently introduced crashes in ggc_collect

2017-05-24 Thread Jason Merrill
On Wed, May 24, 2017 at 7:04 AM, Trevor Saunders wrote: > On Fri, May 19, 2017 at 10:10:23AM -0400, Nathan Sidwell wrote: >> On 05/19/2017 10:05 AM, Bernd Edlinger wrote: >> >> > hwi cannot be the root cause of the problem, >> > because it can only be long_integer_type_node >> > or long_long_integ

Re: C++ PATCH to add __integer_pack built-in for std::make_integer_sequence (c++/80396)

2017-05-24 Thread Jakub Jelinek
Excess errors: > >> /daten/aranym/gcc/gcc-20170524/gcc/testsuite/g++.dg/ext/integer-pack2.C:10:48: > >> error: overflow in constant expression [-fpermissive] > >> /daten/aranym/gcc/gcc-20170524/gcc/testsuite/g++.dg/ext/integer-pack2.C:10:48: > >> error: overf

Re: [patch] build xz (instead of bz2) compressed tarballs and diffs

2017-05-24 Thread Gerald Pfeifer
On Tue, 23 May 2017, Matthias Klose wrote: > Looks like the copy of the script on gcc.gnu.org affects all active branches. Yeah, I just run into this myself (and scratched my had until I debugged this, admittedly quickly). > the May 23 GCC 5 snapshot was created successfully. Is this acceptable?

Re: [PATCH] PR c++/77306 - Unable to specify visibility for explicit template instantiations

2017-05-24 Thread Jason Merrill
On Sat, Apr 22, 2017 at 4:46 PM, James Abbatiello wrote: > PR c++/77306 > * attribs.c (decl_attributes): Allow visibility attributes on explicit > template instantiations. > * gcc-family/c-attribs.c (handle_visibility_attribute): Likewise. This doesn't seem suffici

Re: [PATCH v2] Implement non-trivial std::random_device::entropy (PR libstdc++/67578)

2017-05-24 Thread Jonathan Wakely
On 23/05/17 17:15 +0100, Jonathan Wakely wrote: On 22/05/17 22:28 +0800, Xi Ruoyao wrote: The new patch is attached.  Just merged the patches you sent and fixed the ChangeLog of gnu.ver and testsuite_abi.cc. (For fun:  I had mistakenly attached the Vim .swp file of the patch and almost sent it.

Re: [PATCH] Try harder to fix recently introduced crashes in ggc_collect

2017-05-24 Thread Nathan Sidwell
On 05/24/2017 03:02 PM, Jason Merrill wrote: I think it would be premature to remove PCH from GCC 8. The Modules specification is far from stable, and moving current PCH users to Modules will take significant work (at least if we get something like the current draft rather than the relatively t

Re: [PATCH] PR c++/80544 strip cv-quals from cast results

2017-05-24 Thread Jonathan Wakely
On 24/05/17 14:50 -0400, Jason Merrill wrote: On Wed, May 24, 2017 at 10:20 AM, Jonathan Wakely wrote: On 23/05/17 16:26 -0400, Jason Merrill wrote: On Tue, May 23, 2017 at 2:00 PM, Jonathan Wakely wrote: On 19/05/17 15:14 -0400, Jason Merrill wrote: On Thu, Apr 27, 2017 at 12:59 PM, Jo

Re: [PATCH 0/3] Split off powerpcspe from rs6000 port

2017-05-24 Thread Segher Boessenkool
On Thu, May 18, 2017 at 11:30:11AM -0500, Segher Boessenkool wrote: > On Mon, May 15, 2017 at 08:51:49PM +, Segher Boessenkool wrote: > > As discussed before, here is a series to split powerpcspe from the > > rs6000 port. This series does not yet make any real changes to either > > port: it is

Re: [config PATCH] --enable-languages

2017-05-24 Thread Martin Sebor
On 05/19/2017 10:59 AM, Nathan Sidwell wrote: --enable-languages=all confused me, because as RichardB said, 'it really means default'. So this patch does 2 things 1) allow --enable-languages=default to mean what =all does now. 2) change =all to mean all available languages. 2.1) jit is included

Re: {PATCH] New C++ warning -Wcatch-value

2017-05-24 Thread Jason Merrill
On Mon, May 15, 2017 at 3:58 PM, Martin Sebor wrote: >> So how about the following then? I stayed with the catch part and added >> a parameter to the warning to let the user decide on the warnings she/he >> wants to get: -Wcatch-value=n. >> -Wcatch-value=1 only warns for polymorphic classes that a

Re: [PATCH] warn on mem calls modifying objects of non-trivial types (PR 80560)

2017-05-24 Thread Martin Sebor
On 05/21/2017 10:50 PM, Jason Merrill wrote: On Sun, May 21, 2017 at 7:59 PM, Martin Sebor wrote: On 05/19/2017 03:42 PM, Jason Merrill wrote: On Fri, May 19, 2017 at 4:07 PM, Martin Sebor wrote: On 05/19/2017 01:07 PM, Jason Merrill wrote: On Tue, May 16, 2017 at 5:39 PM, Martin Sebor wro

Re: [PATCH] warn on mem calls modifying objects of non-trivial types (PR 80560)

2017-05-24 Thread Martin Sebor
Attached is an updated patch with the requested changes. I've also renamed the option -Wclass-memaccess to avoid suggesting that the warning focuses solely on non-trivial types, and updated its wording and comments throughout to refer to value initialization rather than default initialization. R

Re: [PATCH v2 04/N] Simplify usage of some TDF_* flags.

2017-05-24 Thread Martin Liška
Hi Richi. That's a small reminder. I CCed Steven, but I guess if the TDF_COMMENT would be really important, it can be eventually added back. May I install the patch now? Thanks, Martin

Re: [PATCH] warn on mem calls modifying objects of non-trivial types (PR 80560)

2017-05-24 Thread Jason Merrill
On 05/24/2017 04:28 PM, Martin Sebor wrote: Attached is an updated patch with the requested changes. I've also renamed the option -Wclass-memaccess to avoid suggesting that the warning focuses solely on non-trivial types, and updated its wording and comments throughout to refer to value initiali

Re: MinGW compilation warnings in libiberty's waitpid.c

2017-05-24 Thread DJ Delorie
Thanks. Committed!

Re: [PATCH] add more detail to -Wconversion and -Woverflow (PR 80731)

2017-05-24 Thread Martin Sebor
On 05/24/2017 11:08 AM, Joseph Myers wrote: On Wed, 17 May 2017, Martin Sebor wrote: @@ -1036,31 +1079,76 @@ warnings_for_convert_and_check (location_t loc, tree type, tree expr, /* This detects cases like converting -129 or 256 to unsigned char. */ if (!int_f

[C++ PATCH] Overload cleanups

2017-05-24 Thread Nathan Sidwell
here are a few more cleanups from the modules branch. firstl type_unknown_p had some historical baggage from when overloads were tree lists. No need for that, and then why not make it inline? is_overloaded_fn and friends are ripe for cleanup, and this does a little of that. Remove some use

[C++ PATCH] Sort Make-lang.in

2017-05-24 Thread Nathan Sidwell
Make-lang.in has a list ripe for sorting. So sorted. nathan -- Nathan Sidwell 2017-05-24 Nathan Sidwell * Make-lang.in (CXX_AND_OBJCXX_OBJS): Alphabetize. Index: Make-lang.in === --- Make-lang.in (revision 248432) +++ Make-lan

[C++ PATCH] Friend decls

2017-05-24 Thread Nathan Sidwell
With pushdecl revamped, there's no need for do_friend to do some checking with a local class friend. And therefore no need for the very specialized lookup_name_innermost_nonclass_level. push_local_binding will get some more cleanup soon. Committed to trunk. nathan -- Nathan Sidwell 2017-05-2

[libibery PATCH] Fix bootstrap

2017-05-24 Thread Nathan Sidwell
We now warn on casts to T const. Applied as obvious to fix bootstrap. nathan -- Nathan Sidwell Index: ChangeLog === --- ChangeLog (revision 248441) +++ ChangeLog (working copy) @@ -1,3 +1,7 @@ +2017-05-24 Nathan Sidwell + + * lib

Re: [libibery PATCH] Fix bootstrap

2017-05-24 Thread Nathan Sidwell
On 05/24/2017 08:34 PM, Nathan Sidwell wrote: We now warn on casts to T const. Applied as obvious to fix bootstrap. And this fixes c-common.c nathan -- Nathan Sidwell 2017-05-24 Nathan Sidwell * c=common.c (field_decl_cmp, resort_field_decl_cmp): Adjust T const casts to avoid warning.

Re: [libibery PATCH] Fix bootstrap

2017-05-24 Thread Nathan Sidwell
On 05/24/2017 08:56 PM, Nathan Sidwell wrote: On 05/24/2017 08:34 PM, Nathan Sidwell wrote: We now warn on casts to T const. Applied as obvious to fix bootstrap. And this fixes c-common.c And fix auto-profile.c nathan -- Nathan Sidwell 2017-05-24 Nathan Sidwell * auto-profile.c (afdo

Re: [libibery PATCH] Fix bootstrap

2017-05-24 Thread Nathan Sidwell
On 05/24/2017 09:13 PM, Nathan Sidwell wrote: On 05/24/2017 08:56 PM, Nathan Sidwell wrote: On 05/24/2017 08:34 PM, Nathan Sidwell wrote: We now warn on casts to T const. Applied as obvious to fix bootstrap. And this fixes c-common.c And fix auto-profile.c and lto-streamer-out.c, sigh -

  1   2   >