[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