Re: New branch for STL Advisor

2008-07-11 Thread Silvius Rus
Paolo Carlini wrote: ... finally (for now ;), an apparently pedantic issue, but really I'm going to strongly object to any use of "STL" together with our implementation of the ISO C++ Runtime Library: it's a *legacy* HP / SGI acronym which is not used anywhere in the ISO Standard in force (C++

Re: New branch for STL Advisor

2008-07-11 Thread Silvius Rus
Paolo Carlini wrote: Also, maybe it's just me, but the specific advantages over normal profiling / existing tools, don't seem completely obvious, I'd like to see that point discussed in better detail... Paolo. The effect of our instrumentation is a meaningful trace of the behavior of co

Re: New branch for STL Advisor

2008-07-11 Thread Silvius Rus
Paolo Carlini wrote: libstdc++-v3 maintainers, could you please comment/advise? Can you explain which is the role of the debug-mode code, here? Because certainly the *performance* of the debug-mode library have *nothing* to do with the performance of the "real" library (whether that coul

Re: New branch for STL Advisor

2008-07-11 Thread Lixia Liu
The advisory tool aims to give high-level suggestions, mostly focuses on the access pattern and size of STL. Such as, shown in example, we want to decide the right initial size for the container to reduce execution time as much as possible. Another case is we can suggest that a hash-map is more

gcc-4.4-20080711 is now available

2008-07-11 Thread gccadmin
Snapshot gcc-4.4-20080711 is now available on ftp://gcc.gnu.org/pub/gcc/snapshots/4.4-20080711/ and on various mirrors, see http://gcc.gnu.org/mirrors.html for details. This snapshot has been generated from the GCC 4.4 SVN branch with the following options: svn://gcc.gnu.org/svn/gcc/trunk

Re: New branch for STL Advisor

2008-07-11 Thread Paolo Carlini
... finally (for now ;), an apparently pedantic issue, but really I'm going to strongly object to any use of "STL" together with our implementation of the ISO C++ Runtime Library: it's a *legacy* HP / SGI acronym which is not used anywhere in the ISO Standard in force (C++98 + TC1) or in the wor

Re: New branch for STL Advisor

2008-07-11 Thread Paolo Carlini
Also, maybe it's just me, but the specific advantages over normal profiling / existing tools, don't seem completely obvious, I'd like to see that point discussed in better detail... Paolo.

Re: New branch for STL Advisor

2008-07-11 Thread Paolo Carlini
> libstdc++-v3 maintainers, could you please comment/advise? Can you explain which is the role of the debug-mode code, here? Because certainly the *performance* of the debug-mode library have *nothing* to do with the performance of the "real" library (whether that could be improved, tweaked, or

New branch for STL Advisor

2008-07-11 Thread Silvius Rus
Lixia Liu ([EMAIL PROTECTED]) and myself started to work on a tool to recognize STL usage patterns that can be corrected to increase application performance. We believe this tool should live in libstdc++-v3. We want to start a GCC branch to share what we have so far and invite contributors.

Re: refinements to definition of TREE_READONLY ?

2008-07-11 Thread Eric Botcazou
> But it also tests TREE_STATIC which should not be true for automatic > variables. The binds_local_p check is only to guard against overriding the > symbol with an external definition. You're right, TREE_STATIC is the key here, in other words you're entitled to assume that !DECL_INITIAL => DECL_

Re: refinements to definition of TREE_READONLY ?

2008-07-11 Thread Richard Guenther
On Fri, Jul 11, 2008 at 8:17 PM, Eric Botcazou <[EMAIL PROTECTED]> wrote: >> For static storage yes, which it seems to test: >> >> else if (TREE_CODE (rhs) == VAR_DECL >>&& TREE_STATIC (rhs) >>&& TREE_READONLY (rhs) >>&& targetm.binds_local_p (r

Re: refinements to definition of TREE_READONLY ?

2008-07-11 Thread Eric Botcazou
> For static storage yes, which it seems to test: > > else if (TREE_CODE (rhs) == VAR_DECL >&& TREE_STATIC (rhs) >&& TREE_READONLY (rhs) >&& targetm.binds_local_p (rhs)) > fns->init (lhs_elt, DECL_INITIAL (rhs), bsi); I think there is a

Re: (new) Failure building GFortran (Cygwin)

2008-07-11 Thread Raksit Ashok
On Fri, Jul 11, 2008 at 1:09 AM, Angelo Graziosi <[EMAIL PROTECTED]> wrote: > Gabriel Dos Reis ha scritto: >> >> On Tue, Jul 8, 2008 at 3:48 AM, Angelo Graziosi >> <> wrote: >>> >>> Ian Lance Taylor ha scritto: This is OK, with a ChangeLog entry, if it passes bootstrap with the appro

Re: Recent libstdc++ regressions

2008-07-11 Thread Daniel Berlin
On Fri, Jul 11, 2008 at 1:17 PM, Paolo Carlini <[EMAIL PROTECTED]> wrote: > Hi, >> >> This is likely to have been my patch. >> I'm minimizing the check_construct_destroy failure right now. >> If someone could give me some idea of what is causing the execution >> failures while i do that, i may be a

Re: refinements to definition of TREE_READONLY ?

2008-07-11 Thread Richard Guenther
On Fri, Jul 11, 2008 at 7:22 PM, Olivier Hainque <[EMAIL PROTECTED]> wrote: > Richard Guenther wrote: >> > Don't set TREE_RO on the decl if !TREE_STATIC, in gigi (we might >> > decide it doesn't make sense and document accordingly). >> >> That would work. > > Both doing the thing in gigi and d

Re: refinements to definition of TREE_READONLY ?

2008-07-11 Thread Olivier Hainque
Richard Guenther wrote: > > Don't set TREE_RO on the decl if !TREE_STATIC, in gigi (we might > > decide it doesn't make sense and document accordingly). > > That would work. Both doing the thing in gigi and deciding that TREE_RO only makes sense if TREE_STATIC as well ? (I have no strong

Re: Recent libstdc++ regressions

2008-07-11 Thread Paolo Carlini
Hi, This is likely to have been my patch. I'm minimizing the check_construct_destroy failure right now. If someone could give me some idea of what is causing the execution failures while i do that, i may be able to fix them faster :) Thanks for fixing the check_construct_destroy problem. I w

Re: refinements to definition of TREE_READONLY ?

2008-07-11 Thread Richard Guenther
On Fri, Jul 11, 2008 at 6:49 PM, Olivier Hainque <[EMAIL PROTECTED]> wrote: > Richard Guenther wrote: >> A doc extension is ok. > > Understood, as soon as we agree on what it should say :) > >> > We'd also like to get to an agreement on what TREE_READONLY means on a >> > decl with non static stora

Re: refinements to definition of TREE_READONLY ?

2008-07-11 Thread Olivier Hainque
Richard Guenther wrote: > A doc extension is ok. Understood, as soon as we agree on what it should say :) > > We'd also like to get to an agreement on what TREE_READONLY means on a > > decl with non static storage, if anything at all. > > Good question... > > Ada front-end sets TREE_RO and D

Re: refinements to definition of TREE_READONLY ?

2008-07-11 Thread Richard Guenther
On Fri, Jul 11, 2008 at 6:15 PM, Olivier Hainque <[EMAIL PROTECTED]> wrote: > Hello, > > According to comments in PR/35493, when DECL_INITIAL is null on a > TREE_READONLY decl, the middle-end may assume the value to be zero. > > This is not explicit from the current definitions in tree.h, we'd > li

Re: refinements to definition of TREE_READONLY ?

2008-07-11 Thread Olivier Hainque
Olivier Hainque wrote: [...] While we're at it, it would be nice to settle similar notions about TREE_CONSTANT. There was an exchange along similar lines at http://gcc.gnu.org/ml/gcc/2005-08/msg00686.html

refinements to definition of TREE_READONLY ?

2008-07-11 Thread Olivier Hainque
Hello, According to comments in PR/35493, when DECL_INITIAL is null on a TREE_READONLY decl, the middle-end may assume the value to be zero. This is not explicit from the current definitions in tree.h, we'd like to understand if this is actually the case and to suggest a doc extension to this eff

RE: (new) Failure building GFortran (Cygwin)

2008-07-11 Thread Dave Korn
Angelo Graziosi wrote on 11 July 2008 09:09: > Gabriel Dos Reis ha scritto: >> >> OK. >> >> OK. > If the patch is OK, may someone apply/commit it so we can test it also > with the next 4.4 snapshot? If nobody else does it first, I could help out with this at the weekend (most likely sunday

Re: What may be the reason behind not implementing some instructions of H8SX targets in H8300 port?

2008-07-11 Thread DJ Delorie
"Deepen Mantri" <[EMAIL PROTECTED]> writes: > Does anybody know the reason behind not implementing these > instructions for H8SX targets? Most likely, it's just a case of nobody volunteering to add them yet. For some instructions, it may be difficult to teach GCC the nuances of how they work.

Re: create_tmp_var() fail to create variable for loop with switch statement

2008-07-11 Thread Fern Moon
Dear Richard Guenther, In my pass code, my printf code use ivvarincedloop from API. create_iv (build_int_cst_type (integer_type_node, 0), build_int_cst_type (integer_type_node, 1), ivvarloop, loop, &bsi, insert_after, &ivvarloop, &ivvarincedloop); How can I use the

Re: create_tmp_var() fail to create variable for loop with switch statement

2008-07-11 Thread Richard Guenther
On Fri, Jul 11, 2008 at 5:23 PM, Fern Moon <[EMAIL PROTECTED]> wrote: > Dear sir, > > I write a pass to instrument loops to count runtime number of iterations. > I insert a variable in each loop by create_tmp_var() and output > the results by inserting a printf function at the exit edge of loop. >

create_tmp_var() fail to create variable for loop with switch statement

2008-07-11 Thread Fern Moon
Dear sir, I write a pass to instrument loops to count runtime number of iterations. I insert a variable in each loop by create_tmp_var() and output the results by inserting a printf function at the exit edge of loop. The code is the attachment. The problem is that for the loops without switch sta

What may be the reason behind not implementing some instructions of H8SX targets in H8300 port?

2008-07-11 Thread Deepen Mantri
Hi, The following instructions of Renesas H8SX targets are not implemented in H8300 port, 1.Arithmetic instructions such as ADDS, SUBS, INC and DEC. 2.Arithmetic instructions that include operations on FLAG register such as ADDX and SUBX. 3.Logical instructions that include operations on FLAG re

Re: (new) Failure building GFortran (Cygwin)

2008-07-11 Thread Angelo Graziosi
Gabriel Dos Reis ha scritto: On Tue, Jul 8, 2008 at 3:48 AM, Angelo Graziosi <> wrote: Ian Lance Taylor ha scritto: This is OK, with a ChangeLog entry, if it passes bootstrap with the appropriate configure option. The following bootstraps rev. 137613, having configured as ${gcc_dir}/configur