[PATCH] D69437: [RAGreedy] Enable -consider-local-interval-cost by default
    Dave Green via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Oct 31 02:11:55 PDT 2019
    
    
  
dmgreen added a comment.
Hmm. That's interesting. I'm guessing the flags you use between base and peak are quite different? The only consistent results was exchange2_r, which was getting better! How noisy are these results? What kind of a confidence interval do you have?
It may be simpler to make this AArch64 only instead, same as the X86 backend previously had. We know that there it fixes the issue with repeatedly spilling registers, and the performance is otherwise not largely effected.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69437/new/
https://reviews.llvm.org/D69437
    
    
More information about the llvm-commits
mailing list