[polly] r244606 - Revise the simplification of regions

Michael Kruse via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 12 00:44:52 PDT 2015


2015-08-12 9:19 GMT+02:00 Michael Kruse <llvm at meinersbur.de>:
> I was also looking into solution number 2 since I was afraid 1) would
> break more things that it would fix. But your patch works well for
> pifft as well. Are you going to commit it? I will add a second test
> case for the pifft case.

Mixed up solution numbers:
I was looking in solution number 1) since I was afraid the others
would break more things than it would fix.

However, I prefer your solution number 2). I am not that familiar with
non-affine regions, but could the contains-check be removed entirely?


Michael


More information about the llvm-commits mailing list