[PATCH] D69437: [RAGreedy] Enable -consider-local-interval-cost by default

Sanne Wouda via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 28 09:21:20 PDT 2019


sanwou01 added a comment.

@jsji I have now committed the test case with -consider-local-interval-cost disabled, so the impact should be a bit more obvious now.

As Dave points out, it is practically impossible to predict where this behaviour would come up in other architectures, so finding test cases for it is not straightforward. As much as I'd like to add tests for the other platforms, I don't have any, nor a good way to find them!

@ZhangKang I'm looking forward to seeing your findings on PowerPC, thanks for running those!


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