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

> 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

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

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 > >

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

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][3/6]Fix PR80815 by handling negative DR_STEPs in runtime alias check

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

[PATCH] [ARC] Fix tst_movb pattern.

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

Re: [PATCH] gcc::context creation

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 > ...

Re: [PATCH] gcc::context creation

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

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

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 > > >

Re: OpenACC 1.0 compatibility: acc_async_wait, acc_async_wait_all

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, > >

[C++ PATCH] Hidden functions

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

Re: Implementing OpenACC's Fortran module

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

Re: [PATCH] gcc::context creation

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

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

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?

Re: [PATCH] gcc::context creation

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

[Committed] S/390: Fix PR80725.

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

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

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 >> *

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

"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

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

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. > >

Re: [PATCH] gcc::context creation

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

Re: OpenACC 1.0 compatibility: acc_async_wait, acc_async_wait_all

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 > >

Re: Implementing OpenACC's Fortran module

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

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

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

Re: Implementing OpenACC's Fortran module

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

Re: [PATCH] Dump function on internal errors

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

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

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

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

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

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

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: [PATCH] rs6000: Fix for separate shrink-wrapping for fp (PR80860, PR80843)

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

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

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 > > >

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

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

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

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

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

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 GCC8][31/33]Set range information for niter bound of vectorized loop

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 >>>

Re: OpenACC 1.0 compatibility: acc_async_wait, acc_async_wait_all

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:

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

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

Re: [config PATCH] --enable-languages

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

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

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

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

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

Re: [PATCH] Fix PR80754

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

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

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 0/5 v3] Vect peeling cost model

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

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

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

Re: Reorganise machmode.h headers

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

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

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

Update libstdc++ baseline symbols (ia64, m68k)

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

[C++ PATCH] Hidden name removal

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

Re: SSA range class and removal of VR_ANTI_RANGEs

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

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

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

Re: SSA range class and removal of VR_ANTI_RANGEs

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

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

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

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

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. > >

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

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

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

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

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

> 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

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

On Wed, May 24, 2017 at 11:08 AM, Jakub Jelinek <ja...@redhat.com> 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-201705

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

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,

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

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.

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

cess 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] Try harder to fix recently introduced crashes in ggc_collect

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

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

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

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

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

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

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

Re: [config PATCH] --enable-languages

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

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

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] build xz (instead of bz2) compressed tarballs and diffs

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

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

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

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

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

Re: [libibery PATCH] Fix bootstrap

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

[PATCH rs6000] Addition fixes to BMI intrinsic test

Bill Seurer pointed out that building the BMI tests on a power8 but with gcc built --with-cpu=power6 fails with link errors. The intrinsics _pdep_u64/32 and _pext_u64/32 are guarded with #ifdef _ARCH_PWR7 as the implementation uses bpermd and popcntd instructions introduced with power7

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

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

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

On Wed, May 24, 2017 at 8:07 PM, Andrew Pinski wrote: > On Wed, May 24, 2017 at 12:29 PM, Jonathan Wakely wrote: >> On 24/05/17 14:50 -0400, Jason Merrill wrote: >>> >>> On Wed, May 24, 2017 at 10:20 AM, Jonathan Wakely >>> wrote:

Re: [libibery PATCH] Fix bootstrap

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 *

Re: Ipa function summary pass

Hi, On mips64el target: New Fail: FAIL: gfortran.dg/pr48636.f90 -O scan-ipa-dump fnsummary "inline hints: loop_iterations" Paul On Wed, May 24, 2017 at 4:47 PM, Christophe Lyon wrote: > Hi, > > On 23 May 2017 at 18:23, Jan Hubicka wrote: >> Hi,

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

On Wed, May 24, 2017 at 12:29 PM, Jonathan Wakely wrote: > 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,

Re: [PATCH v2 3/N] Transform TDF_{lang,tree,ipa,rtl} to dump_kind enum.

On 05/12/2017 07:04 AM, Martin Liška wrote: Third part removes TDF_* flags mentioned in the subject. These flags are used to enable all passes of specific type and Nathan has recently separated these by a new pair of macros. I hope moving these to a separate enum will help even more. Just as

Re: [libibery PATCH] Fix bootstrap

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

[C++ PATCH] Overload cleanups

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] Friend decls

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

Re: [libibery PATCH] Fix bootstrap

On May 25, 2017 3:22:18 AM GMT+02:00, Nathan Sidwell wrote: >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

[libibery PATCH] Fix bootstrap

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

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

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

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

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 bindings individually; users need > to give each of the

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

> The compare-and-branch distance was calculated incorrectly. This occurred > because a -mflat sibcall returned an instruction count of 2 instead of 3. > Fix sparc.md to match the output_sibcall() code in sparc.c. > > * config/sparc/sparc.md: Set the number of instructions correctly >

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

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

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

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)

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

Re: SSA range class and removal of VR_ANTI_RANGEs

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

Fix var-tracking for special record parameters passed indirectly

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

Re: SSA range class and removal of VR_ANTI_RANGEs

..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

Fix pessimistic DImode handling in combine.c:make_field_assignment

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

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

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

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

> 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

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

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

Re: libgomp nvptx plugin: Debugging output for cuInit failure

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

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

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

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

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

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

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

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

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

Re: Ipa function summary pass

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

Re: signed multiplication for pointer offsets

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

  1   2   >