[Bug libstdc++/33293] inlining std::inner_product()

2007-09-04 Thread simon dot marshall at misys dot com


--- Comment #4 from simon dot marshall at misys dot com  2007-09-04 08:01 
---
Many thanks for the quick response.  Any chance it could make it into 4.2.2?


-- 


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



[Bug libstdc++/33293] inlining std::inner_product()

2007-09-04 Thread pcarlini at suse dot de


--- Comment #5 from pcarlini at suse dot de  2007-09-04 08:41 ---
The patch itself is trivial but in our experience changing the inlining
patterns can have far reaching and unpredictable fall-outs. Thus, better not
touching the release branches, sorry.


-- 


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



[Bug libstdc++/33293] inlining std::inner_product()

2007-09-04 Thread simon dot marshall at misys dot com


--- Comment #6 from simon dot marshall at misys dot com  2007-09-04 08:43 
---
OK, thanks.


-- 


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



[Bug libstdc++/33293] inlining std::inner_product()

2007-09-03 Thread pcarlini at suse dot de


--- Comment #1 from pcarlini at suse dot de  2007-09-03 17:20 ---
Note, in GCC any -Ox, x  3 is identical to -O3.

Anyway, I think we can safely add inline to std::accumulate and
std::inner_product.


-- 

pcarlini at suse dot de changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |pcarlini at suse dot de
   |dot org |
 Status|UNCONFIRMED |ASSIGNED
  Component|c++ |libstdc++
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2007-09-03 17:20:12
   date||


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



[Bug libstdc++/33293] inlining std::inner_product()

2007-09-03 Thread paolo at gcc dot gnu dot org


--- Comment #2 from paolo at gcc dot gnu dot org  2007-09-03 17:48 ---
Subject: Bug 33293

Author: paolo
Date: Mon Sep  3 17:48:31 2007
New Revision: 128053

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=128053
Log:
2007-09-03  Paolo Carlini  [EMAIL PROTECTED]

PR libstdc++/33293
* include/bits/stl_numeric.h (accumulate, inner_product):
Add inline function-specifier.

Modified:
trunk/libstdc++-v3/ChangeLog
trunk/libstdc++-v3/include/bits/stl_numeric.h


-- 


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



[Bug libstdc++/33293] inlining std::inner_product()

2007-09-03 Thread pcarlini at suse dot de


--- Comment #3 from pcarlini at suse dot de  2007-09-03 17:50 ---
Fixed for 4.3.0.


-- 

pcarlini at suse dot de changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED
   Target Milestone|--- |4.3.0


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