Re: VN_INFO might be NULL in PRE

2012-10-23 Thread Zhenqiang Chen
On 23 October 2012 18:02, Richard Biener wrote: > On Tue, Oct 23, 2012 at 7:36 AM, Zhenqiang Chen > wrote: >> Hi, >> >> PRE bases on the result of value numbering (run_scc_vn). At the end, >> it free_scc_vn. But before free_scc_vn, it might call cleanup_tree_cfg >> (); >> >> if (do_eh_cleanup |

Re: if-conversion/HOT-COLD partitioning

2012-10-23 Thread Steven Bosscher
On Tue, Oct 23, 2012 at 10:29 PM, Christophe Lyon wrote: > Well, both of these functions appear to check that the 2 blocks to > merge belong to the same partition, so it should be OK. In your first email, you said if-convert was merging two blocks from different partitions. can_merge_block_p() wo

Re: alias template bug?

2012-10-23 Thread Paolo Carlini
.. right now I can't really check, but looks like 54912 & co Paolo

Re: alias template bug?

2012-10-23 Thread Marc Glisse
On Tue, 23 Oct 2012, Andrew Sutton wrote: I just updated to GCC from trunk (r192711) and started getting some peculiar errors. In particular, GCC seems to believe that sizeof(T) is no longer a constant when used as an argument to an alias template. I think Jakub just fixed that one an hour ago

Re: if-conversion/HOT-COLD partitioning

2012-10-23 Thread Christophe Lyon
On 23 October 2012 19:45, Steven Bosscher wrote: > Christophe wrote: >> I think merge_blocks() should be modified to handle such cases; > > I think can_merge_blocks should be fixed. Blocks from different > partitions should not be merged. See cfgrtl.c:rtl_can_merge_blocks and > cfgrtl.c:cfg_layout

Re: alias template bug?

2012-10-23 Thread Andrew Sutton
>> I just updated to GCC from trunk (r192711) and started getting some >> peculiar errors. In particular, GCC seems to believe that sizeof(T) is >> no longer a constant when used as an argument to an alias template. > > I think Jakub just fixed that one an hour ago or so. And committed the changes

Re: alias template bug?

2012-10-23 Thread Marc Glisse
On Tue, 23 Oct 2012, Andrew Sutton wrote: I just updated to GCC from trunk (r192711) and started getting some peculiar errors. In particular, GCC seems to believe that sizeof(T) is no longer a constant when used as an argument to an alias template. I think Jakub just fixed that one an hour ago

alias template bug?

2012-10-23 Thread Andrew Sutton
I just updated to GCC from trunk (r192711) and started getting some peculiar errors. In particular, GCC seems to believe that sizeof(T) is no longer a constant when used as an argument to an alias template. Here's the smallest reproduction I can create with the relevant error: template struct t

Re: bugzilla vs. MAINTAINERS

2012-10-23 Thread Jonathan Wakely
On 23 October 2012 15:58, Joern Rennecke wrote: > > Well, that'd be all right with me, but if there is no @gcc.gnu.org address > that matches the address listed in MAINTAINERS, it should allow the address > from MAINTAINERS. Of course, if there is a @gcc.gnu.org address for that > person that bugz

RE: Documentation problem with TEXT_SECTION_ASM_OP

2012-10-23 Thread Joseph S. Myers
On Tue, 23 Oct 2012, Paulo Matos wrote: > If this -fbuilding-libgcc being passed at current HEAD? I am looking at > 4.7 and no flag like that Is being passed while building libgcc. It's in LIBGCC2_CFLAGS in libgcc/Makefile.in in both 4.7 and trunk. -- Joseph S. Myers jos...@codesourcery.com

RE: Documentation problem with TEXT_SECTION_ASM_OP

2012-10-23 Thread Paulo Matos
> -Original Message- > From: gcc-ow...@gcc.gnu.org [mailto:gcc-ow...@gcc.gnu.org] On Behalf Of > Joseph S. Myers > Sent: 23 October 2012 15:28 > To: Paulo Matos > Cc: gcc@gcc.gnu.org > Subject: Re: Documentation problem with TEXT_SECTION_ASM_OP > > At

Re: bugzilla vs. MAINTAINERS

2012-10-23 Thread Joern Rennecke
Quoting Richard Biener : Can we instead make it sort @gcc.gnu.org accounts first when it asks to confirm a CC member? Hard to pick the correct person from all the noise ... (if you happen to not remember the exact match). Well, that'd be all right with me, but if there is no @gcc.gnu.org a

Re: bugzilla vs. MAINTAINERS

2012-10-23 Thread Richard Biener
On Tue, Oct 23, 2012 at 4:37 PM, Joern Rennecke wrote: > Could we make bugzilla recognize all the email addresses in the MAINTAINERs > file? It is frustrating to identify the maintainer of the piece that's > broken, only to find that bugzilla will refuse to add the person the the CC > list. This

bugzilla vs. MAINTAINERS

2012-10-23 Thread Joern Rennecke
Could we make bugzilla recognize all the email addresses in the MAINTAINERs file? It is frustrating to identify the maintainer of the piece that's broken, only to find that bugzilla will refuse to add the person the the CC list. This affects a number of the sub PRs of PR47093 / PR44756 , which i

Re: Documentation problem with TEXT_SECTION_ASM_OP

2012-10-23 Thread Joseph S. Myers
On Tue, 23 Oct 2012, Paulo Matos wrote: > Since this is most likely a problem with the docs, how can I achieve > something like this and have only a string literal as value or > TEXT_SECTION_ASM_OP? At I listed this macro, and lots of others

if-conversion/HOT-COLD partitioning

2012-10-23 Thread Christophe Lyon
Hi, While debugging a GCC failure on ARM when compiling with -fprofile-use, I came across a case where if_convert merges 2 blocks which belong to different partitions (hot/cold). In turn merge_blocks() (in cfghooks.c) merges the BB flags by ORing them, resulting in a block belonging to both cold

RE: Documentation problem with TEXT_SECTION_ASM_OP

2012-10-23 Thread Paulo Matos
> -Original Message- > From: gcc-ow...@gcc.gnu.org [mailto:gcc-ow...@gcc.gnu.org] On Behalf Of > Paulo Matos > Sent: 23 October 2012 14:47 > To: gcc@gcc.gnu.org > Subject: RE: Documentation problem with TEXT_SECTION_ASM_OP > > This sentence is quite confusing. 'their values must be run-ti

RE: Documentation problem with TEXT_SECTION_ASM_OP

2012-10-23 Thread Paulo Matos
I just noticed at the top of Sections.html in gccint: The normal way of controlling a foo_section variable is to define the associated FOO_SECTION_ASM_OP macro, as described below. The macros are only read once, when varasm.c initializes itself, so their values must be run-time constants. This

Documentation problem with TEXT_SECTION_ASM_OP

2012-10-23 Thread Paulo Matos
Hi, TEXT_SECTION_ASM_OP doc says: "A C expression whose value is a string, including spacing, containing the assembler operation that should precede instructions and read-only data." However, crtstuff.c uses `asm(TEXT_SECTION_ASM_OP)'. I am trying to use a C expression for TEXT_SECTION_ASM_OP l

Re: VN_INFO might be NULL in PRE

2012-10-23 Thread Richard Biener
On Tue, Oct 23, 2012 at 7:36 AM, Zhenqiang Chen wrote: > Hi, > > PRE bases on the result of value numbering (run_scc_vn). At the end, > it free_scc_vn. But before free_scc_vn, it might call cleanup_tree_cfg > (); > > if (do_eh_cleanup || do_ab_cleanup) > cleanup_tree_cfg (); > > cleanup_tree

Re: Question on updating DF info during code hoisting

2012-10-23 Thread Steven Bosscher
On Tue, Oct 23, 2012 at 5:17 AM, Bin.Cheng wrote: > Thanks for your explanation. > Now I understand how df_insn_info is updated when > deleting/modifying/creating insn. One more question is when and how > IN/OUT information is updated. GCC calls df_set_bb_dirty when handling > insns, but I did not