[PATCH] D37571: [Polly][CodegenCleanup] Update cleanup passes according (old) PassManagerBuilder.

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 9 14:50:59 PDT 2017


Meinersbur added a comment.

In https://reviews.llvm.org/D37571#865787, @philip.pfaffe wrote:

> LGTM. It would be great to actually have a testcase for this, so we'd be able to see any changes earlier.


What do you think should be tested?

Testing end-to-end full-pipeline optimizations are purposefully not done in LLVM. These would be fragile tests that potentially break with any change in the cleanup passes.


Repository:
  rL LLVM

https://reviews.llvm.org/D37571





More information about the llvm-commits mailing list