[PATCH] D87328: [Polly][NewPM] Port Simplify to the new pass manager
Pengxuan Zheng via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Sep 20 19:05:29 PDT 2020
pzheng added a comment.
In D87328#2284308 <https://reviews.llvm.org/D87328#2284308>, @Meinersbur wrote:
> LGTM
>
> A single test test file would be enough to check that the new pass manager pipeline is working. Checking all tests with both pass manager doubles the execution time for `check-polly` without improving test coverage. There are not too many tests for the simplify pass, but please consider testing it only once with the scalability tests, as these take the most time.
That makes sense. I will revert changes in the scalability tests when committing.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87328/new/
https://reviews.llvm.org/D87328
More information about the llvm-commits
mailing list