Re: [PATCH 19/89] Const-correctness of gimple_call_builtin_p

2014-05-08 Thread Jeff Law
On 04/21/14 10:56, David Malcolm wrote: gcc/ * gimple.h (gimple_builtin_call_types_compatible_p): Accept a const_gimple, rather than a gimple. (gimple_call_builtin_p): Likewise, for the three variants. * gimple.c (gimple_builtin_call_types_compatible_p): Likewise.

Re: [PATCH, 2/2] shrink wrap a function with a single loop: split live_edge

2014-05-08 Thread Zhenqiang Chen
On 9 May 2014 14:08, Jeff Law wrote: > On 05/08/14 02:07, Zhenqiang Chen wrote: >> >> Hi, >> >> The patch splits the live_edge for move_insn_for_shrink_wrap to sink >> the copy out of the entry block. >> >> Bootstrap and no make check regression on X86-64 and ARM. >> >> OK for trunk? >> >> Thanks!

Re: [PATCH 0/3] Compile-time gimple checking, without typedefs

2014-05-08 Thread Jeff Law
On 05/02/14 15:56, David Malcolm wrote: This patch series demonstrates a way of reimplementing the 89-patch series: "[PATCH 00/89] Compile-time gimple-checking" http://gcc.gnu.org/ml/gcc-patches/2014-04/msg01148.html whilst avoiding introducing a pair of "gimple_foo/const_gimple_foo" ty

Re: [RFC][PING^2] Do not consider volatile asms as optimization barriers #1

2014-05-08 Thread Eric Botcazou
> It has been two weeks since Richard commited this to trunk. Perhaps it's > ok to backport to 4.8 branch now? Richard, can you do that before the 4.8.3 release? Thanks in advance. -- Eric Botcazou

Re: [PATCH, 1/2] shrink wrap a function with a single loop: copy propagation

2014-05-08 Thread Jeff Law
On 05/08/14 02:06, Zhenqiang Chen wrote: Hi, Similar issue was discussed in thread http://gcc.gnu.org/ml/gcc-patches/2013-04/msg01145.html. The patches are close to Jeff's suggestion: "sink just the moves out of the incoming argument registers". The patch and following one try to shrink wrap a

Re: [PATCH 1/3] Handwritten part of conversion of "gimple" to "gimple_stmt *"

2014-05-08 Thread Jeff Law
On 05/02/14 15:56, David Malcolm wrote: gcc/ * coretypes.h (gimple): Drop typedef. (const_gimple): Likewise. (gimple_seq): Convert from being a "gimple" to a "gimple_stmt *". So instead, we just want to convert from "gimple" to "gimple *", right? That was my understandin

Re: [PATCH 2/3] Autogenerated part of conversion of "gimple" to "gimple_stmt *"

2014-05-08 Thread Jeff Law
On 05/02/14 15:56, David Malcolm wrote: This autogenerated patch is 1.2M in size, so I've uploaded it to: http://dmalcolm.fedorapeople.org/gcc/large-patches/e5a7d209a8789e6df9eccb9210d4a6dbe26d0fd4-0002-Autogenerated-part-of-conversion-of-gimple-to-gimple.patch to avoid being anti-social on the

Re: [PATCH 3/3] Introduce gimple_switch and use it in various places

2014-05-08 Thread Jeff Law
On 05/02/14 15:56, David Malcolm wrote: gcc/ * gimple.h (gimple_switch): New subclass of gimple_stmt_with_ops, adding the invariant that stmt->code == GIMPLE_SWITCH. (is_a_helper ::test): New. (is_a_helper ::test): New. (gimple_try): Likewise. * co

<    1   2