[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