Re: [PATCH] Fix RTL DCE ICEs due to TYPE_EMPTY_P (PR c++/83490)

2017-12-19 Thread Jeff Law
On 12/19/2017 03:02 PM, Jakub Jelinek wrote: > Hi! > > The following testcase ICEs because we get: > (call_insn/u 7 4 14 2 (set (reg:QI 0 ax) > (call (mem:QI (symbol_ref:DI ("_Zlt1AS_") [flags 0x3] 0x7efd00f7fc00 operator<>) [0 operator< S1 A8]) > (const_int 0 [0]))) 690 {*ca

Re: [PATCH, PR83423] Don't call targetm.calls.static_chain in non-static function

2017-12-19 Thread Jeff Law
On 12/19/2017 09:34 AM, Tom de Vries wrote: > Hi, > > this patch fixes PR83423. > > > The default_static_chain hook has implemented a sorry if both > STATIC_CHAIN_INCOMING_REGNUM and STATIC_CHAIN_REGNUM are undefined: > ... > rtx > default_static_chain (const_tree ARG_UNUSED (fndecl_or_type), bo

Re: [PATCH] make canonicalize_condition keep its promise

2017-12-19 Thread Segher Boessenkool
Hi Jeff, On Tue, Dec 19, 2017 at 04:40:23PM -0700, Jeff Law wrote: > On 12/15/2017 02:16 AM, Segher Boessenkool wrote: > >> The only way to get into check_simple_exit is via find_simple_exit which > >> is only called from get_simple_loop_desc. > >> > >> And if you're calling get_simple_loop_desc,

[PATCH] skip memcpy-6.c for failing targets (PR 83483)

2017-12-19 Thread Martin Sebor
As discussed in PR 83483, the gcc.dg/memcpy-6.c test I added along with the -Wrestrict changes fails on many targets because of bad assumptions on my part. The test wasn't quite ready to commit but I forgot to get back to it and finish it up. Rather than remove it, the attached patch skips runni

Re: [PATCH] make canonicalize_condition keep its promise

2017-12-19 Thread Segher Boessenkool
On Tue, Dec 19, 2017 at 04:56:16PM -0700, Jeff Law wrote: > With that in mind your patch is fine. Thanks for the reviewing :-) > I will note that I find it highly confusing that we attach a simple loop > descriptor to a loop that is not a simple loop. But clearly you didn't > introduce that oddb

Re: [PATCH, PR83492] Fix selection of aarch64 big-endian shift parameters based on __AARCH64EB__

2017-12-19 Thread Jeff Law
On 12/19/2017 11:23 AM, Jakub Jelinek wrote: > On Tue, Dec 19, 2017 at 07:01:44PM +0100, Michael Weiser wrote: >> Hi Kyrill, >> >> On Tue, Dec 19, 2017 at 05:35:03PM +, Kyrill Tkachov wrote: >> below patch fixes PR 83492. >>> I agree with your analysis of the bug and your patch will fix th

Re: [14/nn] Add helpers for shift count modes

2017-12-19 Thread Jeff Law
On 12/19/2017 12:13 PM, Richard Sandiford wrote: > Richard Sandiford writes: >> Richard Biener writes: >>> On Fri, Dec 15, 2017 at 1:48 AM, Richard Sandiford >>> wrote: Richard Biener writes: > On Mon, Nov 20, 2017 at 10:02 PM, Richard Sandiford > wrote: >> Richard Biener wri

Re: [PATCH] Improve alloca alignment

2017-12-19 Thread Jeff Law
On 12/13/2017 04:17 PM, Eric Botcazou wrote: >> No clue, but ISTM that it should. Eric, can you try that and see if it >> addresses these problems? I'd really like to get this wrapped up, but I >> don't have access to any sparc systems to test it myself. > > Yes, the INIT_EXPANDERS trick works f

Re: [PATCH,PTX] Add support for CUDA 9

2017-12-19 Thread Tom de Vries
On 12/20/2017 12:25 AM, Cesar Philippidis wrote: In CUDA 9, Nvidia removed support for treating the labels of functions as generic address spaces as part of their PTX 6.0 changes. More specifically, :

[PATCH] Eliminate location wrappers in tree_nop_conversion/STRIP_NOPS

2017-12-19 Thread David Malcolm
On Tue, 2017-12-19 at 23:03 +0100, Jakub Jelinek wrote: > On Tue, Dec 19, 2017 at 04:59:34PM -0500, Jason Merrill wrote: > > > Or do you mean it should strip just the special VIEW_CONVERT_EXPR > > > that has type identical to the operand's type? > > > > That; interpreting something as the same typ

[v2 of PATCH 15/14] Use fold_for_warn in get_atomic_generic_size

2017-12-19 Thread David Malcolm
On Tue, 2017-12-19 at 15:35 -0500, Jason Merrill wrote: > On Sat, Dec 16, 2017 at 8:12 PM, David Malcolm > wrote: > > I rebased the v2 patchkit; here's an extra patch to fix an issue > > with it uncovered by a recently-added testcase (in r254990). > > > > With the patch kit, but without this patc

Re: [PATCH] Fix PR83418

2017-12-19 Thread Jeff Law
On 12/15/2017 09:30 AM, Richard Biener wrote: > On December 15, 2017 5:27:14 PM GMT+01:00, Jeff Law wrote: >> On 12/15/2017 01:10 AM, Richard Biener wrote: >>> On Thu, 14 Dec 2017, Richard Biener wrote: >>> On December 14, 2017 4:43:42 PM GMT+01:00, Jeff Law >> wrote: > On 12/14/2017 01:

Re: [v2 of PATCH 15/14] Use fold_for_warn in get_atomic_generic_size

2017-12-19 Thread Jason Merrill
On Tue, Dec 19, 2017 at 7:53 PM, David Malcolm wrote: > On Tue, 2017-12-19 at 15:35 -0500, Jason Merrill wrote: >> On Sat, Dec 16, 2017 at 8:12 PM, David Malcolm >> wrote: >> > I rebased the v2 patchkit; here's an extra patch to fix an issue >> > with it uncovered by a recently-added testcase (in

Re: [PATCH] Eliminate location wrappers in tree_nop_conversion/STRIP_NOPS

2017-12-19 Thread Jeff Law
On 12/19/2017 05:44 PM, David Malcolm wrote: > On Tue, 2017-12-19 at 23:03 +0100, Jakub Jelinek wrote: >> On Tue, Dec 19, 2017 at 04:59:34PM -0500, Jason Merrill wrote: Or do you mean it should strip just the special VIEW_CONVERT_EXPR that has type identical to the operand's type? >>> >>>

Re: [PATCH] skip memcpy-6.c for failing targets (PR 83483)

2017-12-19 Thread Jeff Law
On 12/19/2017 05:18 PM, Martin Sebor wrote: > As discussed in PR 83483, the gcc.dg/memcpy-6.c test I added > along with the -Wrestrict changes fails on many targets because > of bad assumptions on my part.  The test wasn't quite ready to > commit but I forgot to get back to it and finish it up. >

Re: [PATCH] Fix PR83452

2017-12-19 Thread Alan Modra
On Tue, Dec 19, 2017 at 03:37:10PM +0100, Richard Biener wrote: > + while ((gnu_lto = memchr (gnu_lto, 'g', > + strings + strsz - gnu_lto))) > + if (strncmp (gnu_lto, "gnu_lto_v1", > + strings + strsz - gnu_lto) == 0

Re: [PATCH][i386,AVX] Enable VPCLMULQDQ support

2017-12-19 Thread Kirill Yukhin
Hello Julia, On 09 Nov 09:47, Koval, Julia wrote: > Hi, this patch enables VPCLMULQDQ instruction from VPCLMULQDQ isaset, defined > here: > https://software.intel.com/sites/default/files/managed/c5/15/architecture-instruction-set-extensions-programming-reference.pdf > > Ok for trunk? Your patch

Re: [SFN] Bootstrap broken

2017-12-19 Thread Alexandre Oliva
On Dec 19, 2017, Jakub Jelinek wrote: > Has the patch you've posted passed bootstrap/regtest? Yeah, here's the latest version, that survived O1, O2 and O3 bootstraps with bootstrap-debug (-g0 for stage2), bootstrap-debug-lean (-fcompare-debug for stage3) and bootstrap-debug-lib (-fcompare-debug

<    1   2