[PATCH] D41694: [Polly][ScopInfo] Remove RunTimeChecksMaxAccessDisjuncts bail-out condition.

Tobias Grosser via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 5 00:58:08 PST 2018


grosser added a comment.

I am afraid taking a convex hull might also be costly (we could take a simple/affine hull instead).

I feel that in general, we should still decide to bail out early (before potentially costly functions) rather than waiting for compute outs to trigger. So I don't see how we can get rid of magic constants. However, we can potentially just take a hull rather than erroring out here.


Repository:
  rPLO Polly

https://reviews.llvm.org/D41694





More information about the llvm-commits mailing list