[polly] r295713 - [DeLICM] Map values hoisted by LICM back to the array.

[Michael Kruse via llvm-commits]

2017-02-21 13:50 GMT+01:00 Tobias Grosser <tobias.grosser at inf.ethz.ch>:
> Very cool. Thanks for putting this in!
>
> Btw, does it make sense to already enable this pass by default. As this
> is a conservative minimal implementation it seems perfect to already get
> test coverage from the start (also from our per-commit tests).

I don't think it makes sense to enable it by default atm. That would
mean that everybody who tries Polly would get a compile time increase,
potentially more crashes/miscompiles without gaining anything in
practice. It's experimental.

For getting coverage, we could set up a buildbot.

Michael