• bratell@opera.com's avatar
    Save some gcc footprint (15 KB) in ExplodedFrameState. · 51aa9fb7
    bratell@opera.com authored
    Inlining the assignment and copy constructor for a class
    like ExplodedFrameState exploded the machine code in stl::vector.
    
    gcc:
    Total change: -14969 bytes
    ==========================
      2 added, totalling +504 bytes across 1 sources
      1 removed, totalling -3387 bytes across 1 sources
      0 added, totalling 0 bytes across 0 sources
      3 shrunk, for a net change of -12086 bytes (14716 bytes before, 2630 bytes after) across 2 sources
    
    It also helps for clang which normally ignores programmers hints to inline anyway:
    clang:
    Total change: -918 bytes
    ========================
      1 added, totalling +263 bytes across 1 sources
      3 removed, totalling -1098 bytes across 2 sources
      2 grown, for a net change of +161 bytes (1822 bytes before, 1983 bytes after) across 1 sources
      1 shrunk, for a net change of -244 bytes (1401 bytes before, 1157 bytes after) across 1 sources
    
    BUG=
    
    Review URL: https://codereview.chromium.org/400923002
    
    git-svn-id: svn://svn.chromium.org/chrome/trunk/src@284424 0039d316-1c4b-4281-b951-d872f2087c98
    51aa9fb7
page_state_serialization.h 2.57 KB