This is the mail archive of the gcc-bugs@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug libstdc++/14078] Manipulators are slow


------- Additional Comments From gdr at integrable-solutions dot net  2004-02-10 17:03 -------
Subject: Re:  Manipulators are slow

"pcarlini at suse dot de" <gcc-bugzilla@gcc.gnu.org> writes:

| ------- Additional Comments From pcarlini at suse dot de  2004-02-10 16:09 -------
| > The only reason I suggested that the argument should be declared const is
| > that it seems to make a difference in practice (in fact, about 44%). I
| > don't know why it does though, the compiler should be able to figure out
| > on it's own that pf isn't modified in the body of the function.
| 
| These are well known deficiencies of the inliner
| 
|   http://gcc.gnu.org/ml/libstdc++/2003-02/msg00302.html
| 
| which, I'm told, should be already fixed "automatically" in the tree-ssa
| branch. For this reason, I'd rather prefer not adding the "redundant" const
| in the signature, even if (thanks for pointing that out!) technically it would
| still be conforming. Anyway, I'm going to add to the performance testsuite a
| snippet distilled from your manipspeed, so we'll be able to track the issue in
| the future. Thanks again for your report.

The operator should however be declared inline.

-- Gaby


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14078


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]