[PATCH] D35458: [Polly][PM][WIP] Polly pass registration

[Tobias Grosser via Phabricator via llvm-commits]

grosser added a comment.

Hi Philip,

http://polly.llvm.org/docs/Architecture.html#polly-in-the-llvm-pass-pipeline discusses exactly why Polly is run at which position in the pass pipeline. We all seem to agree that the best position is to place it right before the vectorizer. We are just waiting for Michael's delicm to be finally enabled and upstreamed. We are very close, I believe. Maybe @Meinersbur can give an update on the timeline. Would it be possible to just take over the -polly-position command line option as we have it for the old pass manager. This will allow us to play with the different options.

Regarding the other small passes. The idea is that users can enable certain optimizations from the clang command line. E.g. our buildbots use commands such as 'clang -O3 -mllvm -polly -mllvm -polly-enable-simplify". The easiest way to add such options was for us to add/remove passes as needed. However, maybe there is a better design we could evolve towards?

Best,
Tobias


Repository:
  rL LLVM

https://reviews.llvm.org/D35458