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

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 16 14:35:34 PDT 2018


Meinersbur added a comment.

LGTM.

Adding @grosser as reviewer to confirm whether this satisfies his concerns in https://reviews.llvm.org/D41694.

Usually we add a test-case to behaviour-changing commits. However, IMHO this is more a cleanup-up and a test-case difficult to produce.


https://reviews.llvm.org/D45066





More information about the llvm-commits mailing list