Re: [Patch, Fortran] PR56907 - do not 'pack' arrays passed to C_LOC

2013-04-20 Thread Janus Weil
>> What I don't quite understand is: >> @@ -6317,8 +6317,13 @@ conv_isocbinding_function (gfc_se *se, gfc_expr >> *expr) >> { >> if (arg->expr->rank == 0) >> gfc_conv_expr_reference (se, arg->expr); >> - else >> + else if (gfc_is_simply_contiguous (arg->expr, false)) >

Re: [Patch, Fortran] PR56907 - do not 'pack' arrays passed to C_LOC

2013-04-20 Thread Tobias Burnus
Am 20.04.2013 12:42, schrieb Janus Weil: Well, it does: As it doesn't know whether the array is contiguous or not - it packs the array. That's what one usually wants. However, for C_LOC one knows that the array is contiguous - the user promises this to the compiler - and, hence, no packing is n

Re: [Patch, Fortran] PR56907 - do not 'pack' arrays passed to C_LOC

2013-04-20 Thread Janus Weil
>>> Well, it does: As it doesn't know whether the array is contiguous or not >>> - it packs the array. That's what one usually wants. However, for C_LOC one >>> knows that the array is contiguous - the user promises this to the compiler >>> - and, hence, no packing is needed. >> >> Ok, I see. Then

Re: [patch][mips] split mips_reorg in pre- and post-dbr_schedule parts

2013-04-20 Thread Steven Bosscher
*ping* MIPS maintainers... On Mon, Apr 15, 2013 at 5:09 PM, Jeff Law wrote: > On 04/14/2013 08:20 AM, Steven Bosscher wrote: >> >> Hello, >> >> This patch splits mips_reorg.c in a pre-dbr_schedule part and a new, >> machine specific post-dbr_schedule pass. With this patch, >> cleanup_barriers and

Re: [PATCH] Fix PR56982, handle setjmp like non-local labels

2013-04-20 Thread Andreas Schwab
Richard Biener writes: > PR tree-optimization/56982 > * builtins.def (BUILT_IN_LONGJMP): longjmp is not a leaf > function. > * gimplify.c (gimplify_call_expr): Notice special calls. > (gimplify_modify_expr): Likewise. > * tree-cfg.c (make_abnormal_goto_edges):

Re: [patch, fortran, backport, 4.8] PR51825 - Fortran runtime error: Cannot match namelist object name

2013-04-20 Thread Tilo Schwarz
On Sat, 13 Apr 2013 10:29:26 +0200, Janus Weil wrote: Hi Tilo, I would like to backport the fix for PR51825 I posted here http://gcc.gnu.org/ml/gcc-patches/2013-03/msg00316.html to the 4.8 branch. well, the usual gfortran policy is to only do backports of regression fixes. In exceptional

[patch] fix anachronism in libstdc+ docs

2013-04-20 Thread Jonathan Wakely
* doc/xml/manual/extensions.xml: Fix anachronism. committed to trunk commit ab11f53f4046adc88e1f43568836c572bcd3bc44 Author: Jonathan Wakely Date: Sat Apr 20 20:31:32 2013 +0100 * doc/xml/manual/extensions.xml: Fix anachronism. diff --git a/libstdc++-v3/doc/xml/manual/extensio

Re: [PATCH] Fix PR56982, handle setjmp like non-local labels

2013-04-20 Thread Ian Lance Taylor
On Sat, Apr 20, 2013 at 5:14 AM, Andreas Schwab wrote: > Richard Biener writes: > >> PR tree-optimization/56982 >> * builtins.def (BUILT_IN_LONGJMP): longjmp is not a leaf >> function. >> * gimplify.c (gimplify_call_expr): Notice special calls. >> (gimplify_modify_ex

Simple change in dot dumper -- Display profile count and branch probability

2013-04-20 Thread Xinliang David Li
Hi, the graph dump file currently does not show any profile information. The following simple patch fixed that. Ok for trunk? thanks, David 2013-04-20 Xinliang David Li * graph.c (draw_cfg_node): Add count and frequency info. (draw_cfg_node_succ_edges): Add branch probility

Re: Simple change in dot dumper -- Display profile count and branch probability

2013-04-20 Thread Steven Bosscher
On Sun, Apr 21, 2013 at 12:47 AM, Xinliang David Li wrote: > Index: graph.c > === > --- graph.c (revision 198108) > +++ graph.c (working copy) > @@ -110,6 +110,9 @@ draw_cfg_node (pretty_printer *pp, int f >else > { >

Re: [google][4.7] Generate a label for the split cold function while using -freorder-blocks-and-partition

2013-04-20 Thread Sriraman Tallam
Fixed a bug in the previous patch sent, where I did not check if the switch section was actually to the cold function part. Updated patch attached. Thanks Sri On Fri, Apr 19, 2013 at 1:58 PM, Sriraman Tallam wrote: > Updated patch attached. > > Thanks > Sri > > On Fri, Apr 19, 2013 at 1:43 P

Re: GCC does not support *mmintrin.h with function specific opts

2013-04-20 Thread Sriraman Tallam
Ping. On Wed, Apr 17, 2013 at 7:13 PM, Sriraman Tallam wrote: > +HJ > > On Tue, Apr 16, 2013 at 1:54 PM, Sriraman Tallam wrote: >> Hi, >> >> I have attached an updated patch that addresses all the comments raised. >> >> On Fri, Apr 12, 2013 at 1:58 AM, Jakub Jelinek wrote: >>> On Thu, Apr 11,

Re: Simple change in dot dumper -- Display profile count and branch probability

2013-04-20 Thread Xinliang David Li
Thanks. The patch is revised. David 2013-04-20 Xinliang David Li * graph.c (draw_cfg_node_succ_edges): Add branch probility as label. * cfghhooks.c (dump_bb_for_graph): Dump profile count and frquency. * Makefile.in: New dependency. Index: cfghooks.c ===