Skip to content
  • Chandler Carruth's avatar
    Flip the new block-placement pass to be on by default. · 4190b507
    Chandler Carruth authored
    This is mostly to test the waters. I'd like to get results from FNT
    build bots and other bots running on non-x86 platforms.
    
    This feature has been pretty heavily tested over the last few months by
    me, and it fixes several of the execution time regressions caused by the
    inlining work by preventing inlining decisions from radically impacting
    block layout.
    
    I've seen very large improvements in yacr2 and ackermann benchmarks,
    along with the expected noise across all of the benchmark suite whenever
    code layout changes. I've analyzed all of the regressions and fixed
    them, or found them to be impossible to fix. See my email to llvmdev for
    more details.
    
    I'd like for this to be in 3.1 as it complements the inliner changes,
    but if any failures are showing up or anyone has concerns, it is just
    a flag flip and so can be easily turned off.
    
    I'm switching it on tonight to try and get at least one run through
    various folks' performance suites in case SPEC or something else has
    serious issues with it. I'll watch bots and revert if anything shows up.
    
    llvm-svn: 154816
    4190b507
Loading