[PATCH] D45066: [Polly] [ScopInfo] Remove bail out condition in buildMinMaxAccess()

SAHIL GIRISH YERAWAR via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 20 11:35:09 PDT 2018


cs15btech11044 added a comment.

I think the given test case is sufficient enough for this fix. When I tried to run this  test case manually over opt, after applying `Set.simple_hull()`, it gave exactly the same reduced set as mentioned in the previous comment.

I will format this test case with appropriate check statements to ensure correct output is obtained. Also, this diff needs to be updated since I have been working on a version of Polly which was cloned on March 30th.

All these changes will be reflected in the next update.


Repository:
  rPLO Polly

https://reviews.llvm.org/D45066





More information about the llvm-commits mailing list