[PATCH] D27596: Add a PreRASplit pass to enable more shrinkwrap

Quentin Colombet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 9 14:36:57 PST 2016


qcolombet added a comment.

Hi Wei,

This goes against the paradigm used in LLVM, which is aggressive coalescing then split on demand.
If we start having pre-splitting heuristics on top of the existing heuristics I am afraid we are making our future life worse.

Instead I would rather improve the splitting heuristics. For instance, I would encourage to pursue the approach taken in https://reviews.llvm.org/D27366.

Cheers,
-Quentin


Repository:
  rL LLVM

https://reviews.llvm.org/D27596





More information about the llvm-commits mailing list