[Bug other/43816] New: missed inline optimization for boost::shared_ptr::operator =

[Bug other/43816] New: missed inline optimization for boost::shared_ptr::operator =

[pluto at agmk dot net]

please consider .einline2 and .optimized trees for the
boost::thread::start_thread() function from attached testcase.

the 4.3 optimize boost::shared_ptr<>::operator= pretty nice.
the 4.5 avoid operator= inlining.

moreover, the 4.5 code is bigger (152bytes vs 106bytes)
and contains more atomic operations due to temporary shared_ptr
swapping and assignments.

the testcase is big and horrible but the 'pretty 4.3 inlining'
dissapears during testcase reducing :/


-- 
           Summary: missed inline optimization for
                    boost::shared_ptr::operator =
           Product: gcc
           Version: 4.5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: other
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: pluto at agmk dot net
 GCC build triplet: x86_64-gnu-linux
  GCC host triplet: x86_64-gnu-linux
GCC target triplet: x86_64-gnu-linux


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

[Bug other/43816] missed inline optimization for boost::shared_ptr::operator =

[pluto at agmk dot net]

------- Comment #1 from pluto at agmk dot net  2010-04-20 12:36 -------
Created an attachment (id=20442)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20442&action=view)
testcase


-- 


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

[Bug other/43816] missed inline optimization for boost::shared_ptr::operator =

[rguenth at gcc dot gnu dot org]

------- Comment #2 from rguenth at gcc dot gnu dot org  2010-04-20 12:56 -------
Well.  early inlining is then just lucky in 4.3 - the functions are really
too big for early inlining to inline fully.  Btw, IPA inlining does inline
operator= into start_thread.  In 4.3 we do not inline a destructor which
is why the 4.5 code is bigger.


-- 


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

[Bug other/43816] missed inline optimization for boost::shared_ptr::operator =

[pluto at agmk dot net]

------- Comment #3 from pluto at agmk dot net  2010-04-20 13:47 -------
so it's an invalid PR.


-- 

pluto at agmk dot net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |INVALID


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