[PATCH] D45274: [Polly] fixed test cases after removing immediate dominator heuristic for error block detection.
lorenzo chelini via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Apr 6 09:49:29 PDT 2018
chelini added a comment.
In https://reviews.llvm.org/D45274#1058592, @Meinersbur wrote:
> Thanks for the update. Can you run `make polly-update-format` on it (there seems to be some trailing whitespace)? The buildbots check the source code formatting and "fail" if some irregularity is found.
>
> For reviews in Polly:
>
> 1. Add "pollydev" and "llvm-commits" to the subscribers
> 2. Add the tag `[Polly]` in front of the title. (adding llvm-commits to subscribers triggers a mail to the corresponding mailing list. Adding the tag is useful s.t. one doesn't need to open the mail to see that it is a change in Polly)
> 3. Use "https://reviews.llvm.org/source/polly/" for repository.
I addressed the review comments. "pollydev" and "llvm-commits" were already added as subscribers. Could you please double-check?
Repository:
rPLO Polly
https://reviews.llvm.org/D45274
More information about the llvm-commits
mailing list