Re: [PATCH, wwwdocs] Add link to GCC 7 changes.html

2016-10-26 Thread Gerald Pfeifer
On Tue, 25 Oct 2016, Peter Bergner wrote: >> Perhaps add a disclaimer at the top of changes.html that this >> is still work in progress as part of that commit? > Do you mean like the following? If so, we'd have to remember to > remove the last hunk when GCC 7 is released. Yep, something like tha

[PATCH] Show INSN_UIDs in compact mode

2016-10-26 Thread David Malcolm
On Thu, 2016-10-20 at 16:11 +0200, Bernd Schmidt wrote: > On 10/20/2016 03:55 PM, David Malcolm wrote: > > Currently the jump insn in question looks like this: > > > > (cjump_insn (set (pc) > > (label_ref 20)) > > (nil)) > > > > With explicit INSN_UIDs i

Re: [PATCH] Show INSN_UIDs in compact mode

2016-10-26 Thread Bernd Schmidt
On 10/26/2016 08:50 PM, David Malcolm wrote: The following patch adds back in the INSN_UID for all insns. Successfully bootstrapped®rtested on x86_64-pc-linux-gnu. OK for trunk? Yes. Bernd

Re: [PATCH, wwwdocs] Add link to GCC 7 changes.html

2016-10-26 Thread Peter Bergner
On 10/26/16 1:10 PM, Gerald Pfeifer wrote: On Tue, 25 Oct 2016, Peter Bergner wrote: Perhaps add a disclaimer at the top of changes.html that this is still work in progress as part of that commit? Do you mean like the following? If so, we'd have to remember to remove the last hunk when GCC 7 i

[PR debug/77773] segfault when compiling __simd64_float16_t with -g

2016-10-26 Thread Aldy Hernandez
The following one-liner segfaults on arm-eabi when compiled with -mfloat-abi=hard -g: __simd64_float16_t usingit; The problem is that the pretty printer (in simple_type_specificer()) is dereferencing a NULL result from c_common_type_for_mode: int prec = TYPE_PRECISION (t);

Fix MIPS port WRT sprintf and fallthru warnings

2016-10-26 Thread Jeff Law
The fallthrus in mips16_constant_cost are pretty obvious. One buffer overflow in mips16_build_call_stub and one obvious fallthru comment adjustment in mips16_build_call_stub. The usual stuff. Installing. jeff commit c03f0de027e1079edd02e3494601c6def50dbac1 Author: law Date: Wed Oct 26

Fix sh port WRT fallthru

2016-10-26 Thread Jeff Law
This adjusts existing fallthru comments and adds some new ones. I couldn't convince myself the original code in movsicc was correct, It looks like we properly reverse the condition, but not the operands in the case of LT/LE/LEU/LTU. But then we do almost the same th ing in sh_emit_scc_to_t

Re: [RFC PATCH] avoid printing type suffix with %E

2016-10-26 Thread Joseph Myers
On Wed, 26 Oct 2016, Martin Sebor wrote: > The attached patch implements one such approach by having the pretty > printer recognize the space format flag to suppress the type suffix, > so "%E" still prints the suffix but "% E" does not. I did this to > preserve the existing output but I think it

RE: Fix MIPS port WRT sprintf and fallthru warnings

2016-10-26 Thread Matthew Fortune
Jeff Law writes: > The fallthrus in mips16_constant_cost are pretty obvious. One buffer > overflow in mips16_build_call_stub and one obvious fallthru comment > adjustment in mips16_build_call_stub. > > The usual stuff. Installing. Thanks Jeff. Matthew

[nvptx] propagating conditionals in worker-vector partitioned loops

2016-10-26 Thread Cesar Philippidis
Currently, the nvptx backend is only neutering the worker axis when propagating variables used in conditional expressions across the worker and vector axes. That's a problem with the worker-state spill and fill propagation implementation because all of the vector threads in worker 0 all write the t

[PATCH], Allow SImode to go into VSX registers on PowerPC ISA 2.07 (power8) and above

2016-10-26 Thread Michael Meissner
PowerPC GCC has traditionally only allowed DImode to go into FPR registers (and now VSX registers) in order to allow floating point conversions. Conversions to/from SImode have always had to deal with special UNSPECs to allow the generation of the LFIWAX, LXSIWAX, LFIWZX, LXSIWZX, STFIWX, and STXS

Re: [PATCH], Allow SImode to go into VSX registers on PowerPC ISA 2.07 (power8) and above

2016-10-26 Thread Michael Meissner
I forgot to mention, I will be working on a follow-on patch to this that enables QImode and HImode to go in the vector registers for ISA 3.0, since ISA 3.0 now adds load (with zero extend) and store instructions for those types. I probably also will update vector extract for the case where the sma

Re: [PATCH 7/7] make targetm.gen_ccmp{first,next} take rtx_insn **

2016-10-26 Thread Trevor Saunders
On Tue, Oct 18, 2016 at 01:25:55PM +0200, Bernd Schmidt wrote: > On 10/17/2016 09:46 PM, tbsaunde+...@tbsaunde.org wrote: > > From: Trevor Saunders > > > > gcc/ChangeLog: > > > > 2016-10-17 Trevor Saunders > > > > * ccmp.c (expand_ccmp_expr_1): Adjust. > > (expand_ccmp_expr): Likewis

[RFC PATCH] expand_strn_compare should attempt expansion even if neither string is constant

2016-10-26 Thread Aaron Sawdey
I'm currently working on a builtin expansion of strncmp for powerpc similar to the one for memcmp I checked recently. One thing I encountered is that the code in expand_strn_compare will not attempt to expand the cmpstrnsi pattern at all if neither string parameter is a constant string. This doesn'

Re: [PATCH][GIMPLE FE] Split out parser into separate file

2016-10-26 Thread Trevor Saunders
On Tue, Oct 25, 2016 at 03:33:36PM +0200, Richard Biener wrote: > > Hi, > > so I did the massaging to split out the GIMPLE parsing routines out > to a separate file (quite tricky to get the gengtype issues correctly > so I thought to help out here and get things started). actually it looks like

[PATCH] fix an uninitialized use of loc when parsing gimple switches

2016-10-26 Thread tbsaunde+gcc
From: Trevor Saunders gcc/c/ChangeLog: 2016-10-27 Trevor Saunders * gimple-parser.c (c_parser_gimple_switch_stmt): Fix uninitialized use of loc. --- gcc/c/gimple-parser.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/gcc/c/gimple-parser.c b/gcc/c/

Re: [PATCH] fix an uninitialized use of loc when parsing gimple switches

2016-10-26 Thread Richard Biener
On Thu, 27 Oct 2016, tbsaunde+...@tbsaunde.org wrote: > From: Trevor Saunders > > gcc/c/ChangeLog: > > 2016-10-27 Trevor Saunders > > * gimple-parser.c (c_parser_gimple_switch_stmt): Fix > uninitialized use of loc. Thanks, applied & pushed. Richard. > --- > gcc/c/gimple-pars

<    1   2