[PATCH] D29169: Do not apply redundant LastCallToStaticBonus

Taewook Oh via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 27 14:28:06 PST 2017


twoh added a comment.

@eraman Thank you for the comments. I evaluated the size with spec2006 c/c++ benchmarks, and only 401.bzip2 is affected by this patch. Below is the result for -O2 and -O3. 'Binary' is for the entire binary size and 'Text' is for the text section size.

-O3 result:

                                       Binary                         Text
                     before      after   diff     before      after   diff
   400.perlbench  1,244,288  1,244,288  0.00%  1,124,296  1,124,296  0.00%
       401.bzip2     94,320     98,392  4.32%     77,951     81,247  4.23%
         403.gcc  3,832,632  3,832,632  0.00%  3,569,708  3,569,708  0.00%
         429.mcf     23,088     23,088  0.00%     13,682     13,682  0.00%
        433.milc    156,768    156,768  0.00%    132,977    132,977  0.00%
        444.namd    382,464    382,464  0.00%    343,740    343,740  0.00%
       445.gobmk  3,982,344  3,982,344  0.00%  1,467,536  1,467,536  0.00%
      447.dealII  5,003,664  5,003,664  0.00%  3,846,935  3,846,935  0.00%
      450.soplex  1,178,304  1,178,304  0.00%    873,994    873,994  0.00%
      453.povray  1,356,640  1,356,640  0.00%  1,167,752  1,167,752  0.00%
       456.hmmer    348,072    348,072  0.00%    312,659    312,659  0.00%
       458.sjeng    156,088    156,088  0.00%    132,903    132,903  0.00%
  462.libquantum     55,608     55,608  0.00%     43,694     43,694  0.00%
     464.h264ref    710,960    710,960  0.00%    642,200    642,200  0.00%
         470.lbm     22,392     22,392  0.00%     13,593     13,593  0.00%
     471.omnetpp  1,561,704  1,561,704  0.00%  1,136,964  1,136,964  0.00%
       473.astar    125,504    125,504  0.00%     96,129     96,129  0.00%
     482.sphinx3    214,280    214,280  0.00%    186,181    186,181  0.00%
   483.xalancbmk  6,977,136  6,977,136  0.00%  4,762,598  4,762,598  0.00%
    999.specrand      8,696      8,696  0.00%      1,870      1,870  0.00%

-O2 result:

                                       Binary                         Text
                     before      after   diff     before      after   diff
   400.perlbench  1,207,504  1,207,504  0.00%  1,085,608  1,085,608  0.00%
       401.bzip2     94,320     94,296 -0.03%     76,831     79,967  4.08%
         403.gcc  3,772,152  3,772,152  0.00%  3,508,612  3,508,612  0.00%
         429.mcf     23,088     23,088  0.00%     13,490     13,490  0.00%
        433.milc    148,576    148,576  0.00%    125,633    125,633  0.00%
        444.namd    382,464    382,464  0.00%    343,388    343,388  0.00%
       445.gobmk  3,953,920  3,953,920  0.00%  1,440,448  1,440,448  0.00%
      447.dealII  4,958,384  4,958,384  0.00%  3,795,567  3,795,567  0.00%
      450.soplex  1,170,464  1,170,464  0.00%    867,618    867,618  0.00%
      453.povray  1,335,872  1,335,872  0.00%  1,149,056  1,149,056  0.00%
       456.hmmer    340,088    340,088  0.00%    305,427    305,427  0.00%
       458.sjeng    156,088    156,088  0.00%    132,199    132,199  0.00%
  462.libquantum     55,608     55,608  0.00%     42,094     42,094  0.00%
     464.h264ref    686,464    686,464  0.00%    621,280    621,280  0.00%
         470.lbm     22,392     22,392  0.00%     13,177     13,177  0.00%
     471.omnetpp  1,558,112  1,558,112  0.00%  1,131,296  1,131,296  0.00%
       473.astar    125,504    125,504  0.00%     95,241     95,241  0.00%
     482.sphinx3    210,184    210,184  0.00%    182,541    182,541  0.00%
   483.xalancbmk  6,937,672  6,937,672  0.00%  4,710,754  4,710,754  0.00%
    999.specrand      8,696      8,696  0.00%      1,870      1,870  0.00%


https://reviews.llvm.org/D29169





More information about the llvm-commits mailing list